Skip to content

Refactor bodyXML → bodyTree Go transformer to use internalComponents format only#115

Merged
lokendersinghft merged 2 commits into
mainfrom
fix/UPPSF-6457
Oct 19, 2025
Merged

Refactor bodyXML → bodyTree Go transformer to use internalComponents format only#115
lokendersinghft merged 2 commits into
mainfrom
fix/UPPSF-6457

Conversation

@lokendersinghft
Copy link
Copy Markdown
Contributor

@lokendersinghft lokendersinghft commented Oct 10, 2025

Modify the bodyXML → bodyTree Go transformer to support only the bodyXML format stored in the internalComponents collection, removing support for the format returned by the internal-content-api. The transformer is intended for use in Content Tree API.

This update ensures that the generated bodyTree output more closely matches the content-tree expected to be published from Spark.

Jira --> https://financialtimes.atlassian.net/browse/UPPSF-6457

@lokendersinghft lokendersinghft marked this pull request as ready for review October 10, 2025 09:07
@lokendersinghft lokendersinghft requested review from a team as code owners October 10, 2025 09:07
@lokendersinghft lokendersinghft marked this pull request as draft October 13, 2025 07:24
@tsvetko24 tsvetko24 requested a review from a team October 16, 2025 09:20
@lokendersinghft lokendersinghft marked this pull request as ready for review October 16, 2025 09:32
Comment thread libraries/from-bodyxml/go/README.md Outdated
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.

suggestion (non-blocking): For future iterations - the "kitchen sink" test case is a good one, gathering in one document all kinds of body elements. We may re-introduce it when the Go transformers are more stable.

Co-authored-by: Elitsa Pavlova <epavlova@users.noreply.github.com>
@lokendersinghft lokendersinghft merged commit 0574799 into main Oct 19, 2025
2 checks passed
@lokendersinghft lokendersinghft deleted the fix/UPPSF-6457 branch October 19, 2025 20:41
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