Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Sending E-Mail Notification Once
Addition for the
frosh:monitor
CommandThis addition introduces a configuration feature to send an E-Mail notification only once within a specified time frame. The goal is to reduce spam by preventing repeated notifications when a mail has already been sent.
There are two settings:
To implement this, I moved the email-sending functionality to its own method to keep the code clean and maintainable.
Persistent E-Mail Sent Flag
To track when an E-Mail has been sent, I use the cache instead of the database. This approach avoids writing to the database or creating a migration if changes are needed later.
The cache is a suitable solution because we can simply check if the cache is set. If it isn't, the notification mail is sent and the cache key added.
I hope the use of the cache is acceptable in this context.