-
Notifications
You must be signed in to change notification settings - Fork 34
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
[DEPR]: studio-frontend repo #381
Comments
I'm excited for this one 🔥 |
@KristinAoki We are in the process of assigning owners for DEPR tickets to ensure that we have a point of contact on the current status of them. I assigned you to this one since you seem to be handling most of the movement of the ticket. Out of curiosity, why is this in 'deprecated' instead of 'removing'? Is it just because we're not doing active work on it? |
@dianakhuang I was not fully sure which it belonged under. The code has been moved to course authoring, but the act of removing the repo and references to it in |
@KristinAoki , I think Deprecated is the right status. I see you marked it as deprecated already, which is great. How can we move it forward? Ideally, 1 release should go by where both the old pages and the new pages are available for operators to use; has that happened yet? And are the new pages at feature-parity with the old pages? If it's Yes on both counts, then your team or anyone else in the community would be all-clear to start removing studio-frontend. |
Redwood will be the first release with the new pages and the old pages. The old pages for the Checklist page and the Accessibility Request page have redirects in edx-plaform that always redirect to the new page. The Files and Uploads page redirect can be toggled with the |
Awesome! Do you have any opposition if Tutor toggles on the new Files & Uploads page by default? We could add opt-out instructions to the Redwood release notes. That would allow removal to begin as early as the day after the Redwood cut date. |
No opposition. |
Cool, I will take care of making sure Tutor runs the new page in Redwood, can you update the timeline in the issue description? |
In Studio, this switches from the Django-rendered Files & Uploads page to the new version of the page in the course-authoring MFE. Operators can opt-out for Redwood by disabling the `contentstore.new_studio_mfe.use_new_files_uploads_page` flag, either per-course or site-wide. Operators can also still disable the course-authoring MFE entirely. However, starting in Sumac, this option will be removed for the Files & Uploads page (and potentially for other pages) so we recommend that operators switch to the new page during Redwood. Part of: openedx/studio-frontend#381
Thanks Kristin. Here's where I'll be working on adding this to Tutor: overhangio/tutor-mfe#201 |
Searching for the
I'm trying to confirm right now whether all of these pages will be enabled in Redwood and ready to remove in Sumac. |
When navigating to the Checklist page, users should automatically redirected to {course authoring host}/checklists
Can be found at{studio host}/accessibility and is moved to {course authoring host}/accessibility All of the references are not yet deleted in edx-platform |
Nice, thanks. Is the MFE page gated by a waffle, or enabled by default?
Got it, thanks. |
It is enabled by default. It is just showing information so it was low risk to just redirect by default |
Sounds great @KristinAoki 👍🏻 Because studio-frontend powers the image uploader for the legacy Text component, it seems like we would need to deprecate the legacy Text editor too. Would you or someone on TNL be able to write that up? |
Yes, I can create the deprecation notice. Do I create the deprecation ticket in |
Yep. Thanks! |
Issue for legacy text editor deprecation in edx-platform, #34692 |
Proposal Date
02 June 2023
Ticket Acceptance Date
15 August 2023
Technology Removal Date
Before Sumac, exact date TBD
Removal blocked by openedx/edx-platform#34692
First Open edX Named Release Without This Functionality
Sumac
Rationale
This repo is currently only used inside the edX Studio. The React code is imported into Django templates and rendered inside of Studio. The React code is only used on four pages, Assets, Accessibility Form, Course Outline Status, and Checklist. The pages in edX Studio are being update to use React and moved to
frontend-app-course-authoring
. To reduce the redundancy of two studio frontend repos, this one will be removed and the components that are not slated to be redesigned will be copied over tofrontend-app-course-authoring
.Removal
The entire repo will be removed and the following references inside
edx-platform
:Replacement
The majority of the code will be rewritten in
frontend-app-course-authoring
. The accessibility and checklist will be moved tofrontend-app-course-authoring
. The code won't change, but will live in thefrontend-app-course-authoring
.Deprecation
No response
Migration
No response
Additional Info
Communicated here: https://discuss.openedx.org/t/deprecation-removal-studio-frontend-381/10280
The text was updated successfully, but these errors were encountered: