search

AlexanderZaytsev / domp

Devise & Omniauth Multiple Providers
MIT License
106 stars 15 forks source link

fixed issue with linking multiple providers with same email account. #10

Closed dhruvparmar372 closed 10 years ago

dhruvparmar372 commented 10 years ago

There is some issue when we try to link two providers with the same email account. E.g

  1. Sign in with google.
  2. Sign out.
  3. Sign in with facebook.

Considering both accounts use the same email only new authentication model should be created for the user with email of google. That doesn't happen as right now the create method only checks for signed_in_user. Fixed in this merge request

AlexanderZaytsev commented 10 years ago

Merged, thanks!