-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Olive: list of new MFEs #200
Comments
In the nutmeg release notes, these MFEs were identified as "pre-alpha" and presumably should be considered for olive: |
The Discussions MFE has |
I discovered another MFE, via its setting, ORA_GRADING_MICROFRONTEND_URL |
@Tj-Tracy FYI |
According to @nedbat in Slack, these are the MFEs that will get olive branches:
|
Are we any closer to agreeing which ones will be added to the tutor-mfe plugin? |
There's a PR to add |
There's a ticket to test Course Authoring. #202 |
Considering the huge list of new MFEs that are expected to land in Olive, I'm very anxious that we merge the runtime configuration feature. If we don't, I'm afraid we won't be able to release Olive at all. I suggest we have this conversation here: https://discuss.openedx.org/t/mfe-runtime-configuration/8495 |
I think we can safely omit frontend-template-application from the list of potential MFEs. I don't even know why it has an olive branch. |
Copy-pasta of a post I made in the forum: Do we need to include every MFE in that list in Tutor/Olive?TLDR: No. MFE developers (as of now mostly employed at or by 2U) are aware they can't remove a feature in edx-platform before the MFE that replaces it is officially accepted as being production-ready by the community. This should apply to each MFE in that list, and it means we don't have to enable any of them for Olive if we don't want to. How do we decide which ones to include?TLDR: Don't know. We need to decide how to decide. While the ultimate decision on whether an MFE is ready is in fact up to the community, and in particular, the BTR group, there is no official method to arrive at this decision. This is a problem we should fix pronto, precisely so we can answer the question of which ones to include. This comment in the ticket where we're tracking this question is a collation of the information I could find. Please refer to it and add your comments, including which MFEs you would like to have included, and why. Once we have a reasonable idea of where the community's opinion lies, we can start to narrow down the list in a technical fashion: by checking each MFE against the criteria. (And yes, right off the bat it looks like a sine qua non criterion is whether the MFE supports runtime configuration.) Product considerationsTLDR: Let's consult the Product Working Group as well. There are product considerations to take into account. Ultimately, I think the list will be narrowed down a lot by them. I'm not saying they have ultimate say, but it doesn't make sense to start maintaining Tutor support for an MFE if it doesn't make sense to make it part of the Open edX product at a given point in time. I've started discussing this with @jmakowski, yesterday. The short of it is that my job will be to try and put all stakeholders together (mostly asynchronously, such as via this post) so we can decide ASAP which MFEs to work on for Olive. |
@arbrandes @pdpinch Just noting the parallel ticket I'm working on here to try and get as much product-oriented data as possible for these. Each MFE has a ticket now on the community roadmap and a template for base level of acceptance criteria data: openedx/axim-engineering#495 The discussions MFE has some info. The rest I'm investigating... |
FYI @arbrandes / @pdpinch I've worked with Jenna to get openedx/frontend-app-learner-record#107 fleshed out for LRMFE. Our goal is for it to be included in Olive, off by default, for any adventurous beta adopters who have worked out a way to run the Credentials IDA. For Palm, we will aim to have Credentials supported in Tutor and the old UI will be removed, so the MFE will be mandatory in Palm. Note that overhangio/tutor-mfe#63 is blocked pending a consensus on the merge target and documentation. Edit: since it sounds like runtime config is also required I'm adding that to our running todo list for releasing with Olive but please lmk when an official checklist exists. |
The following MFEs have at least one champion in the BTR group for inclusion/enablement by default in the
The following has an open PR for inclusion and no objections were raised, so it will be tested:
The following met no objections to "optional enablement", where there will be instruction in the Release Notes on how to get them running with Tutor, but they won't actually be part of the
Last but not least, the group thinks frontend-app-discussions will likely not meet requirements in time, but a suggestion was made to make it its own plugin. The question was raised: are the authors of frontend-app-discussions willing to create and maintain such a plugin? |
It was raised in the forum that frontend-app-authn has a (possibly) pressing reason to be included now, so I'm adding it to the championed list. |
The final list of nominees:
Their status against the list of requirements is also being kept in the wiki: (There are some more details in comments to this other issue.) Hopefully we'll have a decision on which ones we'll actually include tomorrow. |
From yesterday's meeting, BTR have elected the following two new MFEs for potential inclusion, pending volunteers to fix individual issues. If individual issues listed below are not fixed in the next two weeks (leaving two further weeks for testing/translation), they will not be included:
|
Having a list of new MFEs will help with planning, testing and documentation of olive.
To be included in the olive release, a MFE must:
To be supported in olive, a MFE must:
The text was updated successfully, but these errors were encountered: