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

Couple of links that need to update #268

Closed
2 tasks
asadiqbal08 opened this issue Oct 27, 2021 · 0 comments · Fixed by mitodl/frontend-component-header-mitol#2
Closed
2 tasks

Couple of links that need to update #268

asadiqbal08 opened this issue Oct 27, 2021 · 0 comments · Fixed by mitodl/frontend-component-header-mitol#2
Assignees

Comments

@asadiqbal08
Copy link
Contributor

reference to #112

From the menu in MFE in MITx Online edX, there are a couple links that need to be updated. (Sample page)

  • "Dashboard" menu item should link to: MARKETING_SITE_URL/dashboard rather than LMS_BASE_URL/dashboard/
  • "Profile" menu item should link to: MARKETING_SITE/profile rather than LMS_BASE_URL/u/USERNAME
    image
    I added these to the frontend-component-header-edx MFE changes section in the description above.

Also, in the menu in the 'legacy' theme, clicking on the username will take you to LMS_BASE_URL/dashboard, but should take you to MARKETING_SITE/dashboard. (Sample page)
Screen Shot 2021-10-26 at 12 00 04 AM
I added this to the mitxonline-theme changes section above (feel free to move it if it's the wrong section)

Acceptance Criteria:

  • Update the corresponding env files to reflect the changes.
  • Test it
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant