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

Devise, Multiple Omniauth Providers #41

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

arbales
Copy link
Collaborator

@arbales arbales commented Jul 13, 2012

  • Update Sidekiq.
  • Update to latest NPM version and specify requirement in package.json.
  • Update to latest OmniAuth.
  • Drop Facebook-specific tables, add Identities table for multiple providers.

arbales and others added 2 commits July 12, 2012 16:55
`sidekiq_options queue: :name` and `npm start`
Switch to Devise form hand-rolled solution and add support for multiple
identity providers.
@ghost ghost assigned nixme Jul 13, 2012
@@ -1,12 +1,12 @@
GIT
remote: https://github.com/mperham/sidekiq.git
revision: 01bae6eef79d33438a489493b96a8f48f5bababe
revision: c232f5e53b2c19e41e75058d49323073f76194cb
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Switch to regular sidekiq while you're in here, not a git-specified one.

@nixme
Copy link
Owner

nixme commented Oct 11, 2012

rebase with master

@arbales
Copy link
Collaborator Author

arbales commented Mar 8, 2013

Should I still revisit this or is it irrelevant?

@nixme
Copy link
Owner

nixme commented Mar 8, 2013

Keep it open. Will use the diff to update master this weekend.

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

Successfully merging this pull request may close these issues.

2 participants