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(productViewMissing): ent-3753 missing product view #630

Merged
merged 1 commit into from
Apr 28, 2021

Conversation

cdcabrera
Copy link
Member

@cdcabrera cdcabrera commented Apr 22, 2021

What's included

  • feat(productViewMissing): apply a missing product view
    • i18n, missing product view strings, product titles
    • pageLayout, allow additional classNames
    • routerConfig, fallback root route, add nav filter prop, isSearchable
    • routerHelpers, basePath, consistency with baseName
    • styling, add product view styling for missing view, clean up

Notes

  • Utilizes the Curiosity navigation and routing config to display relevant product cards based on the individual directory/path facets within window.pathname
    • Curiosity's attempt to match a path in the form of /beta/insights/subscriptions or /rhel/subscriptions or /random-path/openshift/subscriptions will attempt a match on each path/dir segment, ie. for...
      • /beta/insights/subscriptions that would be "beta" and "insights",
      • /rhel/subscriptions that would be "rhel"
      • /random-path/openshift/subscriptions that would be "random-path" and "openshift"
    • If a match isn't made Curiosity will return ALL cards on searchable/listable products
      • * Note, "insights" is a reference towards the Cloud Dot platform, a non-graphable-all-encompassing-product, "insights" will NOT bring up RHEL products. When "insights" is converted towards "RHEL" it will bring up Curiosity's RHEL product card(s)

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. navigate towards /beta/insights/subscriptions or /beta/openshift/subscriptions

Example

Screen Shot 2021-04-22 at 1 18 27 AM

Updates issue/story

Relates towards RHCLOUD-13632
Relates ent-3753

@cdcabrera cdcabrera added 202104 project phase platform Contains, or is, platform specific work and issues labels Apr 22, 2021
@cdcabrera cdcabrera force-pushed the ci branch 3 times, most recently from 6cc546d to a3a563c Compare April 26, 2021 13:17
@cdcabrera cdcabrera force-pushed the ci-fallback-view branch 2 times, most recently from 421f415 to 8660425 Compare April 28, 2021 19:30
@cdcabrera cdcabrera marked this pull request as ready for review April 28, 2021 19:31
@cdcabrera cdcabrera changed the title feat(productViewMissing): apply a missing product view feat(productViewMissing): ent-3753 missing product view Apr 28, 2021
* i18n, missing product view strings, product titles
* pageLayout, allow additional classNames
* routerConfig, fallback root route, add nav filter prop, isSearchable
* routerHelpers, basePath, consistency with baseName
* styling, add product view styling for missing view, clean up
@cdcabrera cdcabrera merged commit cf49ba0 into RedHatInsights:ci Apr 28, 2021
cdcabrera added a commit that referenced this pull request Apr 29, 2021
* i18n, missing product view strings, product titles
* pageLayout, allow additional classNames
* productViewMissing, display products and nav
* routerConfig, fallback root route, add nav filter prop, isSearchable
* routerHelpers, basePath, consistency with baseName
* styling, add product view styling for missing view, clean up
cdcabrera added a commit that referenced this pull request May 12, 2021
* i18n, missing product view strings, product titles
* pageLayout, allow additional classNames
* productViewMissing, display products and nav
* routerConfig, fallback root route, add nav filter prop, isSearchable
* routerHelpers, basePath, consistency with baseName
* styling, add product view styling for missing view, clean up
cdcabrera added a commit that referenced this pull request May 18, 2021
* i18n, missing product view strings, product titles
* pageLayout, allow additional classNames
* productViewMissing, display products and nav
* routerConfig, fallback root route, add nav filter prop, isSearchable
* routerHelpers, basePath, consistency with baseName
* styling, add product view styling for missing view, clean up
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
202104 project phase platform Contains, or is, platform specific work and issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant