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.

Sign up for GitHub

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

Jump to bottom

Add adminIsNotified to sites and aggregate emails #283

Open
wants to merge 1 commit into
base: devel
Choose a base branch
from
Open

Add adminIsNotified to sites and aggregate emails #283

wants to merge 1 commit into from

Conversation

nlsvgtr
Copy link
Collaborator

@nlsvgtr nlsvgtr commented Jan 23, 2023

The API sends daily notifications to administrators about sites that have issues.
These notifications have two inconveniences:

  • one email is sent for each site with issues
  • since crontabs are running on the api: multiple api's means multiple mails

This update adds a 'notificationSent' paramater to fix the second, while the content of emails is aggregated per email address to fix the first.

@nlsvgtr nlsvgtr requested a review from LorenzoJokhan January 23, 2023 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LorenzoJokhan LorenzoJokhan Awaiting requested review from LorenzoJokhan

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@nlsvgtr