Skip to content
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.

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

Adding notes objects to be retrieved with base post class #8

Merged
merged 1 commit into from
Jun 23, 2017

Conversation

rosalindwills
Copy link
Contributor

Hello! I'm in need of your library for a fairly urgent update to a site I run to allow it to interact with Tumblr's OAuth flow; however, discovered in the process that you don't yet have the notes objects populated when mapping from Tumblr's responses, which I also need. Took the liberty of adding it myself for a PR; fairly simple object structure update and Newtonsoft does most of the work, but let me know if you need any further updates on my end!

@piedoom
Copy link
Owner

piedoom commented Jun 23, 2017

Thanks for the contribution. Looks all good to me. I don't have the time right now to test (I really should set up a test suite/CI at some point) but just reading your source looks good from what I remember about the JSON API.

@piedoom piedoom merged commit 5545362 into piedoom:master Jun 23, 2017
@rosalindwills
Copy link
Contributor Author

Spiffy :D Is this going to go out to the NuGet package? (I'd rather use that than importing the projects, obviously. :) )

@piedoom piedoom mentioned this pull request Jun 23, 2017
@piedoom
Copy link
Owner

piedoom commented Jun 23, 2017

I'm going to set up AppVeyor to automatically build master and upload to NuGet. I'll update soon.

@piedoom
Copy link
Owner

piedoom commented Jun 24, 2017

I've published the latest version with the PR changes to NuGet, and set up an automatic build/deploy when commits go to master. Let me know if you have any problems as I'm still pretty new to NuGet in general!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants