Skip to content

[Suggestion] Use RestSharp to replace ApiInvoker in CSharp #673

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.

Sign up for GitHub

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

Jump to bottom
Closed
wing328 opened this issue Apr 22, 2015 · 2 comments
Closed

[Suggestion] Use RestSharp to replace ApiInvoker in CSharp #673

wing328 opened this issue Apr 22, 2015 · 2 comments
Labels
Client: C-Sharp Enhancement: Feature
Milestone
v2.1.0

Comments

@wing328
Copy link
Contributor

wing328 commented Apr 22, 2015

I would suggest using RestSharp to replace ApiInvoker in the CSharp client.

I've tested GetPetById (GET), AddPet (POST JSON) and UpdatePetWithForm(POST FORM) using RestSharp and it works perfectly fine. To me, the code looks cleaner and easier to understand.

Here is a list of projects/SDKs using RestSharp:
https://github.com/restsharp/RestSharp/wiki/Projects-Using-RestSharp

May I know if anyone has concern using RestSharp to replace ApiInvoker in CSharp?
@fehguy
Copy link
Contributor

fehguy commented Apr 28, 2015

no concerns from my side. Looks like a solid library.

@wing328 wing328 mentioned this issue Apr 29, 2015
Merged
@webron webron added this to the v2.1.0 milestone May 4, 2015
@wing328
Copy link
Contributor Author

wing328 commented May 21, 2015

PR merged

@wing328 wing328 closed this as completed May 21, 2015
@schrooom schrooom mentioned this issue Feb 25, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Client: C-Sharp Enhancement: Feature
Projects
None yet
Milestone
v2.1.0
Development

No branches or pull requests
3 participants
@webron @fehguy @wing328