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

Readme "Usage TL;DR" section - add confirm_success_url to required params? #955

Closed
pnewsam opened this issue Sep 13, 2017 · 2 comments
Closed

Comments

@pnewsam
Copy link
Contributor

pnewsam commented Sep 13, 2017

/ | POST | Email registration. Requires email, password, and password_confirmation params. A verification email will be sent to the email address provided. Accepted params can be customized using the devise_parameter_sanitizer system.

It took me a little while to figure out that RegistrationsController expected a confirm_success_url param in addition to email, password, and password_confirmation. (I'm using Rails in API mode with a React front-end sending the form post, and only received a 422 response). Could be helpful to others to add that to the Readme?

@lynndylanhurley
Copy link
Owner

Sure - can you send a PR?

@pnewsam
Copy link
Contributor Author

pnewsam commented Sep 13, 2017

Submitted, thanks!

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

2 participants