search

USGS-WiM / whispers

Wildlife Health Information Sharing Partnership event reporting system(WHISPers) v2
Other
2 stars 1 forks source link

don't allow end date in the future #1294

Closed JChipault closed 1 year ago

JChipault commented 1 year ago

Undesired Design or Confusing behavior Currently, folks can enter a location end date that is in the future on both the create event page and event details page. We initially purposefully allowed this but it's causing more problems than we predicted.

If Possible, Describe a Solution Don't allow location end dates that are in the future.

Time zones could make this tricky. If that causes too big of a problem and this is actually a big lift, let us know.

BlakeDraper commented 1 year ago

Added validation to prevent end date in future. Implementation live on test server.

JChipault commented 1 year ago

Working well on create event and also works when I add a new location to an already existing event - can we apply it to the event details page when I'm editing a pre-existing location too? I was just able to save 10/12/22 as an end date on 171092.

BlakeDraper commented 1 year ago

Done. Coming soon to test server.

JChipault commented 1 year ago

Looks good!

BlakeDraper commented 1 year ago

Done in v2.18.0.