Skip to content

current_user replaced by sorcery_current_user (+ backward compatibility) - #735

Open
rivsc wants to merge 1 commit into
NoamB:masterfrom
rivsc:master
Open

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

Conversation

@rivsc

@rivsc rivsc commented Oct 30, 2015

Copy link
Copy Markdown

Fix devise compatibility.

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

rivsc commented Oct 30, 2015

Copy link
Copy Markdown
Author

See #730 why Travis CI fail !

@arnvald

arnvald commented Feb 13, 2016

Copy link
Copy Markdown
Collaborator

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

rivsc commented Feb 13, 2016

Copy link
Copy Markdown
Author

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

Ch4s3 commented May 24, 2016

Copy link
Copy Markdown
Contributor

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