- Go to your My Account page and remove your email address - Click "Save" - AMO responds with "Profile Updated" Expected behavior is that this field should not be blank
Created attachment 330085 [details] [diff] [review] Display error on empty email field in user edit form While I don't think this has caused bug 434540, good catch anyway, Basil. Before, an empty email address was simply ignored, but it's a better experience if this is actually pointed out to the user as an error.
Assignee: nobody → fwenzel
Status: NEW → ASSIGNED
Attachment #330085 - Flags: review?(bkrausz)
Summary: Removing one's email address and saving → Empty email address in user edit form should throw error
Comment on attachment 330085 [details] [diff] [review] Display error on empty email field in user edit form Looks good to me :)
Attachment #330085 - Flags: review?(bkrausz) → review+
Thanks. Committed to r17167.
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
I now correctly get "There are errors in this form. Please correct them and resubmit." above the login form, and below the "Email address" textfield, "This field must not be empty." Validation of a bogus email address still works too: "s" yields, "This is not a valid email address." Verified FIXED
Status: RESOLVED → VERIFIED
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.