Skip to content

TCA-838 Uni Nav Release - 2022-12-19 -> master #448

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 64 commits into from
Dec 19, 2022
Merged

TCA-838 Uni Nav Release - 2022-12-19 -> master #448

merged 64 commits into from
Dec 19, 2022

Conversation

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
vas3a and others added 19 commits December 9, 2022 23:03
PROD-3115 Integrate Universal Nav -> dev
TCA-790 - use localstorage flag for showing survey
TCA-790 Delay all calls to the survey -> dev
…service-title

PROD-3372 rename self service title -> dev
TCA-830 Remove the loading spinner from the home page -> dev
TCA-787 Link to complete a certification from any lesson -> dev
…ding

TCA-834 - updates code for uni-nav integration -> dev
@brooketopcoder brooketopcoder self-assigned this Dec 19, 2022
@brooketopcoder brooketopcoder requested a review from vas3a December 19, 2022 18:33
Copy link
Collaborator

@vas3a vas3a left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

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