search

nens / lizard-catalogue

Lizard Catalogue
https://demo.lizard.net/catalogue/
0 stars 0 forks source link

Filter period in timeseries selection doesn't cover use cases #249

Closed lexvand closed 3 years ago

lexvand commented 3 years ago

See: https://github.com/nens/lizard-catalogue/issues/220#issuecomment-769171512

Additionally, I think it's wrong to use this start and end for both filtering locations and for the export period, especially when we let the datetime entries respond to the selected timeseries (which according to the title was a requirement in #220, but fortunately seems not yet implemented).

To make the filter work we need a special intersect filter. This should be discussed with backend, as it may not be very straightforward to add.

Also we need to have a critical look at the workflow, what use cases we want to support and what that means for possible improvements in the interface. See also #250.

lexvand commented 3 years ago

@joeriverheijden @LirryPinter Some unintended behavior we really need to have a look at. It makes quite an unprofessional impression if we release this, I'm afraid.

joeriverheijden commented 3 years ago

Let's discuss this today Lex. I think i don't really get the point while reading the comments..

lexvand commented 3 years ago

Was fixed by adjusting the filter.