Skip to content
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

Display Openshift route #164

Merged
merged 7 commits into from
Dec 16, 2019
Merged

Display Openshift route #164

merged 7 commits into from
Dec 16, 2019

Conversation

cdcabrera
Copy link
Member

@cdcabrera cdcabrera commented Dec 16, 2019

What's included

  • fix(rhsmApiTypes): issues/161 expand rhsm types
    • rhsmApiTypes, expanded rhsm response types
    • routerTypes, update product ids
    • rhelGraphCard, update default product id, rhsm types
    • rhelGraphCardHelpers, expanded rhsm types for tooltips
    • graphCardSelectors, expand towards additional rhsm types
  • fix(locales): align keys to rhsm types
  • fix(rhelGraphCardHelpers): issues/161 generate tooltips
    • generate tooltips vs hard coded
    • expand switch granularity defaults
    • no data display
  • fix(rhelGraphCardTypes): issues/161 pass param optionsType
    • convert towards generic select type
    • pass granularity select options type
  • fix(rhelView): issues/161 pass graph card options
    • rhelView, pass props, graph card options, filter date
    • rhelGraphCard, props offset towards parent component
  • fix(graphCard,helpers,types): issues/161 directory, file rename
  • feat(openShiftView,rhelView): issues/157 openshift route (Display Openshift route #164)
    • routerTypes, apply openshift view, update route detail filter
    • router, expand passed values to getRouteDetail
    • rhelView, apply translation for titles
    • openshiftView, translation, mirror rhelView, filter core values
    • graphCard, prop for cardTitle
    • i18n, locale, heading, title strings

Notes

  • To access the Openshift route we'll need to temporarily enter it manually for testing. An update for the "source of truth" platform config is required to have it display in the left navigation.

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. manually enter the route /beta/subscriptions/openshift-sw and confirm that cores related content is being displayed, this includes:
    • card title
    • graph/chart legend

Example

Screen Shot 2019-12-15 at 11 34 37 PM

Updates issue/story

Closes #157 #161

* rhsmApiTypes, expanded rhsm response types
* routerTypes, update product ids
* rhelGraphCard, update default product id, rhsm types
* rhelGraphCardHelpers, expanded rhsm types for tooltips
* graphCardSelectors, expand towards additional rhsm types
* generate tooltips vs hard coded
* expand switch granularity defaults
* no data display
* convert towards generic select type
* pass granularity select options type
* rhelView, pass props, graph card options, filter date
* rhelGraphCard, props offset towards parent component
cdcabrera added a commit to cdcabrera/curiosity-frontend that referenced this pull request Dec 16, 2019
…hts#164)

* routerTypes, apply openshift view, update route detail filter
* router, expand passed values to getRouteDetail
* rhelView, apply translation for titles
* openshiftView, translation, mirror rhelView, filter core values
* graphCard, prop for cardTitle
* i18n, locale, heading, title strings
@cdcabrera cdcabrera changed the title WIP Display Openshift route Display Openshift route Dec 16, 2019
@codecov-io
Copy link

codecov-io commented Dec 16, 2019

Codecov Report

Merging #164 into ci will decrease coverage by 0.15%.
The diff coverage is 89.83%.

Impacted file tree graph

@@            Coverage Diff            @@
##              ci     #164      +/-   ##
=========================================
- Coverage   90.6%   90.45%   -0.16%     
=========================================
  Files         28       29       +1     
  Lines        596      618      +22     
  Branches     156      156              
=========================================
+ Hits         540      559      +19     
- Misses        47       50       +3     
  Partials       9        9
Impacted Files Coverage Δ
src/components/router/router.js 82.35% <0%> (ø) ⬆️
src/types/rhsmApiTypes.js 100% <100%> (ø) ⬆️
src/components/graphCard/graphCardTypes.js 100% <100%> (ø)
src/redux/selectors/graphCardSelectors.js 100% <100%> (ø) ⬆️
src/components/router/routerTypes.js 100% <100%> (ø) ⬆️
src/components/rhelView/rhelView.js 100% <100%> (ø) ⬆️
src/components/graphCard/graphCardHelpers.js 100% <100%> (ø)
src/components/openshiftView/openshiftView.js 100% <100%> (ø)
src/components/graphCard/graphCard.js 69.56% <73.68%> (ø)
... and 1 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 26110ca...a2ceeef. Read the comment docs.

cdcabrera added a commit to cdcabrera/curiosity-frontend that referenced this pull request Dec 16, 2019
…hts#164)

* routerTypes, apply openshift view, update route detail filter
* router, expand passed values to getRouteDetail
* rhelView, apply translation for titles
* openshiftView, translation, mirror rhelView, filter core values
* graphCard, prop for cardTitle
* i18n, locale, heading, title strings
…hts#164)

* routerTypes, apply openshift view, update route detail filter
* router, expand passed values to getRouteDetail
* rhelView, apply translation for titles
* openshiftView, translation, mirror rhelView, filter core values
* graphCard, prop for cardTitle
* i18n, locale, heading, title strings
@cdcabrera cdcabrera merged commit 0b5be21 into RedHatInsights:ci Dec 16, 2019
cdcabrera added a commit that referenced this pull request Dec 19, 2019
* routerTypes, apply openshift view, update route detail filter
* router, expand passed values to getRouteDetail
* rhelView, apply translation for titles
* openshiftView, translation, mirror rhelView, filter core values
* graphCard, prop for cardTitle
* i18n, locale, heading, title strings
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants