johndpjr / AgTern

19 stars 5 forks source link

Feature/142/implement sign up #151

Closed johndpjr closed 11 months ago

johndpjr commented 11 months ago

Closes #142

johndpjr commented 11 months ago

Thanks for letting me know - I fixed the issue. The backend logic wouldn't allow the duplicate usernames anyway, but it is much better practice to let the user know - so I handled the HTTP 400 error in the frontend and displayed the appropriate error.

Jwdegames commented 11 months ago

Confirmed that can't signup duplicate users

Jwdegames commented 11 months ago

Accidentally closed it instead of approving it.

Jwdegames commented 11 months ago

Should be good to approve - tested locally.