Skip to content

87 Display RHEL and OCP hours for each account & 81 hours only graphs

CD Cabrera requested to merge 87-account-hours into master

What's included

87 Display RHEL and OCP hours for each account
  • fix(accountView): 87 RHEL and OCP hours for account list
    • accountServices mock update to include rhel ocp seconds
    • api types added for rhel ocp api response properties, snapshot update
    • css for account list
    • accountViewListItem component hours display, test data & snapshots
    • accountViewListItem hour, seconds copy updates for popover, tooltips
  • fix(graphHelpers): 80 rounding for hours
    • test snapshot update related to issue 80 rounding hours
81 hours only graphs
  • 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

Notes

  • this is a combination merge for two related issues
  • 81 hours only graphs commits have already been UI developer reviewed, see !92 (merged)

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

87 Display RHEL and OCP hours for each account

Screen_Shot_2018-10-25_at_10.36.49_AM

Screen_Shot_2018-10-25_at_10.42.06_AM

Oct-30-2018_14-12-23

81 hours only graphs

Screen_Shot_2018-10-15_at_12.02.13_PM

Updates issue/story

Updates #87 (closed) #81 (closed)

Edited by CD Cabrera

Merge request reports