Skip to content

DOC-3309: Update content for various documentation demos.#3908

Merged
kemister85 merged 7 commits into
tinymce/8from
hotfix/8/DOC-3309
Nov 11, 2025
Merged

DOC-3309: Update content for various documentation demos.#3908
kemister85 merged 7 commits into
tinymce/8from
hotfix/8/DOC-3309

Conversation

@kemister85
Copy link
Copy Markdown
Contributor

@kemister85 kemister85 commented Nov 10, 2025

Ticket: DOC-3309

Site: basic-example
Site: full-featured-open-source-demo
Site: full-featured-premium-demo

Changes:

  • Update demo html content and include uploadcare to replace image | editimage plugins.

Pre-checks:

  • Branch prefixed with feature/<version>/, hotfix/<version>/, staging/<version>/, or release/<version>/.

Review:

  • Documentation Team Lead has reviewed

@kemister85 kemister85 marked this pull request as ready for review November 10, 2025 02:58
Copy link
Copy Markdown
Member

@lostkeys lostkeys left a comment

Choose a reason for hiding this comment

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

There are a lot of new comment annotations in the js file, and I just want to verify they are not showing up for prospects viewing the demos. Please hold of merging while we discuss this.

Comment thread modules/ROOT/examples/live-demos/full-featured/index.js Outdated
Comment thread modules/ROOT/examples/live-demos/full-featured/index.js Outdated
@kemister85 kemister85 dismissed lostkeys’s stale review November 11, 2025 04:47

removing blocker as this has been approved in JIRA

@kemister85 kemister85 requested a review from lostkeys November 11, 2025 04:48
Copy link
Copy Markdown
Contributor

@ShiridiGandham ShiridiGandham left a comment

Choose a reason for hiding this comment

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

LGTM

@kemister85 kemister85 merged commit 0d1be97 into tinymce/8 Nov 11, 2025
5 checks passed
@kemister85 kemister85 deleted the hotfix/8/DOC-3309 branch November 11, 2025 05:13
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