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

Unrequired route #110

Closed
nicolas-besnard opened this issue Jan 11, 2015 · 3 comments
Closed

Unrequired route #110

nicolas-besnard opened this issue Jan 11, 2015 · 3 comments

Comments

@nicolas-besnard
Copy link
Contributor

I figured it out that there is some route that aren't required for this project such as '/resource/sign_up' on 'devise_token_auth/registrations#new'. Is it possible to delete them on Devise ?

@lynndylanhurley
Copy link
Owner

This will require several of the devise mapping methods to be overridden (starting here).

devise_for is a really tricky method, and I try to avoid modifying it whenever I can. But I've been getting several issues a week related to the new routes not being used, so I'll probably end up fixing this when I get a chance.

@nicolas-besnard
Copy link
Contributor Author

I tried to play with Devise's route yesterday : I didn't end well :(. I got some issues with the prefix of my routes. Didn't get why.

@booleanbetrayal
Copy link
Collaborator

@nicolas-besnard - going to close this unless you feel it's still a pressing issue / have a PR for me to look at. can re-open as needed.

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

No branches or pull requests

3 participants