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

Account form is broken #2377

Closed
faucomte97 opened this issue Nov 2, 2024 · 4 comments · Fixed by #2378
Closed

Account form is broken #2377

faucomte97 opened this issue Nov 2, 2024 · 4 comments · Fixed by #2378

Comments

@faucomte97
Copy link
Contributor

faucomte97 commented Nov 2, 2024

Describe the bug
The edit account details form, both for Teacher and Independent Student, is broken. Since 01/11/2024, for no apparent reason, the form has stopped processing data properly.

The form allows the user to do 3 things:

  • update their name
  • update their email address
  • update their password

In each case, the user needs to input their current password to submit the password. The name field is pre-populated, and every other field is optional.

However, now, the form acts like the new password field is required. Because of that, the user can't just update their name or email address without changing password, which is not the intended behaviour.

To Reproduce
Steps to reproduce the behaviour:

  1. Log in as a teacher or an independent student.
  2. Go to the update account details form.
  3. Change the value of the name field in the form.
  4. Input your password in the "current password" field.
  5. Click the submit button.
  6. Notice that the form complains that the new password field does not meet the password criteria, despite the field being optional.

This means the form submitted the new password field as an empty string instead of ignoring it.

Expected behaviour
The form should only check the fields that have been filled in.

Screenshots
Screenshot from 2024-11-02 02-47-19

Desktop (please complete the following information):
All

Good Places to Start
Independent Student Edit Account Form
Independent Student Edit Account View
Independent Student Edit Account Template

Teacher Edit Account Form
Teacher Edit Account View
Teacher Edit Account Template

Additional context
This is causing some of our existing tests to fail.

@SKairinos SKairinos self-assigned this Nov 4, 2024
@SKairinos SKairinos linked a pull request Nov 4, 2024 that will close this issue
@faucomte97 faucomte97 reopened this Nov 4, 2024
@faucomte97
Copy link
Contributor Author

LGTM

1 similar comment
@lauracumming
Copy link

LGTM

@lauracumming
Copy link

LGTM (PROD)

@faucomte97
Copy link
Contributor Author

LGTM

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

Successfully merging a pull request may close this issue.

3 participants