Skip to content

Commit

Permalink
chore: make sure links point to MFE not legacy (#445)
Browse files Browse the repository at this point in the history
* chore: make sure links point to MFE not legacy

now that the legacy profile and account pages have been removed, we need to make sure that all of the links point to the MFE URLs; we were relying on the legacy applications to do redirection before.

FIXES: APER-3884
FIXES: openedx/public-engineering#71
  • Loading branch information
deborahgu authored Feb 12, 2025
1 parent ea4506b commit 8a63b99
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
2 changes: 2 additions & 0 deletions .env.development
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,8 @@ ECOMMERCE_BASE_URL='http://localhost:18130'
ENTERPRISE_ACCESS_BASE_URL='http://localhost:18270'
LANGUAGE_PREFERENCE_COOKIE_NAME='openedx-language-preference'
LMS_BASE_URL='http://localhost:18000'
ACCOUNT_PROFILE_URL='http://localhost:1995'
ACCOUNT_SETTINGS_URL='http://localhost:1997'
LICENSE_MANAGER_URL='http://localhost:18170'
LICENSE_MANAGER_DJANGO_URL='http://localhost:18170'
SUPPORT_CONFLUENCE='localhost:18450'
Expand Down
6 changes: 4 additions & 2 deletions src/supportHeader/Header.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,8 @@ import MobileHeader from './MobileHeader';
import ROUTES from '../data/constants/routes';

ensureConfig([
'ACCOUNT_PROFILE_URL',
'ACCOUNT_SETTINGS_URL',
'LMS_BASE_URL',
'LOGOUT_URL',
'LOGIN_URL',
Expand Down Expand Up @@ -143,12 +145,12 @@ export default function Header() {
dashboardMenuItem,
{
type: 'item',
href: `${config.LMS_BASE_URL}/u/${authenticatedUser.username}`,
href: `${config.ACCOUNT_PROFILE_URL}/u/${authenticatedUser.username}`,
content: 'Profile',
},
{
type: 'item',
href: `${config.LMS_BASE_URL}/account/settings`,
href: config.ACCOUNT_SETTINGS_URL,
content: 'Account',
},
logoutMenuItem,
Expand Down

0 comments on commit 8a63b99

Please sign in to comment.