ThatConference / that-website

THAT Conference
https://www.thatconference.com
MIT License
32 stars 22 forks source link

Inconsistent profile field requirements #563

Open brettski opened 4 years ago

brettski commented 4 years ago

Describe the bug On initial create of a profile, and that profile is made public, a photo is not required. If you go back and edit your profile, a photo is then required to save your profile.

To Reproduce Steps to reproduce the behavior:

  1. Create a test account
  2. Select to create your profile
  3. fill in required fields and MAKE SURE to click the button on the last page to make your profile publicf
  4. save profile
  5. go back to edit your profile
  6. on the profile edit screen you will that now the photo file is required

Expected behavior if the photo is required for a public profile then it needs to be required in both cases. If it isn't required then it should be the same in both cases.

Device(s)/Browser(s) this is occurring on

saragibby commented 4 years ago

Is the to do here to make photo required in both cases? Or make it optional in both cases?

Also, is this reproducible when running the app locally without ability to authorize?

brettski commented 4 years ago

@csell5 what’s the final word on public profiles and photos? Are photos required for a public profile?