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

Allow Devise modules to register multiple routes #5630

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

Conversation

abevoelker
Copy link
Contributor

This change was precipitated by https://github.com/abevoelker/devise-passwordless, a Devise extension which adds a :magic_link_authenticatable strategy which enables registration and sign-in via emailed magic links.

This change is needed we can register both a :magic_link and a :session route and controllers, because we need to both provide a custom sessions controller as well as handle magic links. We are currently monkey patching this logic in and it works great - Devise.add_module continues to work exactly the same for modules with singular routes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant