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.

Comments

frans’s picture

Hmmm 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.

jlyon’s picture

StatusFileSize
new1.85 KB

I 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.

frans’s picture

Component: Code » Documentation
Status: Needs review » Fixed

Added also some lines myself.
Tnx!

frans’s picture

Btw '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 :)

jlyon’s picture

StatusFileSize
new1.86 KB

Noticed 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.

jlyon’s picture

StatusFileSize
new1.86 KB

Argh, Facebook too. Should be: https://developers.facebook.com/apps.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.