search

mekurokito / powermonitor

This is the code for the power monitoring project
0 stars 0 forks source link

Validation is not done on update user form #26

Closed mattsains closed 10 years ago

mattsains commented 10 years ago

Duplicate: update a user on manage_users and enter their email as a@a.c This is not a valid email according to django because an email must end in at least two non-dot characters

To be more precise, only html5 validation is done on the email. a@a.c is accepted by html5 but not by django. Another way to reproduce this problem is by using a browser that does not support html5 input validation

mattsains commented 10 years ago

Appears to work on the edit profile page

Confirmed that there is absolutely no validation on manage users. Since this was copied everywhere, there's probably no validation anywhere

HoboProgrammer commented 10 years ago

seeems to make sense bud

On Wed, Oct 29, 2014 at 2:23 PM, Matthew Sainsbury <notifications@github.com

wrote:

Appears to work on the edit profile page

Confirmed that there is absolutely no validation on manage users. Since this was copied everywhere, there's probably no validation anywhere

— Reply to this email directly or view it on GitHub https://github.com/mekurokito/powermonitor/issues/26#issuecomment-60914478 .

mattsains commented 10 years ago

Closed as too difficult and it mostly works anyway