Telemetry v1 updates#305
Merged
Merged
Conversation
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
caio-pizzol
reviewed
Feb 20, 2025
caio-pizzol
left a comment
caio-pizzol
left a comment
Contributor
There was a problem hiding this comment.
@VladaHarbour overall LGTM - just a few comments.
@harbournick @artem-harbour want to hear your thoughts as well.
superdoc-bot Bot
pushed a commit
that referenced
this pull request
Jul 3, 2026
…changes (SD-3563) (#305) * fix(super-editor): export tracked citation hyperlinks SD-3563 fixes v1 headless structural insertion of citation hyperlinks when track changes are enabled. The structural writer already materialized linked citation text with both link and trackInsert marks, but DOCX export prioritized the track-change wrapper and paragraph merge could emit a single w:ins around both linked and unlinked text. That lost the valid tracked-hyperlink shape Bloomberg needs for reviewable citations. Text export now routes linked tracked text through the hyperlink translator first so DOCX emits w:hyperlink containing w:ins/w:del content. Hyperlink export preserves remaining revision marks while stripping only the link mark, and hyperlink import now includes revision children so w:hyperlink > w:ins/w:del round-trips back to linked tracked text. Adds SD-3563 regression coverage for the v1 headless plan flow plus focused translator tests for linked tracked export/import. * fix(super-editor): drop final linked deletions * fix: tracked hyperlink rejection cleanup Ported-From-Source-Repo: superdoc/orbit Ported-From-Source-Commit: 8a0951df3b15045bf47390e82ca24e5714aa9f1d Ported-Public-Prefix: superdoc/public
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.