riparias / gbif-alert

GBIF Alert is a GBIF occurrence based alert system.
https://gbif-alert-demo.thebinaryforest.net/
MIT License
8 stars 2 forks source link

Filter the observations per province or per region #176

Closed niconoe closed 2 years ago

niconoe commented 2 years ago

Feedback received by email:

To further improve the alert website, it would be nice to be able to filter the observations per province or per region (Flanders/Wallonia/Brussels) in the table. This would improve the usability for the managers because the responsible agency is different in these regions. In addition, it would be nice to be able to click on the map and receive the data of the observation immediately. It is otherwise just guessing on the map which species was observed there. I have no idea how easy these suggestions are to implement, so hopefully these are useful.

niconoe commented 2 years ago

If I understand correctly, the Area filter already allows selcting regions, and adding provinces there wouldn't technically be a problem at all (just load a province shapefile in the system and we should be good).

niconoe commented 2 years ago

The reporter confirmed that I missed the "Region" option, and that an additional "provinces" one is not necessary in his opinion.

damianooldoni commented 2 years ago

Feedback received by email on Jul 27, 3:16PM:

Volgt er een mogelijkheid om per provincie de alerts te selecteren?

I think adding provinces polygons can still be an added value (people working in provicincial organizations) and technically shouldn't be complicated, as @niconoe mentioned above in https://github.com/riparias/early-warning-webapp/issues/176#issuecomment-1183282588.

niconoe commented 2 years ago

This is now implemented (I also prefixed the different areas by there type - Basin/province/region) to make things clearer and grouped by type:

Screenshot 2022-08-02 at 12 50 38
damianooldoni commented 2 years ago

Great! Thanks for implementing this so fast. As basins and regions are very few and in general way more used than provinces, I would leave provinces as last ones. Something like:

However, I see how things are now probably easy as the filters are now alphabetically ordered. If changing the dropbox order means messying up things, then forget this comment 😃

niconoe commented 2 years ago

Yes, the filtering is currently alphabetical, I wouldn't change the logic/overcomplicate things at this step just to avoid this small inconvenience. A quick and dirty way to circumvent that would be to add a prefix such as A) Basin: Dijle / Dyle, B) Region: Bruxelles, ...

damianooldoni commented 2 years ago

Indeed, @niconoe, leave as it is: it's just a detail after all. I would not add a), b), c). Once this goes to prod, please, let me know so that I will contact the interested user. Thanks.

niconoe commented 2 years ago

It's already in prod!

damianooldoni commented 2 years ago

Indeed! I was still on dev without noticing it. Thanks. I think you can close this issue.