Skip to content

91 Add flag icon to accounts list

CD Cabrera requested to merge 91-accounts-flag into master

What's included

  • feat(accountViewListItem, accountImagesViewListItem): 91 flag icon to accounts list

    • accountViewListItem flag icon added, tests updated
    • accountImagesViewListItem logic for consistency, test updated
    • apiConstants added, test updated
    • accountServices api annotations
  • chore(oc setup): remove local run oc references

    • update README local run links
    • remove non-functional oc references

How to test

The 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

Accounts view

Screen_Shot_2018-12-04_at_5.21.28_PM

Images view

Labels no longer gray when challenged Screen_Shot_2018-12-04_at_5.23.12_PM

Updates issue/story

updates #91 (closed)

Edited by CD Cabrera

Merge request reports