Skip to content

Conversation

@GoSimplicity
Copy link
Contributor

This fix #8558 ensures that the Jira plugin no longer throws JSON deserialization errors when encountering rich-text descriptions, allowing the pipeline to continue running smoothly. Existing string-only descriptions remain fully supported, guaranteeing backward compatibility.

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. size:M This PR changes 30-99 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Aug 29, 2025
@dosubot dosubot bot added component/plugins This issue or PR relates to plugins needs-cherrypick-v1.0 pr-type/bug-fix This PR fixes a bug severity/p1 This bug affects functionality or significantly affect ux labels Aug 29, 2025
@GoSimplicity GoSimplicity reopened this Aug 29, 2025
@davinogueiradev
Copy link

I'm testing locally, and not work for all scenarios, reading the documentation others field can be text rich.

Issue.Description 
Comment.Body
Worklog.Comment

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Aug 29, 2025
@GoSimplicity
Copy link
Contributor Author

I'm testing locally, and not work for all scenarios, reading the documentation others field can be text rich.

Issue.Description 
Comment.Body
Worklog.Comment

Thanks for the review! This fix covers all rich text fields (Issue.Description, Comment.Body, Worklog.Comment) that can cause the ADF unmarshaling error, including Epic descriptions since they use the Issue struct. The FlexibleDescription type handles both string and ADF formats with full backward compatibility. If you encounter specific scenarios that don't work, please share the error details.

Copy link

@davinogueiradev davinogueiradev left a comment

Choose a reason for hiding this comment

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

LGTM.
I did some tests and passed them all

Copy link
Contributor

@klesh klesh left a comment

Choose a reason for hiding this comment

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

LGTM

@klesh klesh merged commit 314124a into apache:main Sep 3, 2025
10 checks passed
@davinogueiradev davinogueiradev mentioned this pull request Sep 3, 2025
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/plugins This issue or PR relates to plugins needs-cherrypick-v1.0 pr-type/bug-fix This PR fixes a bug severity/p1 This bug affects functionality or significantly affect ux size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug][Jira] Unmarshal error for epic description with rich text

3 participants