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

open edx (maple): Incorrect links in user menu and top navigation #2331

Closed
1 of 6 tasks
briangrossman opened this issue Dec 18, 2021 · 2 comments · Fixed by mitodl/frontend-component-header-mitol#2
Closed
1 of 6 tasks
Assignees
Labels
Bug Something isn't working

Comments

@briangrossman
Copy link
Contributor

Steps to Reproduce

  • In a page with the new MFE interface, click on the username in the upper right to view the user menu. Several of the links need to link to a different location (see Acceptance Criteria below)
    image
  • The MIT xPRO logo in the upper left also needs to link to a different page (see Acceptance Criteria below)

06e92400-5f44-11ec-9410-cc49ae06265c

Acceptance Criteria

Related Issues

We are currently sorting out similar issues in MITx Online. We can leverage the work being done there. Here's some related issues:

@briangrossman briangrossman added the Bug Something isn't working label Dec 18, 2021
@pdpinch
Copy link
Member

pdpinch commented Dec 20, 2021

I think we need to fork frontend-component-header, similar to how edx has frontnend-component-header-edx.

Ideally, we would use one fork for xPRO, Residential MITx and mitxonline, and we could use settings for managing links and logos.

@asadiqbal08
Copy link
Contributor

asadiqbal08 commented Dec 21, 2021

@Wassaf-Shahzad can you confirm the change set in the PR mitodl/frontend-component-header-mitol#2 will effect the changes as requested in this ticket except the Account Link.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants