Skip to content

Conversation

@rogerhu
Copy link
Member

@rogerhu rogerhu commented Apr 24, 2017

  • Add constants

  • Delete perms=delete (do you know what this was for?)

@nesquena

Add constants

Delete perms=delete
@nesquena
Copy link
Member

nesquena commented Apr 24, 2017

IIRC, perms=delete or perms=write is a required flag on the FlickrAPI which had to be set when getting the access token if you wanted to be able to write/delete data. Without that, uploading photos on Flickr used to fail. Is there some cleaner (and easy to add) way to be able to add custom query params to the authorization URL? And/or we could just ignore this problem since uploading photos to Flickr is now not as core as it once was to the course. But I can imagine adding extra query params might be needed for other APIs as well. Will merge this for now.

@nesquena nesquena merged commit f06e1c1 into codepath:master Apr 24, 2017
@rogerhu
Copy link
Member Author

rogerhu commented Apr 24, 2017

That sucks. :) Just modified FlickrApi for ScribeJava to allow these permissions:

scribejava/scribejava#763

The syntax would then be new FlickrApi("delete") instead of FlickrApi.instance()

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