docs: [FC-0006] ADR #3 - suggest micro-frontend recomposition

[wowkalucky]

No description provided.

[openedx-webhooks]

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

[codecov]

Codecov Report

Patch coverage has no change and project coverage change: -0.04% ⚠️

Comparison is base (3ee263e) 69.64% compared to head (4de990c) 69.60%.
Report is 27 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #143      +/-   ##
==========================================
- Coverage   69.64%   69.60%   -0.04%     
==========================================
  Files          16       27      +11     
  Lines         280      408     +128     
  Branches       65       90      +25     
==========================================
+ Hits          195      284      +89     
- Misses         85      123      +38     
- Partials        0        1       +1     

see 14 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[e0d]

@hurtstotouchfire Do you have a sense of when this will be schedule? We're not blocked, but are trying to plan.

[hurtstotouchfire]

I would really like to get one of our front end engineers to review this and he is booked pretty solid this sprint and potentially next sprint as well. I feel confident we can review it in mid-April as we're currently pushing to hit an early April deadline. Would that be problematic? The best I can offer is bringing it into our next sprint, which runs 3/27 to 4/10.

[e0d]

@wowkalucky see Kelly's comment. Does that cause schedule issues on your end?

[wowkalucky]

@wowkalucky see Kelly's comment. Does that cause schedule issues on your end?

No, currently we are adding the extra feature in the same way this MFE was initiated. But we treat these suggestions as a tech debt (we'd like to resolve).

[hurtstotouchfire]

This will be in our next two week sprint, which starts Monday, fyi.

[MaxFrank13]

Thanks for doing this work. I had a few clarifying questions and one minor suggestion

[jsnwesson]

I'm picking up from what Max was looking at last week. Thanks for the additional explanations, this looks good to me (aside for one paragraph that might be worth revising).

[jsnwesson]

edit suggestion: At this point, the micro-frontend has a few interactions (data fetches) with a single backend API. We use getAuthenticatedHttpClient on a low level (e.g. directly, each time it's configured), and then handle the API request results (e.g. error checking, loading states, etc.).

[wowkalucky]

Updated ✔️

[justinhynes]

It looks like the requested updates have been completed and the PR is approved. Are there are additional updates expected? Can this be merged?

@wowkalucky @hurtstotouchfire @MaxFrank13 @jsnwesson @GlugovGrGlib

[wowkalucky]

@justinhynes nothing from my side.

[GlugovGrGlib]

As all CR fixes were added and PR was approved, I am changing ADR status to Accepted and merging it. Therefore, the further VC work should introduce project structure changes documented in this ADR.

[openedx-webhooks]

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