The documentation does not state nor infer that the hybrid library must be installed separately. The module did not complete this task on its own. Not a big deal as it was easy to download and add to the libraries folder but you will have people asking about this in the future.

Also from working with a social login option previously I would highly recommend adding (in the module configuration page) a small snippet of code to allow for the flexibility of deploying the buttons anywhere. This should be simple enough as well, simple image with the target at the same href the image points to now (at least that's how I made it work when I was playing with the module).

I'll be keeping an eye on this one, as I am interested in switching our sites social login system over to something more simple like this at some point, any ETA on the next development release?

Comments

rwohleb’s picture

Assigned: Unassigned » rwohleb

There was a reason I only did a dev release ;)

All of the things you mentioned are planned. I just got busy with a new job. I'm going to be working on this again now that things are settling down. I'm totally open to patches, btw :)

jmomandown’s picture

That's great to hear, I will be more then happy to pass my modifications on as patches as you release things. Although most things are probably tailored to our site, I'm sure someone can find a use for them! Any ETA? And I am seeing you list many plugin options with facebook.. what are the plans there? (I may be grabbing the post function for some modifications that someone may be able to use :P )

One more tiny thing I noticed, although as you mentioned I am sure you are already aware. The name field is not being filled properly. I will take a look at some of the code when I get a chance, got law finals over the next couple weeks.

duozersk’s picture

Status: Active » Closed (won't fix)

Jason,

As we have just agreed with Robert (rwohleb) the 7.x-1.x branch is deprecated, please use the 7.x-2.x branch. It has instructions to download the library and put into 'sites/all/libraries'.
Please open another issue against 7.x-2.x branch if you have any other questions.

Thanks
AndyB

duozersk’s picture

Issue summary: View changes

Grammar