Fixed doc typos re: @Jsonb(De)Serializer#183
Conversation
Closes jakartaee#70 Signed-off-by: Markus KARG <markus@headcrashing.eu>
|
I marked this PR as a draft, as the spec source is not contained. Please tell me if I shall keep it in draft state until the spec source is available, or whether I shall turn it in a real PR so you could merge it. Thanks. :-) |
|
@aguibert You better ask Eclipse folks. They need to do some legal checks before allowing us using the old spec text. |
|
@m0mus Did EF already ask you for your approval of the spec? I was asked weeks ago and it seems nothing happend so far. So how to proceed? Keep the PR in draft state until we have the OK from the EF, or merge this PR, keep the issue open, and create another PR for the spec change? |
|
@mkarg No, I didn't have any request from EF regarding the spec. I suppose that it will happen after Jakarta EE 8 release. |
|
Spec document has been merged a long time ago (3b879d7), can we get this PR merged or possibly even extended to also fix the spec? |
| customer.setName("Freddie"); | ||
|
|
||
| // Also configurable with @JsonbSerializer / JsonbDeserializer on properties and class. | ||
| // Also configurable with @JsonbTypeSerializer / @JsonbTypeDeSerializer on properties and class. |
There was a problem hiding this comment.
| // Also configurable with @JsonbTypeSerializer / @JsonbTypeDeSerializer on properties and class. | |
| // Also configurable with @JsonbTypeSerializer / @JsonbTypeDeserializer on properties and class. |
|
@mkarg - please resolve the merge conflicts and see my review above. I'd like to get this merged. |
6 participants
Partial fix of #70 (JavaDocs only)
Unfortunately I could only fix the JavaDocs, as there is just boilerplate for the actual spec.