eSlider/go-onlyoffice
1
0
Fork 0
Code Issues 1 Pull Requests 1 Actions Packages Projects Releases 3 Wiki Activity

fix(docs): pair comment ids with the document order (#92) #93

Merged
eSlider merged 1 commits from fix/docs-comment-id#92 into main 2026-10-07 18:45:20 +01:00
Conversation 0 Commits 1 Files Changed 2
+47 -6
eSlider commented 2026-10-07 18:45:16 +01:00
Owner
Copy Link
Copy Source

Closes #92

comments.xml was built in spec order while marker w:id values were assigned in document order, so any spec whose order differed from the document attached the wrong text to an anchor. applied is now recorded in document order; TestInjectDocxCommentsOrder guards it.

Closes #92 comments.xml was built in spec order while marker w:id values were assigned in document order, so any spec whose order differed from the document attached the wrong text to an anchor. applied is now recorded in document order; TestInjectDocxCommentsOrder guards it.
eSlider added 1 commit 2026-10-07 18:45:17 +01:00
fix(docs): pair comment ids with the document order, not the spec order (#92)
Release / GoReleaser (push) Skipped
Tests / Secret scan (gitleaks) (push) Skipped
Tests / Test (Go 1.25) (push) Skipped
Tests / Test (Go stable) (push) Skipped
Tests / Secret scan (gitleaks) (pull_request) Successful in 5s
Details
Tests / Test (Go 1.25) (pull_request) Successful in 22s
Details
Tests / Test (Go stable) (pull_request) Successful in 37s
Details
00b55caff8
eSlider merged commit e4794380a0 into main 2026-10-07 18:45:20 +01:00
eSlider deleted branch fix/docs-comment-id#92 2026-10-07 18:45:20 +01:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
docs
epic
feature
infra
refactor
research
test
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
Yftyr (Edelweiss Bot) mdx-1 eSlider (Andrey)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: eSlider/go-onlyoffice#93