Authenticated user dropdown personal dashboard redirects to hardcoded edX reference

[mariajgrimaldi]

Description

As an authenticated student/instructor, when you do User dropdown > Switch dashboard > Personal redirects to /edx-dashboard.

Possible solutions

First, a few context questions:

What's the purpose of the personal dashboard?
How does it work for edX? What does it do?
How would that look like for the community?

As for the Quince release, we could hide the Switch dashboard behind a feature flag here:

frontend-app-learner-dashboard/src/containers/LearnerDashboardHeader/ExpandedHeader/AuthenticatedUserDropdown.jsx

Lines 32 to 49 in 1a51ac0

	<Dropdown.Header>SWITCH DASHBOARD</Dropdown.Header>
	<Dropdown.Item as="a" href="/edx-dashboard" className="active">
	Personal
	</Dropdown.Item>
	{!!dashboard && (
	<Dropdown.Item as="a" href={dashboard.url} key={dashboard.label}>
	{dashboard.label} {formatMessage(messages.dashboard)}
	</Dropdown.Item>
	)}
	<Dropdown.Divider />
	{!dashboard && getConfig().CAREER_LINK_URL && (
	<Dropdown.Item href={`${getConfig().CAREER_LINK_URL}`}>
	{formatMessage(messages.career)}
	<Badge className="px-2 mx-2" variant="warning">
	{formatMessage(messages.newAlert)}
	</Badge>
	</Dropdown.Item>
	)}

I'll be opening a PR with this proposed change.

PRs

• refactor: hide switch dashboard option behind flag #237

[mariajgrimaldi]

Another possible solution might be: using the LearningHeader (or similar?) instead of the custom header it currently uses: https://github.com/openedx/frontend-app-learner-dashboard/blob/1a51ac07a284b206ea071d6ab067950d74eb3c47/src/containers/LearnerDashboardHeader/ExpandedHeader/index.jsx#L24-L76 which might solve as well: #178

[mariajgrimaldi]

@arbrandes: I found this issue while testing the MFE in a Quince tutor dev environment. And I'd like to know why this MFE uses a custom header instead of the LearningHeader or something similar. That might be the best approach for this for the long term, but it sounds like a lot of work to do before the release. Is hiding the Personal dashboard behind a feature flag enough?

[mariajgrimaldi]

Here's a PR with the suggested change (hiding behind a flag): #237