Skip to content

Extended JsonDeserializer to handle Json.NET's JsonPropertyAttribute - #138

Closed
aroder wants to merge 2 commits into
restsharp:masterfrom
aroder:master
Closed

Extended JsonDeserializer to handle Json.NET's JsonPropertyAttribute#138
aroder wants to merge 2 commits into
restsharp:masterfrom
aroder:master

Conversation

@aroder

@aroder aroder commented Jun 1, 2011

Copy link
Copy Markdown

Hey John. Json.NET includes the ability to map JSON properties to .NET properties with different names via an attribute called JsonPropertyAttribute. I wrote a test and extended JsonDeserializer.

Let me know if you need more info

@johnsheehan

Copy link
Copy Markdown
Contributor

Thanks for the request. The default serializers are opinionated and will never support attributes. JSON.NET may not always be used under the covers so I don't want any hard dependencies on it. I'm will to add a JsonNetDeserializer that works more like JSON.NET does if someone wants to contribute that.

@johnsheehan johnsheehan closed this Jun 1, 2011
@aroder

aroder commented Jun 1, 2011

Copy link
Copy Markdown
Author

I wondered about the hard dependency on JSON.NET. Thanks for the response.

Adam Roderick
aroder@gmail.com

On Wed, Jun 1, 2011 at 1:53 PM, johnsheehan <
reply@reply.github.com>wrote:

Thanks for the request. The default serializers are opinionated and will
never support attributes. JSON.NET may not always be used under the covers
so I don't want any hard dependencies on it. I'm will to add a
JsonNetDeserializer that works more like JSON.NET does if someone wants to
contribute that.

Reply to this email directly or view it on GitHub:
https://github.com/johnsheehan/RestSharp/pull/138#issuecomment-1276737

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

Labels

None yet

2 participants