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
Resend confirmation instructions #1267
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you! rebase, I think we fixed the test suite issue
@MaicolBen i rebased and fixed your comments. I had to force-push because of the rebase, is that ok? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
def create | ||
return head :bad_request if params[:email].blank? | ||
|
||
@resource = resource_class.dta_find_by(uid: params[:email].downcase, provider: provider) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
perfect 👌
@MaicolBen @lpsBetty Doesn't this allow non authorised users to send confirmation emails for other users? Furthermore, if I understood the code correctly it doesn't check if the user has been already confirmed before re-sending the email. What is more, resource_params method is not needed since they are not mass assigned to the resource. |
#920
#390