fix(build): issues/502 add recommendations link

[cdcabrera]

What's included

• fix(build): issues/502 add recommendations link

Notes

• Activates the dotenv link for the banner recommendation
• @rblackbu @bclarhk

How to test

Proxy run check

1. update the NPM packages with $ yarn
2. make sure Docker is running, plus on network, then
3. $ yarn start:proxy
4. log in with an account that has mismatched Cloud Meter data, expecting the banner to be displayed
5. click on the link and confirm it navigates towards https://access.redhat.com/subscription-watch-accuracy

Example

Updates issue/story

#502

[cdcabrera]

As of 20201116 temporarily applying the blocked label to this since the link currently doesn't go anywhere.

[codecov-io]

Codecov Report

Merging #508 (6cad86d) into qa (e2b0a9c) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##               qa     #508      +/-   ##
==========================================
- Coverage   93.47%   93.44%   -0.04%     
==========================================
  Files          74       74              
  Lines        1963     2379     +416     
  Branches      496      813     +317     
==========================================
+ Hits         1835     2223     +388     
- Misses        110      138      +28     
  Partials       18       18              

Impacted Files	Coverage Δ
src/components/router/redirect.js	65.00% <0.00%> (-5.59%)	⬇️
src/components/router/routerHelpers.js	91.42% <0.00%> (-5.45%)	⬇️
src/redux/selectors/inventoryListSelectors.js	97.56% <0.00%> (-2.44%)	⬇️
src/components/inventoryList/inventoryList.js	90.90% <0.00%> (-2.43%)	⬇️
src/services/platformServices.js	97.67% <0.00%> (-2.33%)	⬇️
src/components/rhelView/rhelView.js	90.69% <0.00%> (-1.90%)	⬇️
src/components/openshiftView/openshiftView.js	93.93% <0.00%> (-1.90%)	⬇️
src/redux/selectors/graphCardSelectors.js	98.21% <0.00%> (-1.79%)	⬇️
src/components/graphCard/graphCardHelpers.js	98.61% <0.00%> (-1.39%)	⬇️
src/components/c3GraphCard/c3GraphCardHelpers.js	75.58% <0.00%> (-1.35%)	⬇️
... and 37 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e2b0a9c...6cad86d. Read the comment docs.

[bclarhk]

We'll likely edit the text once we have a writer with a working computer. ;-)

[rblackbu]

Copy update proposal prior to activating the link.

Title: Cloud provider mismatch
Body: There may be inconsistencies between data in the graph and your inventory display. The issue is currently being resolved.

[cdcabrera]

As of 20201202 we were expecting the copy and link to be active, since the link is not we'll go ahead and direct this PR towards the CI branch

The base branch was changed.

[cdcabrera]

Per @jlprevatt updates to copy requested

Original:

There may be inconsistencies between data in the graph and your inventory display. The issue is currently being resolved.

Change 1

"There may be" > "There might be" (Reason: "may" can mean "can" or "might" and the meaning can be misunderstood by ESL folks. Always use the more precise word, "can" or "might")

Change 2

It would be better to more closely match the terms used in the GUI/doc for the graph and table. (The doc uses "usage and utilization graph" and shortens to "the graph" so I don't mind "graph" so much. But instead of "inventory" we should be using "current systems table.") So "...between the data in the graph and the current systems table."

Additional comment

I had it pounded into my head to never promise the user a fix in writing. But if [the team approved this already], then I'll let it be.

[cdcabrera]

@ntkathole we're moving forward with getting this into QA in prep for an incremental push to prod-beta

[ntkathole]

Per @jlprevatt updates to copy requested

Original:

There may be inconsistencies between data in the graph and your inventory display. The issue is currently being resolved.

Change 1

"There may be" > "There might be" (Reason: "may" can mean "can" or "might" and the meaning can be misunderstood by ESL folks. Always use the more precise word, "can" or "might")

@jlprevatt this change should go into the document as well. Currently it say "There may be inconsistencies between data in the graph and your inventory display".

[ntkathole]

@cdcabrera

• There is typo in message "recommmend"

• We are seeing only "View recommended actions to take in order to improve Subscription Watch reporting." message. The other message is not visible.
  "There might be inconsistencies between data in the graph and your current systems table. The issue is currently being resolved."
  can you please confirm if following banner is expected ?

[jlprevatt]

@ntkathole Regarding your direct comment to me above, I was initially confused and thought you meant the product doc, but I think you mean the draft of the customer portal article. Yes, if the customer portal article directly quotes the on-screen messaging, the exact text of the message should appear there.

[cdcabrera]

Looks like we missed your ask @ntkathole ...

can you please confirm if following banner is expected ?

That is correct. The more verbose banner is the fallback for when the message has no link available. Now that the link is active this is expected