search

erikzaadi / GithubSharp

No longer maintained, please use https://github.com/octokit/octokit.net
96 stars 47 forks source link

AddEmail/RemoveEmail #1

Closed rumpl closed 14 years ago

rumpl commented 14 years ago

Hi,

I was looking at your library and found two bugs: The AddEmail and RemoveEmail methods didn't do anything, the url was wrong (it's user/email/[add|remove] and not user/emails) and you forgot to add the formValues to the ConsumeJsonUrlAndPostData method.

Plus there was some incosistency with method parameters names.

Cheers.

erikzaadi commented 14 years ago

Thanks Djordje, merged it.

Much appreciated!

Erik