-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(bannerMessages): issues/502 cloud meter messaging #503
Conversation
19ed943
to
0c8ac95
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It was also noted there may be some iterative changes
- making the Cloud Meter mismatch banner ONLY display using the most recent day/last 2 days. Unclear whether this would help reset the banner if the end user was able to correct their setup
- making the banner message remember it has been closed. multiple options/levels of remembering
- at the Redux state level where it resets on product changes, but attempts to maintain being closed across architecture navigation
- at a localStorage, sessionStorage level... sessionStorage resetting if the user closes the browser tab
- at a cookie level, possibly set an expiration, resetting when a user resets their cookies
0c8ac95
to
a53b336
Compare
Codecov Report
@@ Coverage Diff @@
## ci #503 +/- ##
==========================================
+ Coverage 93.13% 93.27% +0.13%
==========================================
Files 70 73 +3
Lines 2301 2348 +47
Branches 780 792 +12
==========================================
+ Hits 2143 2190 +47
Misses 139 139
Partials 19 19
Continue to review full report at Codecov.
|
* styling, banner messages spacing * appMessagesSelectors, base selector * bannerMessages, display app banner messages * rhelView, openshiftView, apply bannerMessages display
…s#503) * build, dotenv flag for recommendations link * locale, cloud meter related banner message strings * appMessagesSelectors, apply tally cloud meter data, mismatch * bannerMessages, cloud meter mismatch message * graphCardSelectors, expose cloud meter flags * helpers, ui link for recommendations * rhsmApiTypes, cloud meter data and mismatch flags * rhsmServices, mock update for cloud meter data
a53b336
to
78d31ed
Compare
…nsights#503) * build, dotenv flag for recommendations link * locale, cloud meter related banner message strings * appMessagesSelectors, apply tally cloud meter data, mismatch * bannerMessages, cloud meter mismatch message * graphCardSelectors, expose cloud meter flags * helpers, ui link for recommendations * rhsmApiTypes, cloud meter data and mismatch flags * rhsmServices, mock update for cloud meter data
78d31ed
to
3db6705
Compare
…s#503) * build, dotenv flag for recommendations link * locale, cloud meter related banner message strings * appMessagesSelectors, apply tally cloud meter data, mismatch * bannerMessages, cloud meter mismatch message * graphCardSelectors, expose cloud meter flags * helpers, ui link for recommendations * rhsmApiTypes, cloud meter data and mismatch flags * rhsmServices, mock update for cloud meter data
3db6705
to
5ac7361
Compare
* pageLayout, allow pageMessages * pageMessages, messaging display * pageToolbar, alignment with pageMessages
* styling, banner messages spacing * appMessagesSelectors, base selector * bannerMessages, display app banner messages * rhelView, openshiftView, apply bannerMessages display
* pageLayout, allow pageMessages * pageMessages, messaging display * pageToolbar, alignment with pageMessages
* styling, banner messages spacing * appMessagesSelectors, base selector * bannerMessages, display app banner messages * rhelView, openshiftView, apply bannerMessages display
* build, dotenv flag for recommendations link * locale, cloud meter related banner message strings * appMessagesSelectors, apply tally cloud meter data, mismatch * bannerMessages, cloud meter mismatch message * graphCardSelectors, expose cloud meter flags * helpers, ui link for recommendations * rhsmApiTypes, cloud meter data and mismatch flags * rhsmServices, mock update for cloud meter data
* pageLayout, allow pageMessages * pageMessages, messaging display * pageToolbar, alignment with pageMessages
* styling, banner messages spacing * appMessagesSelectors, base selector * bannerMessages, display app banner messages * rhelView, openshiftView, apply bannerMessages display
* build, dotenv flag for recommendations link * locale, cloud meter related banner message strings * appMessagesSelectors, apply tally cloud meter data, mismatch * bannerMessages, cloud meter mismatch message * graphCardSelectors, expose cloud meter flags * helpers, ui link for recommendations * rhsmApiTypes, cloud meter data and mismatch flags * rhsmServices, mock update for cloud meter data
Found blocker issue with backend https://issues.redhat.com/browse/ENT-3275 .. will verify this once it is resolved. |
* pageLayout, allow pageMessages * pageMessages, messaging display * pageToolbar, alignment with pageMessages
* styling, banner messages spacing * appMessagesSelectors, base selector * bannerMessages, display app banner messages * rhelView, openshiftView, apply bannerMessages display
* build, dotenv flag for recommendations link * locale, cloud meter related banner message strings * appMessagesSelectors, apply tally cloud meter data, mismatch * bannerMessages, cloud meter mismatch message * graphCardSelectors, expose cloud meter flags * helpers, ui link for recommendations * rhsmApiTypes, cloud meter data and mismatch flags * rhsmServices, mock update for cloud meter data
* pageLayout, allow pageMessages * pageMessages, messaging display * pageToolbar, alignment with pageMessages
* styling, banner messages spacing * appMessagesSelectors, base selector * bannerMessages, display app banner messages * rhelView, openshiftView, apply bannerMessages display
* build, dotenv flag for recommendations link * locale, cloud meter related banner message strings * appMessagesSelectors, apply tally cloud meter data, mismatch * bannerMessages, cloud meter mismatch message * graphCardSelectors, expose cloud meter flags * helpers, ui link for recommendations * rhsmApiTypes, cloud meter data and mismatch flags * rhsmServices, mock update for cloud meter data
* pageLayout, allow pageMessages * pageMessages, messaging display * pageToolbar, alignment with pageMessages
* styling, banner messages spacing * appMessagesSelectors, base selector * bannerMessages, display app banner messages * rhelView, openshiftView, apply bannerMessages display
* build, dotenv flag for recommendations link * locale, cloud meter related banner message strings * appMessagesSelectors, apply tally cloud meter data, mismatch * bannerMessages, cloud meter mismatch message * graphCardSelectors, expose cloud meter flags * helpers, ui link for recommendations * rhsmApiTypes, cloud meter data and mismatch flags * rhsmServices, mock update for cloud meter data
What's included
Notes
@jlprevatt @ntkathole @bclarhk @rblackbu
has_cloudigrade_mismatch
flag istrue
on any one data point within the Tally API endpointhas_cloudigrade_mismatch
flag isfalse
across all data points on a "granularity". Subsequent granularity selections may fire the banner and display ithas_cloudigrade_mismatch
flag, the banner will stay on.has_cloudigrade_mismatch
flag istrue
How to test
Proxy run check
$ yarn
$ yarn start:proxy
Example
With link
Without the link
Behavior
Includes closing the banner
Updates issue/story
#502