Skip to content
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]: Legacy Text Editor #34692

Open
KristinAoki opened this issue May 3, 2024 · 2 comments
Open

[DEPR]: Legacy Text Editor #34692

KristinAoki opened this issue May 3, 2024 · 2 comments
Labels
depr Proposal for deprecation & removal per OEP-21

Comments

@KristinAoki
Copy link
Member

Proposal Date

2024-05-03

Target Ticket Acceptance Date

2024-05-17

Earliest Open edX Named Release Without This Functionality

Sumac - 2024-10

Rationale

The EditImageModal component that is used it the legacy Text Editor is also set for deprecation and removal. To prevent errors and regression while using the legacy Text Editor, it is best to remove it completely from the platform. The legacy Text Editor already has replacement that can be toggled with new_core_editors.use_new_text_editor and will be come the default once this deprecation occurs.

Removal

  1. Remove toggle for switching between editors
  2. Remove dependency on toggle for text editor redirect and make it always true
  3. Remove dependencies on toggles when fetching the course authoring mfe url

Replacement

This code is being replaced by the frontend-lib-content-components/TextEditor. This text editor is a React based component and is rendered in the Course Authoring MFE.

Deprecation

No response

Migration

No response

Additional Info

No response

Task List

No response

@KristinAoki KristinAoki added the depr Proposal for deprecation & removal per OEP-21 label May 3, 2024
@felipemontoya
Copy link
Member

The removal of the ENABLE_NEW_TEXT_EDITOR_FLAG was done at the frontend in this PR openedx/frontend-app-course-authoring#951. This happened due to a bug before the redwood cut.

@kdmccormick
Copy link
Member

If you're watching the issue, then FYI there is some conversation about StudioEditableXBlockMixin going on in the related forum thread: https://discuss.openedx.org/t/deprecation-removal-text-editor-in-edx-platform-34692

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
depr Proposal for deprecation & removal per OEP-21
Projects
Status: Proposed
Development

No branches or pull requests

3 participants