FC-0001: Account pages -> micro-frontend

[UvgenGen]

NOTE: Don't merge

Merge after the Olive release.

Description

replaced edx-platform's older implementation of the Django-server-side rendering of the following pages with new micro-frontend-based implementations:

• Learner Profile page: edx/frontend-app-profile
• Order History page: edx/frontend-app-ecommerce
• Account pages: edx/frontend-app-account ** Account Settings

PR to frontend-platform: https://github.com/openedx/frontend-platform/pull/326 PR to frontend-component-header: https://github.com/openedx/frontend-component-header/pull/210

Supporting information

https://github.com/openedx/public-engineering/issues/71 Original Jira Issue: https://openedx.atlassian.net/browse/DEPR-17

[openedx-webhooks]

Thanks for the pull request, @UvgenGen! 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.

[natabene]

@dianakhuang This seems to be with arch-bom, do you mind reviewing and merging?

[abdullahwaheed]

@UvgenGen i have reviewed and tested your changes and resolved merge conflicts locally. I cannot push my changes in your forked repo. Please either give me write access to your repo or i'll provide the patch file for changes.

[UvgenGen]

@abdullahwaheed Could you provide the patch file please?

[abdullahwaheed]

@UvgenGen the patch couldn't created for all the commits as i have merged it with master and resolved conflicts only. Could you please rebase it with master and i'll help you resolve the conflicts

[UvgenGen]

@abdullahwaheed yes, sure. I rebased this PR and resolved conflicts. Files affected: lms/djangoapps/discussion/views.py and common/djangoapps/Third_Party_auth/tests/specs/base.py.

[abdullahwaheed]

@UvgenGen could you please rebase it again

[UvgenGen]

@abdullahwaheed PR rebased)

[e0d]

@UvgenGen should the name of this PR be updated? We're after olive, can this now be merged?

[UvgenGen]

@e0d I removed Note: don't merge in description, did I understand you correctly?. Yes, I think nothing prevents merge it now)

[e0d]

@dianakhuang is this good to merge now? @arbrandes FYI as MFE guru and Olive release lead.

[dianakhuang]

I will try to get it on FED-BOM's agenda to review and help get this merged. Very excited for this to land!

[abdullahwaheed]

@UvgenGen need rebasing again due to frequent changes in platform :) or you can give me write access to your forked repo so i can update changes.

[UvgenGen]

@abdullahwaheed PR rebased)

[mphilbrick211]

Hi @UvgenGen @arbrandes @dianakhuang - is this good to merge once the branch conflicts are resolved?

[UvgenGen]

@mphilbrick211 PR rebased, conflicts resolved)

[mphilbrick211]

@arbrandes @dianakhuang Would one of you be able to merge this?

[openedx-webhooks]

@UvgenGen 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future.

[edx-pipeline-bot]

EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production.

[edx-pipeline-bot]

EdX Release Notice: This PR has been deployed to the production environment.