Skip to content

chore: Update tinymce version to 6.8.3 in package.json and yarn.lock#36909

Merged
rahulbarwal merged 5 commits intoreleasefrom
rahulbarwal/issue36859
Oct 17, 2024
Merged

chore: Update tinymce version to 6.8.3 in package.json and yarn.lock#36909
rahulbarwal merged 5 commits intoreleasefrom
rahulbarwal/issue36859

Conversation

@rahulbarwal
Copy link
Contributor

@rahulbarwal rahulbarwal commented Oct 16, 2024

Description

TinyMCE initially switched from LGPL to MIT in 2022, aiming for a more permissive license. However, it reverted to GPL v2+ for version 7, which is a concern for us.

TinyMCE 6.8.3 under the MIT license until October 31, 2024, it will stop receiving any security updates post that.

Fixes #36859
or
Fixes Issue URL

Warning

If no issue exists, please create an issue first, and check with the maintainers if the issue is valid.

Automation

/ok-to-test tags="@tag.TextEditor"

🔍 Cypress test results

Tip

🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/11379278624
Commit: 8aa96b6
Cypress dashboard.
Tags: @tag.TextEditor
Spec:


Thu, 17 Oct 2024 06:29:32 UTC

Communication

Should the DevRel and Marketing teams inform users about this change?

  • Yes
  • No

@rahulbarwal rahulbarwal self-assigned this Oct 16, 2024
@rahulbarwal rahulbarwal added the Widgets & Accelerators Pod Issues related to widgets & Accelerators label Oct 16, 2024
@github-actions github-actions bot added the skip-changelog Adding this label to a PR prevents it from being listed in the changelog label Oct 16, 2024
@rahulbarwal rahulbarwal added ok-to-test Required label for CI and removed skip-changelog Adding this label to a PR prevents it from being listed in the changelog labels Oct 16, 2024
@github-actions github-actions bot added the skip-changelog Adding this label to a PR prevents it from being listed in the changelog label Oct 16, 2024
The code changes in this commit fix the issue with selecting the text color in the RichTextEditor. The previous code was using an incorrect selector for the text color button, causing the selection to fail. This commit updates the selector to correctly select the text color button and allows users to choose the desired color.

Refactor the selector for the text color button in the RichTextEditor to use the correct attribute and value.
@rahulbarwal
Copy link
Contributor Author

/ci-test-limit-count run_count=15

@github-actions
Copy link

@rahulbarwal rahulbarwal marked this pull request as ready for review October 17, 2024 06:01
@github-actions github-actions bot added Needs Triaging Needs attention from maintainers to triage Rich Text Editor Widget Task A simple Todo Widgets Product This label groups issues related to widgets labels Oct 17, 2024
@rahulbarwal rahulbarwal merged commit 9719e92 into release Oct 17, 2024
@rahulbarwal rahulbarwal deleted the rahulbarwal/issue36859 branch October 17, 2024 06:48
@github-actions
Copy link

Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/11379004416.
Cypress dashboard: Click here!
The following are new failures, please fix them before merging the PR:
To know the list of identified flaky tests - Refer here

***** Repeat Run Summary ***** Total Tests with repeat: 240 Total Passed: 240 Total Failed: 0 Total Skipped: 0 *****************************

github-actions bot pushed a commit to Zeral-Zhang/appsmith that referenced this pull request Nov 20, 2024
…ppsmithorg#36909)

## Description

TinyMCE initially switched from LGPL to MIT in 2022, aiming for a more
permissive license. However, it reverted to GPL v2+ for version 7, which
is a concern for us.

TinyMCE 6.8.3 under the MIT license until October 31, 2024, it will stop
receiving any security updates post that.


Fixes appsmithorg#36859
_or_  
Fixes `Issue URL`
> [!WARNING]  
> _If no issue exists, please create an issue first, and check with the
maintainers if the issue is valid._

## Automation

/ok-to-test tags="@tag.TextEditor"

### 🔍 Cypress test results
<!-- This is an auto-generated comment: Cypress test results  -->
> [!TIP]
> 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
> Workflow run:
<https://github.com/appsmithorg/appsmith/actions/runs/11379278624>
> Commit: 8aa96b6
> <a
href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=11379278624&attempt=1"
target="_blank">Cypress dashboard</a>.
> Tags: `@tag.TextEditor`
> Spec:
> <hr>Thu, 17 Oct 2024 06:29:32 UTC
<!-- end of auto-generated comment: Cypress test results  -->


## Communication
Should the DevRel and Marketing teams inform users about this change?
- [ ] Yes
- [x] No
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs Triaging Needs attention from maintainers to triage ok-to-test Required label for CI Rich Text Editor Widget skip-changelog Adding this label to a PR prevents it from being listed in the changelog Task A simple Todo Widgets & Accelerators Pod Issues related to widgets & Accelerators Widgets Product This label groups issues related to widgets

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Task]: Check if your repository is affected by licence change of TinyMCE

3 participants