search

Nerds-Who-Code / Mental-Health-Tracker

mental-health-tracker.vercel.app
MIT License
6 stars 6 forks source link

Age input/Create account #29

Closed med4kat closed 2 years ago

med4kat commented 2 years ago

Describe the bug Age input on Create account page takes a negative number.

To Reproduce Steps to reproduce the behavior:

  1. Go to http://localhost:3000/register
  2. Click on Age
  3. Enter -100
  4. See no given error

Expected behavior See an error message that age should be positive and an integer.

Screenshots

Screenshot 2022-07-17 at 22 18 31
med4kat commented 2 years ago

Do we even need the age?

steph-koopmanschap commented 2 years ago

On which branch is this? If I am corrrect this is already fixed on the latest branch I submitted (delete-entry) branch.

I think we wanted to know the user's age to give mental health info and advice for a specific age. My suggestion is to ask for the age once the user already registered, maybe it can be changed in the future.

med4kat commented 2 years ago

Got it! Sorry my bad.