feat: display profile name in navbar if available

[navinkarkera]

Display profile name if set, else fallback to username

[openedx-webhooks]

Thanks for the pull request, @navinkarkera! Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket as you can:

• supporting documentation
• Open edX discussion forum threads
• timeline information ("this must be merged by XX date", and why that is)
• partner information ("this is a course on edx.org")
• any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

[jsnwesson]

Hey all! I'm responding for the Aperture team. Any chance this would need a product review, or could you add context for where this change is being requested? Thanks!

[hurtstotouchfire]

Once https://github.com/openedx/frontend-component-header/pull/466/files merges, we will replace this custom header in learner dash with the standard component header, and then I think openedx/frontend-component-header#483 would provide this functionality, so likely this OSPR can just be closed.

That said, we should incorporate this into the product convo around removing username from the header. I think we'd need to pick between this approach and openedx/frontend-component-header#465 which puts username behind a flag.

FYI @arbrandes / @jmakowski1123

[itsjeyd]

Hey @hurtstotouchfire, thanks for weighing in here.

That said, we should incorporate this into the product convo around removing username from the header.

Where is that conversation happening? Specifically, is there a product proposal that we could link this PR back to?

CC @jmakowski1123 @arbrandes @jsnwesson @navinkarkera

[arbrandes]

@itsjeyd, I think the proposal is this one:

https://openedx.atlassian.net/wiki/spaces/OEPM/pages/4060938253/Proposal+Autogenerate+username+behind+a+flag

@hurtstotouchfire, shall we close this PR, then?

[jsnwesson]

I'll go ahead and close this PR as there's already a PR #327 to replace the header with the frontend-component-header, which will allow us to customize the navbar in that library with this PR.

[openedx-webhooks]

@navinkarkera Even though your pull request wasn’t merged, please take a moment to answer a two question survey so we can improve your experience in the future.