-
Notifications
You must be signed in to change notification settings - Fork 19
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
fix(routerHelpers): dynamic route, handle additional directories #600
Conversation
090e3f8
to
01a72c2
Compare
Marking this update blocked until we can get a more specific response regarding the decision to use the paths
Currently the baseName and app identifiers are all based off of the path containing Under the assumption that the platform left nav still requires coordination from the GUI...
|
01a72c2
to
e23e715
Compare
We have some additional routing updates that need to be performed to work with the platform navigation updates. We're considering adding to this PR or a new PR. They are...
The old routes are dependent on the prior way of navigating and grouping and should probably be removed, we'll need to evaluate the navigation updates to determine their removal, but that can be in a follow up PR. Relates RHCLOUD-12690 Coordinated config updates are here, RedHatInsights/cloud-services-config#446 |
e23e715
to
99c1a8b
Compare
99c1a8b
to
d8abb67
Compare
d8abb67
to
6fff5e4
Compare
What's included
Notes
How to test
Coverage and basic unit test check
$ yarn
$ yarn test
Example
...
Updates issue/story
Ongoing