-
-
Notifications
You must be signed in to change notification settings - Fork 331
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add @JsonKey
annotation.
#179
Conversation
This annotation will be used like `@JsonValue` but only when serializing Map keys. See jackson-databind/#2871
* Boolean argument is only used so that sub-classes can "disable" | ||
* annotation if necessary. | ||
*<p> | ||
* NOTE: when use for Java <code>enum</code>s, one additional feature is |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably needs to be removed for now
* to define mapping, but can not be done in general for POJO types; as such, | ||
* this is not used for POJO deserialization. | ||
* | ||
* @see JsonCreator |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure @JsonCreator
handling would be yet supported either.
Aside from javadoc comments (which I can tweak too), looks good. I don't think I have CLA yet from you? If so, it's here: https://github.com/FasterXML/jackson/blob/master/contributor-agreement.pdf and usual way is to print, fill & sign, scan/photo, email to |
Indeed's signed a corporate CLA, should be covered. |
This stuff was copied from JsonValue and isn't correct here.
This annotation will be used like
@JsonValue
but only whenserializing Map keys.
See jackson-databind/#2871