Once of the biggest problems I had setting this up was figuring out where I could sign up with each provider to get the the app and secret keys. This patch adds a new variable $provider->register_url, which is displayed as a link on the Provider Edit form (admin/structure/oauthconnector/facebook/edit).
There isn't any way to set this value within the UI, but it serves its purpose for the default providers.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | oauthconnector-readme_6.patch | 1.86 KB | jlyon |
| #5 | oauthconnector-readme_5.patch | 1.86 KB | jlyon |
| #2 | oauthconnector-readme.patch | 1.85 KB | jlyon |
| oauthconnector-add-register-url.patch | 2.9 KB | jlyon |
Comments
Comment #1
frans commentedHmmm I get the issue but am in doubt. I see this as documentation.
The presets are only meant to avoid repeating tasks and not for connections 'out of the box'. The thing is that those urls are changing, and changing and changing. All providers have their own rules and restriction. I think developers should be aware of that and that they need to know what they are doing (take alone the gazillion options for scopes with Facebook).
Will better documentation help?
Then we don't have to have 'documentation' in code and we can explain a little bit more about the service.
Comment #2
jlyon commentedI think that makes sense. Direct links seemed to make the most sense in terms of my use case, but I can see the value in putting this info in the documentation.
Here's my crack at a better README. I didn't include info on mapping fields to users' accounts because I don't quite understand the process yet myself. I also think a recipe for setting up oauthloginprovider and configuring it with oauthconnector would be really valuable. I have spent a couple hours trying to set this up with no luck.
Comment #3
frans commentedAdded also some lines myself.
Tnx!
Comment #4
frans commentedBtw 'fields' is still under development.
You might understand better what happens on an endpoint when installing devel (on the devel tab).
Fields now only works with text values. We need to rethink the whole info/fields strategy. Also this is done in connector.module...
Patches welcome :)
Comment #5
jlyon commentedNoticed that the URL for Twitter was wrong. It should be https://dev.twitter.com/apps. Here's a re-roll of the patch from #2.
Thanks for the hint about fields and devel. I'll try it out in a bit.
Comment #6
jlyon commentedArgh, Facebook too. Should be: https://developers.facebook.com/apps.