Skip to content

81 Hours-only graphs

DO NOT MERGE WORK INTO MASTER FOR THIS FEATURE UNTIL OTHER FEATURE COMPLETES: #87 (closed)

What's included

  • fix(accountGraphCard): 81 hours only graphs
    • build, adjusted coverage include common files, raise mins
    • graphHelpers, removed combined graph data, renamed method
    • graphHelpers, unit test & snapshots with mock data
    • redux, adjust hour graphs, reducers, constants, actions
    • redux, reduce snapshots for redux reducers
    • add accountGraphCard component, unit tests
    • updated accountView, accountImagesView to use accountGraphCard
    • remove old accountView, accountImagesView graph components
    • accountService annotation for testing graphHelpers
  • fix(unit tests): accountWizard, accountEdit reducers
    • reduce snapshot tests for accountWizard, accountEdit reducers
  • fix(helper): serverStatus messaging, about modal sha check

    • serverStatus messaging display check
    • minor rename for displaying SHAs for UI & API within about modal
  • chore(readme): yarn description, snapshots, code coverage

    • installing yarn description
    • running snapshot updates
    • checking code coverage
    • update gitlab issue and merge templates

How to test

The QE review setup can be used to help a quick confirmation. You'll need a login, or access to a Cloudigrade instance.

  1. Clone the updates locally, then
  2. From the terminal, within the repo context, run $ yarn start:review
  3. You may need to log in, do so

The mock dev setup can be used to help a quick confirmation.

  1. Clone the updates locally, then
  2. From the terminal, within the repo context, run $ yarn start
  3. You may need to log in, do so
    • Note: the local dev server will take any combo of email/made up password to log in

Example

  • fix(accountGraphCard): 81 hours only graphs (i.e. graphs have been reduced to 2 and now use time instead of instances)

NEW Screen_Shot_2018-10-15_at_12.02.13_PM

Screen_Shot_2018-10-15_at_12.02.31_PM

CURRENT

Updates issue/story

Updates #81 (closed)

Edited by Brad Smith

Merge request reports