Open TJHeuvel opened 10 years ago
Hi,
Thank you for the contribution. This seems interesting and probably something I'd like to add. A few comments:
JsonIgnore
commit, it seems like it's the same issue from pull request #7. I'd invite you to share your point of view there, and leave this pull request only for the JsonName
issue (please remove the commit https://github.com/TJHeuvel/litjson/commit/f503075789c06114bbcd7131aa36bda037c6facb if that's the case).JsonName
attribute, that's great; otherwise I'd like to choose the name for that symbol carefully based on what introduces the least disruption in codebases that currently use other libraries. If you can help me with this task of reviewing what other .Net libraries do, I'd appreciate it very much :).Thanks.
Whenever a field has the JSONName property it will use that name, instead of the fieldname. This allows you to slightly deviate from the incoming or outgoing JSON.