Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Inlude area in alert email #19

Open
timadriaens opened this issue Dec 23, 2023 · 3 comments
Open

Inlude area in alert email #19

timadriaens opened this issue Dec 23, 2023 · 3 comments
Assignees
Labels
enhancement New feature or request

Comments

@timadriaens
Copy link

Hi, I think it is more logical to include the area rather than the source dataset in the alert email. If you have set a few alerts, it is in fact nog very clear anymore which ones you want to look at. Area is the most important to a manager. Alternatively, both could be included of course. Lat/Lon is in fact also totally uninformative so perhaps these 2 columns can go and we use area instead there?

@timadriaens timadriaens self-assigned this Dec 23, 2023
@timadriaens timadriaens added the enhancement New feature or request label Dec 23, 2023
@damianooldoni
Copy link
Collaborator

Hi @timadriaens

In email there is a section called "Alert details" where the area is mentioned. An example follows here below for one of my alerts. Maybe you refers to the table with the occs examples? Mentioning the area has not really sense as it could be "Belgium" if no more specific area is selected. Moreover, it will be a redundant information as this info is already present as I mentioned before. Notice also that the details shown in the example table are the same as the ones shown in the table mode of the web application. In my opinion we shouldn't change it as lat lon are always present while municipality (just to mention the field which would make more sense) is not always filled in.

Alert details

  • Alert name: All IAS fishes in West Flanders
  • Species: Ameiurus melas, Channa argus, Fundulus heteroclitus, Gambusia affinis, Gambusia holbrooki, Lepomis gibbosus, Morone americana, Perccottus glenii, Plotosus lineatus, Pseudorasbora parva
  • Areas: West Flanders
  • Datasets: all
  • Email notifications frequency: Daily

@timadriaens
Copy link
Author

Indeed, I refer to the table

@damianooldoni
Copy link
Collaborator

Getting back to this a year later 😄

Adding the area the occurrence belongs to means intersecting areas and observations (again). Also, the area is not always unique. Just an example here:

A user creates an alert selecting as locations:

  • Bruxelles
  • Zenne

Which area should be linked to an observation falling in the Brussels part of the Zenne river basin? Brussels? Zenne? Both?

We could have many of this kind of situations. So, I am not really fan of adding area in the table with examples.

I agree that lat/lon is not really informative in example table. Adding municipality is way more interesting, if given, of course. It's quite precise, unique and retrieving it doesn't require any spatial operation.

So, my proposal is:

  • add municipality
  • leave lat/lon (as municipality could be not present in the data)

However, I think it's easier to have the same fields in the example table as in the table view of the early alert application. @niconoe: am I correct? Should we maybe add municipality in the table view as well?

@timadriaens: about your comment:

If you have set a few alerts, it is in fact not very clear anymore which ones you want to look at.

Users can (should) provide meaningful names to alerts. For example: "IAS mammals in Zenne and Dijle" is informative, while "Alert 1" is not.

@niconoe, any thought in general?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants