Closed lisannewapstra closed 3 years ago
What needs to be done to make this work for users:
@LirryPinter Can you adjust the design with this second modal?
Modal with:
Design is updated.
Added a modal for selection the period for the export!
@lexvand @hoanphungt
@Derryrover Hi Tom,
Could you review this PR: https://github.com/nens/lizard-catalogue/pull/255/? Design link is in the above comment.
Kr, Hoan
@hoanphungt
Hi Hoan,
Overall looks and works good. Ready to merge
2 notes:
kr Tom
@lexvand @joeriverheijden Hi Joeri and Lex,
I think Tom's above comment makes sense here. Adding another modal on top of the timeseries selection modal seems to be quite cumbersome since it hides some of the information in the other modal. However, since I already made it, I think it would be best if I put it on staging and let you two and @LirryPinter experiment with this to see the pros and cons compared to the previous exporting method.
Kr, Hoan
Via Joeri
Analysis Support
I do not think it has to do with being logged in. And it also works with rasters. For timeseries, there seems to be a Start time required.
With Start
Without start
Is this by design?