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

feat(openshiftView): issues/185 add cores sockets selector #205

Merged
merged 1 commit into from
Feb 13, 2020

Conversation

cdcabrera
Copy link
Member

@cdcabrera cdcabrera commented Feb 13, 2020

What's included

  • feat(openshiftView): issues/185 add cores sockets selector
    • locales, add generic cardHeading string
    • graphCard, allow children to populate card actions
    • openshiftView, add cores sockets select list for graph filtering
    • rhelView, code style consistency against openshiftView

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 a user that has related data
  5. navigate to the Openshift path and confirm the select list filters the displayed graph data

Example

Screen Shot 2020-02-13 at 2 16 42 PM

Screen Shot 2020-02-13 at 2 16 52 PM

Updates issue/story

Close #185

cdcabrera added a commit to cdcabrera/curiosity-frontend that referenced this pull request Feb 13, 2020
…ights#205)

* locales, add generic cardHeading string
* graphCard, allow children to populate card actions
* openshiftView, add cores sockets select list for graph filtering
* rhelView, code style consistency against openshiftView
…ights#205)

* locales, add generic cardHeading string
* graphCard, allow children to populate card actions
* openshiftView, add cores sockets select list for graph filtering
* rhelView, code style consistency against openshiftView
@cdcabrera cdcabrera changed the title WIP feat(openshiftView): issues/185 add cores sockets selector feat(openshiftView): issues/185 add cores sockets selector Feb 13, 2020
@cdcabrera cdcabrera requested a review from bclarhk February 13, 2020 18:29
@codecov-io
Copy link

Codecov Report

Merging #205 into ci will increase coverage by 0.18%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##               ci     #205      +/-   ##
==========================================
+ Coverage   90.51%   90.69%   +0.18%     
==========================================
  Files          38       38              
  Lines         759      774      +15     
  Branches      189      192       +3     
==========================================
+ Hits          687      702      +15     
  Misses         67       67              
  Partials        5        5
Impacted Files Coverage Δ
src/components/rhelView/rhelView.js 100% <100%> (ø) ⬆️
src/components/openshiftView/openshiftView.js 100% <100%> (ø) ⬆️
src/components/graphCard/graphCard.js 67.3% <100%> (ø) ⬆️

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 319050a...3a5c4b4. Read the comment docs.

@cdcabrera cdcabrera merged commit a713c9e into RedHatInsights:ci Feb 13, 2020
@bclarhk
Copy link

bclarhk commented Feb 13, 2020

LGTM!

cdcabrera added a commit that referenced this pull request Feb 25, 2020
* locales, add generic cardHeading string
* graphCard, allow children to populate card actions
* openshiftView, add cores sockets select list for graph filtering
* rhelView, code style consistency against openshiftView
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.

3 participants