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

Removed ckeditor5 #35507

Merged
merged 7 commits into from
Dec 11, 2024
Merged

Conversation

orangejenny
Copy link
Contributor

Product Description

https://dimagi.atlassian.net/browse/USH-5187

This replaces the rich text editor with a plain text area. Existing alerts will still be editable, but they may display differently in email due to the removal of rich_text_email_styles.css

Screenshot 2024-12-10 at 5 16 04 PM

Feature Flag

Rich text emails

Safety Assurance

Safety story

Limited to a feature flag used by one project. Delivery will be testing on staging.

Automated test coverage

No

QA Plan

Not requesting QA

Rollback instructions

  • This PR can be reverted after deploy with no further considerations

Labels & Review

  • Risk label is set correctly
  • The set of people pinged as reviewers is appropriate for the level of risk of the change

@orangejenny orangejenny added the product/feature-flag Change will only affect users who have a specific feature flag enabled label Dec 11, 2024
@orangejenny orangejenny requested review from esoergel, a team and minhaminha and removed request for a team December 11, 2024 12:21
@orangejenny orangejenny requested a review from a team as a code owner December 11, 2024 12:21
@orangejenny orangejenny requested a review from Jtang-1 December 11, 2024 12:21
@dimagimon dimagimon added the dependencies Pull requests that update a dependency file label Dec 11, 2024
@@ -294,6 +292,7 @@ gnureadline==8.1.2
google-api-core==2.11.0
# via
# firebase-admin
# google-api-core
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jingcheng16 I'm getting a test error about inconsistent requirements. Didn't you run into this recently?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep. What versions of Python and pip-tools do you have locally? I upgrade pip-tools to 7.4.1 solved the problem.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was behind, thank you! Updated and committed new requirements files.

@orangejenny
Copy link
Contributor Author

Tests passed, ready to go.

@orangejenny orangejenny merged commit 4df3805 into master Dec 11, 2024
12 of 13 checks passed
@orangejenny orangejenny deleted the jls/webpack-scheduling-old-drop-ckeditor branch December 11, 2024 18:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file product/feature-flag Change will only affect users who have a specific feature flag enabled
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants