Skip to content

PROD-3115 Integrate Universal Nav -> dev #437

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

Merged
merged 37 commits into from
Dec 10, 2022
Merged

Conversation

brooketopcoder
Copy link
Contributor

@brooketopcoder brooketopcoder commented Dec 9, 2022

https://topcoder.atlassian.net/browse/PROD-3115

Initial integration of the universal nav.

This PR also includes some lint fixes and a refactor of the platform routes in order to identify the current tool.

brooketopcoder and others added 30 commits November 18, 2022 15:04
…v outside the scope of the Header component. #time 15m
PROD-3232 Adds the uni nav snippet -> PROD-3115_uni-nav
PROD-3233 Set the Tool Name to Active Route Name ->PROD-3115_uni-nav
…the loading spinner until the header is ready. It adds the url to the tool landing page. #time 2h
PROD-3234 Add Spinner while Header Loads -> PROD-3115_uni-nav
PROD-3246 update user when profile is ready -> PROD-3115_uni-nav
…fic top nav. It also adds IDs to the routes so we can ID them separately from the title and the titles don't need to be unique. Finally, it sets the self-service tool to use the same tool name for all its pages. #time 2h
…ation

PROD-3115 handle navigation -> PROD-3115_uni-nav
Prod 3245 Integrate Self-Service Challenges, Support, Settings, Footer -> TCA-3115_uni-nav
PROD-3245 README & Version 1 -> PROD-3115_uni-nav
@brooketopcoder brooketopcoder self-assigned this Dec 9, 2022
@brooketopcoder brooketopcoder merged commit af3fa18 into dev Dec 10, 2022
@brooketopcoder brooketopcoder deleted the PROD-3115_uni-nav branch December 10, 2022 19:04
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