Hi!

I have created another version of HybridAuth module - which is attached to this issue (THERE IS NO UPGRADE PATH, yet or never). It is based on the code of the current code in git, but changes quite a few things introducing features from the successful uLogin module.
I don't see any sense in creating patches to the current version, I would rather suggest to commit it as a new branch, 7.x-2.x.

Changes and improvements:
- Integration with Token and Rules
- Pluggable icon packs via Ctools plugins system
- widget is now a form element
- Much more settings in the admin UI
- Full identities management
- Changed the way the user is created - it now creates a user using Drupal core user_external_login_register() function, not using the registration form.

Things to improve:
- Not all auth providers (supported by HybridAuth library) are supported yet
- Default icon packs are incomplete
- Only one widget type is now tested and working, 2 others need some love
- Migrate from Colorbox to Ctools overlay

I would love to become a co-maintainer of this module and continue improving it, including release for Drupal 6. This version was created for a contest at http://startupturbo.com/contest requiring the Drupal 7 module to allow social login through Mail.ru auth provider. It may contain some glitches and bugs to hunt, but it works.

Thanks
AndyB

CommentFileSizeAuthor
#6 hybridauth20121003.zip183.87 KBduozersk
hybridauth.zip171.81 KBduozersk

Comments

duozersk’s picture

duozersk
he/him
Primary language Russian
Location Moscow
commented
Title: HybridAuth - new version! » Offering to maintain HybridAuth - new version attached!
Assigned: duozersk » Unassigned
Category: task » support

Changing meta data according to http://drupal.org/node/251466

azin’s picture

azin commented

Many thanks for this - let me know if you need any help in running test cases or otherwise checking the module. I installed it on two portals and checked against Facebook, Google, LinkedIn and Twitter and it works just as good as before, and was very happy to see the profile merge page working well, as well as the new admin pages and username create functions.

Thanks again.

duozersk’s picture

duozersk
he/him
Primary language Russian
Location Moscow
commented

Here is a sandbox for this new version - http://drupal.org/node/1799842/git-instructions/7.x-2.x/nonmaintainer - it already has some fixes against the version attached in this issue.

AndyB

rafamd’s picture

rafamd commented

Thanks for taking the lead @duozersk!!

duozersk’s picture

duozersk
he/him
Primary language Russian
Location Moscow
commented

I did some progress in the above sandbox:
- Ctools overlay is implemented
- All 3 widget types are also done

Basically, the code in the sandbox can be considered production ready (as the architecture is mostly based on the tested uLogin module) - all the issues are welcome to the issue queue at http://drupal.org/project/issues/1799842?categories=All

Thanks
AndyB

duozersk’s picture

duozersk
he/him
Primary language Russian
Location Moscow
commented
StatusFileSize
new hybridauth20121003.zip183.87 KB

Here comes the latest snapshot of the sandbox - for those who don't have the ability to get the code from git.

Thanks to everyone who uses it, appreciate your support! ;)

Thanks
AndyB

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Active » Needs review

@duozersk you should write a mail (use contact form) to current maintainer to get co-maintainership

rwohleb’s picture

rwohleb
he/him
Primary language English
commented
Assigned: Unassigned » rwohleb
Status: Needs review » Fixed

Sorry for the delay in response. Things always get in the way.

@duozersk, I've made you a co-maintainer. Please merge in all your fixes. I've been meaning to get back to this module but I've been insanely busy, so I'm glad to have your help. Create issues for each of the new features/changes you want to implement so that we can discuss.

duozersk’s picture

duozersk
he/him
Primary language Russian
Location Moscow
commented

Robert,

Thanks a lot! I will commit my version to 7.x-2.x branch and then will create a new issue describing the architecture and changes so that we can discuss all the aspects.

Thanks
AndyB

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

(not verified) commented
Issue summary: View changes

fixed a typo