Skip to content

current_user replaced by sorcery_current_user (+ backward compatibility)#735

Open
rivsc wants to merge 1 commit intoNoamB:masterfrom
rivsc:master
Open

current_user replaced by sorcery_current_user (+ backward compatibility)#735
rivsc wants to merge 1 commit intoNoamB:masterfrom
rivsc:master

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