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

added path to put_photo #104

Closed
wants to merge 1 commit into from
Closed

added path to put_photo #104

wants to merge 1 commit into from

Conversation

nobane
Copy link

@nobane nobane commented Nov 21, 2013

Added the option to set the endpoint location for put_photo. This was needed in at least one instance to set the picture of an event (/EVENT_ID/picture).

@martey
Copy link
Member

martey commented Nov 26, 2013

This seems relatively straightforward, is backwards compatible, and explains why it is necessary.

My only complaint is that the addition of text to line 194 makes it 86 characters long, which breaks PEP8 compatibility.

I might normally ask you to make the change and rebase the pull request, but I think it makes sense to wrap this in the next milestone, so I should be able to handle fixing this myself. Thanks!

@martey martey self-assigned this Jan 26, 2015
@martey martey closed this in 80afaac Jan 26, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants