Skip to content
This repository was archived by the owner on Dec 27, 2018. It is now read-only.

Hi! What about my fixes? #5

Merged
merged 2 commits into from
Jul 10, 2015
Merged

Hi! What about my fixes? #5

merged 2 commits into from
Jul 10, 2015

Conversation

pramen
Copy link
Contributor

@pramen pramen commented Jul 10, 2015

No description provided.

pramen added 2 commits July 11, 2015 02:12
string formatting syntax error has been fixed in two methods:
update_content_by_id
delete_content_by_id
@rpcope1
Copy link
Owner

rpcope1 commented Jul 10, 2015

Wow, yeah I'll give these a look. I didn't realize anyone would pick this
library up so quick, but cool! I'll see if I can't get these merged in...

On Fri, Jul 10, 2015 at 5:19 PM, Alexander notifications@github.com wrote:


You can view, comment on, or merge this pull request online at:

#5
Commit Summary

File Changes

Patch Links:


Reply to this email directly or view it on GitHub
#5.

rpcope1 added a commit that referenced this pull request Jul 10, 2015
Merging in some fixes graciously provided to us from pramen. Thank you sir!
@rpcope1 rpcope1 merged commit 132d824 into rpcope1:master Jul 10, 2015
@rpcope1
Copy link
Owner

rpcope1 commented Jul 10, 2015

Fixes look good, I should have caught those before committing. I am still doing clean up on this library, but we should have unit tests on this library here in the next couple of weeks, so stay tuned, and thanks again for your help!

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

Successfully merging this pull request may close these issues.

2 participants