Skip to content

Commit b813085

Browse files
navinkarkeraArturGaspar
authored andcommitted
feat: display profile name in navbar if available
Display profile name if it is set, else fallback to username
1 parent 1ccb586 commit b813085

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

src/learning-header/AuthenticatedUserDropdown.jsx

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ import { Dropdown } from '@openedx/paragon';
99

1010
import messages from './messages';
1111

12-
const AuthenticatedUserDropdown = ({ intl, username }) => {
12+
const AuthenticatedUserDropdown = ({ intl, username, name }) => {
1313
const dashboardMenuItem = (
1414
<Dropdown.Item href={`${getConfig().LMS_BASE_URL}/dashboard`}>
1515
{intl.formatMessage(messages.dashboard)}
@@ -23,7 +23,7 @@ const AuthenticatedUserDropdown = ({ intl, username }) => {
2323
<Dropdown.Toggle variant="outline-primary">
2424
<FontAwesomeIcon icon={faUserCircle} className="d-md-none" size="lg" />
2525
<span data-hj-suppress className="d-none d-md-inline">
26-
{username}
26+
{name || username}
2727
</span>
2828
</Dropdown.Toggle>
2929
<Dropdown.Menu className="dropdown-menu-right">
@@ -51,6 +51,7 @@ const AuthenticatedUserDropdown = ({ intl, username }) => {
5151
AuthenticatedUserDropdown.propTypes = {
5252
intl: intlShape.isRequired,
5353
username: PropTypes.string.isRequired,
54+
name: PropTypes.string.isRequired,
5455
};
5556

5657
export default injectIntl(AuthenticatedUserDropdown);

src/learning-header/LearningHeader.jsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,7 @@ const LearningHeader = ({
5151
{showUserDropdown && authenticatedUser && (
5252
<AuthenticatedUserDropdown
5353
username={authenticatedUser.username}
54+
name={authenticatedUser.name}
5455
/>
5556
)}
5657
{showUserDropdown && !authenticatedUser && (

0 commit comments

Comments
 (0)