Skip to content

Conversation

IMurzich
Copy link

@IMurzich IMurzich commented Oct 8, 2025

Use this annotation when need to have different result for json and xml output. Parameters for this annotation same as for JsonInclude.

@cowtowncoder
Copy link
Member

Thank you for your contribution.

Unfortunately I don't think we want to go down the route of per-format annotations, duplicating all functionality.

So while the idea of having a mechanism for annotating different behavior for different format is valid, we need to figure out more maintainable approach.

@IMurzich
Copy link
Author

IMurzich commented Oct 8, 2025

Hi @cowtowncoder !
You absolutely true about new mechanism, but I don't see any other way for branch 2.x. Therefore I suggest stay this change in 2.x branch only and use this case (different incluthion rules for different output formats) when develop new approach.

Thanks.

@pjfanning
Copy link
Member

Hi @cowtowncoder ! You absolutely true about new mechanism, but I don't see any other way for branch 2.x. Therefore I suggest stay this change in 2.x branch only and use this case (different incluthion rules for different output formats) when develop new approach.

Thanks.

@IMurzich 2.x is no longer the dev branch. Most changes that will go in 2.x in future will be bug fixes. This is not a bug fix.

@IMurzich
Copy link
Author

IMurzich commented Oct 8, 2025

Ok, will I move this change to 3.x branch?

@cowtowncoder
Copy link
Member

cowtowncoder commented Oct 8, 2025 via email

@IMurzich
Copy link
Author

IMurzich commented Oct 9, 2025

@cowtowncoder , thank you for your time.

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