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

Add form to update full name and email of user #1047

Merged
merged 5 commits into from
May 14, 2022
Merged

Add form to update full name and email of user #1047

merged 5 commits into from
May 14, 2022

Conversation

ysangkok
Copy link
Member

@ysangkok ysangkok commented Apr 3, 2022

Fixes #167. Note how the existing server endpoint is used, so this is only adding a front-end for existing functionality.

@ysangkok
Copy link
Member Author

@gbaz I have added pushed additional improvements:

  • The email can now only contain one at-sign, even during signup
  • The user details form now uses the same validation that the signup form uses
  • The user details form no longer allows you to submit a blank email, which would be rejected by the now-common server side email validation anyway

@gbaz gbaz merged commit cf4ed45 into haskell:master May 14, 2022
@ysangkok ysangkok deleted the janus/set-user-details branch May 16, 2022 02:19
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.

Account update page
2 participants