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

Improvement to Oauth Documentation #30

Open
saturdaywalkers opened this issue Sep 20, 2017 · 3 comments
Open

Improvement to Oauth Documentation #30

saturdaywalkers opened this issue Sep 20, 2017 · 3 comments
Assignees

Comments

@saturdaywalkers
Copy link

I had a lot of issues authorizing a server-side calls to Flickr (not all yours!).

But once I realised how it worked, it was very easy. Your script worked a treat.

Where I went wrong.

  • The old auth method info and script in Flickr::Upload
  • Then, I just looked at the 'Authenticate' example in this module. But I hadn't noticed the 'saved config' in the first example, so it died on a file-not-found error. So I gave up and looked elsewhere...

So, I suggest a small change.

In Section "Authenticate an OAuth API Object starting with saved configuration"

Either

  • Add: # this uses the stored configuration from the first example above
    Or
  • Add the key and secret variables to the example (which are the current flickr names for them), i.e. not from the saved config

To test it, you'll have to ask someone who has never used it before to follow the instructions :)

It might be worth adding this script to the package.

I'll chase up the errors in Flickr::Upload

Thanks for maintaining this package. Its appreciated!

Andrew

@lbmoore lbmoore self-assigned this Sep 27, 2017
@iamcal
Copy link
Owner

iamcal commented Oct 5, 2017

thanks for the feedback! are you able to send a pull request with whichever update you think makes the most sense? either of them sound good and i don't have a strong opinion either way

@lbmoore
Copy link
Collaborator

lbmoore commented Oct 5, 2017 via email

@saturdaywalkers
Copy link
Author

Great. But, will need someone who's never used the module before to try using it, in order to test it :)

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

No branches or pull requests

3 participants