Skip to content

TINYINT-3453: Update details - #32

Merged
tiny-ben-tran merged 6 commits into
mainfrom
feature/TINYINT-3453
Oct 2, 2026
Merged

tiny-ben-tran merged 6 commits into
mainfrom
feature/TINYINT-3453

Conversation

@tiny-ben-tran

@tiny-ben-tran tiny-ben-tran commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

Related ticket: TINYINT-3453

Description of changes:

  • Updated readme.md
  • Added contributing.md
  • Updated author, description and keywords in package.json

@tiny-ben-tran
tiny-ben-tran requested review from a team, TheSpyder, kimwoodfield, ltrouton and metricjs and removed request for a team September 25, 2026 02:36
@tiny-ben-tran
tiny-ben-tran marked this pull request as ready for review September 25, 2026 02:36
@tiny-ben-tran
tiny-ben-tran requested a review from a team as a code owner September 25, 2026 02:36
Comment thread package.json Outdated
Co-authored-by: Millie <milliemacdonald4@gmail.com>

@ArvinJ-H ArvinJ-H left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The version in package.json needs to go back to 2.3.1-rc before this merges; details inline. I couldn't open the Notion template linked from the epic (it returns 404 for me), so I haven't checked the files against it.

The README on main explained that the jQuery integration was part of the TinyMCE bundle until TinyMCE 6. Is dropping that paragraph intended? It is the only place in this repo that says why this is a separate package.

Comment thread package.json Outdated
Comment thread README.md
Co-authored-by: Arvin(Jintao) Han <20640097+ArvinJ-H@users.noreply.github.com>
@tiny-ben-tran
tiny-ben-tran merged commit 1f7e1f3 into main Oct 2, 2026
4 checks passed
@tiny-ben-tran
tiny-ben-tran deleted the feature/TINYINT-3453 branch October 2, 2026 03:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants