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

current_user replaced by sorcery_current_user (+ backward compatibility) #735

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rivsc
Copy link

@rivsc rivsc commented Oct 30, 2015

Fix devise compatibility.

… devise) + backward compatibility : always current_user (call sorcery_current_user) and current_user= (call sorcery_current_user=).
@rivsc
Copy link
Author

rivsc commented Oct 30, 2015

See #730 why Travis CI fail !

@arnvald
Copy link
Collaborator

arnvald commented Feb 13, 2016

Hey @rivsc!

While I wanted to do this myself (changing name of that method), I was wondering - why would you need Devise compatibility? Since these 2 libraries provide the same functionality, is there any reason to use both of them in the same project?

@rivsc
Copy link
Author

rivsc commented Feb 13, 2016

Hi @arnvald,

I use Sorcery for customers of new public website (easy to custom controllers and views) and Devise to manage the website (older Rails engine...). The two parts are in the same Rails engine.

@Ch4s3
Copy link
Contributor

Ch4s3 commented May 24, 2016

This is an interesting use case. @rivsc, what are you doing with this problem now?

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.

3 participants