Skip to content

TINYDOC-3562 - Document beforeSetContent listener for AI-only revision history - #4291

Open
kemister85 wants to merge 5 commits into
tinymce/8from
hotfix/8/TINYDOC-3562
Open

TINYDOC-3562 - Document beforeSetContent listener for AI-only revision history#4291
kemister85 wants to merge 5 commits into
tinymce/8from
hotfix/8/TINYDOC-3562

Conversation

@kemister85

@kemister85 kemister85 commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Ticket: TINYDOC-3562

Site: Staging branch

Changes:

  • Document that the TinyMCE AI plugin sets ai to true on both the BeforeSetContent and SetContent events, so either event can signal AI involvement.
  • Explain both events up front in the "Attributing AI-assisted revisions" introduction, immediately after the note that the Revision History plugin does not set the marker automatically, rather than introducing BeforeSetContent later as an addition.
  • Present the two events as equally valid choices: SetContent alone records the AI-assisted result, and listening to both also preserves the content that preceded each AI edit as its own revision. The choice is left to the integrator's use case.
  • Add a BeforeSetContent example alongside the existing SetContent example in "Saving an AI-assisted revision", showing how to capture the pre-AI content via editor.getContent().

Pre-checks:

  • Branch is correctly prefixed: hotfix/8/TINYDOC-3562
  • Files have been included where required (if applicable).
  • Build passes without console errors, warnings, or issues.

Prepared for documentation team review.

@kemister85
kemister85 requested a review from metricjs July 24, 2026 05:59
@kemister85
kemister85 marked this pull request as ready for review July 24, 2026 06:01
@kemister85
kemister85 requested review from a team and soritaheng as code owners July 24, 2026 06:01
Comment thread modules/ROOT/pages/revisionhistory.adoc Outdated
…ns for AI attribution

Move the explanation of both content events into the Attributing AI-assisted
revisions introduction and remove the framing that treated BeforeSetContent as
an optional addition to SetContent.
Comment thread modules/ROOT/pages/revisionhistory.adoc Outdated
Comment thread modules/ROOT/pages/revisionhistory.adoc Outdated
Comment thread modules/ROOT/pages/revisionhistory.adoc Outdated
Comment thread modules/ROOT/pages/revisionhistory.adoc Outdated
Comment thread modules/ROOT/pages/revisionhistory.adoc Outdated
Comment thread modules/ROOT/pages/revisionhistory.adoc

@metricjs metricjs 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.

A couple details need clarification, and the examples aren't consistent - left comments. But almost there

Comment thread modules/ROOT/pages/revisionhistory.adoc Outdated
@metricjs

Copy link
Copy Markdown
Contributor
Screenshot 2026-07-30 at 11 51 12 am Do we still need both these examples really? I expected either one example with both events, or one example for each event. This structure aligns more with the old structure of the section, with the initial focus on just setcontent

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.

3 participants