Skip to content

PR for JsonApplyView (#78)#338

Draft
f-aubert wants to merge 4 commits intoFasterXML:2.xfrom
f-aubert:2.x
Draft

PR for JsonApplyView (#78)#338
f-aubert wants to merge 4 commits intoFasterXML:2.xfrom
f-aubert:2.x

Conversation

@f-aubert
Copy link
Copy Markdown

@f-aubert f-aubert commented Mar 10, 2026

As discussed, here a Draft PR to solve #78.

* @JsonApplyView(BasicView.class)
*</pre>
* which would specify that property annotated would be processed
* (serialized) using View identified by <code>BasicView.class</code>.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Need to add

@SInCE 2.22

to indicate inclusion in the next version of jackson-annotations

@cowtowncoder
Copy link
Copy Markdown
Member

Looks good, only one minor comment.

*</pre>
* which would specify that property annotated would be processed
* (serialized) using View identified by <code>BasicView.class</code>.
*/
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@since 2.22

@cowtowncoder
Copy link
Copy Markdown
Member

@f-aubert One thing before I can merge this and other pr: we need CLA (unless you sent one earlier).
It's from: https://github.com/FasterXML/jackson/blob/main/CLA-jackson-2026.md and the usual way is to print, fill & sign, scan/photo, email to cla at fasterxml dot com.
Once we get I, I can proceed.

Note this only needs to be done once before the first contribution & CLA is good for all future Jackson prs.

Thank you again for contributing this, looking forward to merging!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants