search

opendatateam / udata

Customizable and skinnable social platform dedicated to open data.
http://udata.readthedocs.org
GNU Affero General Public License v3.0
241 stars 88 forks source link

Temporal coverage with only start date #3192

Open magopian opened 1 week ago

magopian commented 1 week ago

Fixes #1268

Code from @thibaudDauce: https://github.com/opendatateam/udata/compare/temporal_coverage_with_only_start_date?expand=1

Was copy/pasted from his branch, because cherry-picking led to humongous conflicts for some reason...

magopian commented 1 week ago

udata (old) admin daterange may crash as well?

Regarding the (old) admin daterange, I've just tested and it seems to work as usual, no crashes if you provide both start and end date (this behavior wasn't changed by this PR as far as I can tell).

Not providing an end date shows a frontend validation error asking for both the start and end date.

One thing that fails in the old admin: editing a dataset that has no end date. In this case, the dataset can't be updated unless an end date is provided.

I'll work on allowing that in this PR.

magopian commented 1 week ago

Updated the PR with a change to the old admin so it also allows setting/editing a temporal coverage with only a start date.