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

Daily/weekly/monthly email digests #110

Open
schlessera opened this issue Feb 4, 2017 · 10 comments
Open

Daily/weekly/monthly email digests #110

schlessera opened this issue Feb 4, 2017 · 10 comments

Comments

@schlessera
Copy link
Contributor

We currently have the option of sending an email when spam is detected. This can result in a huge number of emails being sent.

It would be great to have the option to only send daily/weekly/monthly email digests.

There's already a proposed solution through a third-party wp_mail handler, but given the nature of what these emails do, Antispam Bee should already come with a clean solution for this out of the box.

This would replace the suggested solution in #26 .

This would also solve the cause of #84.

@schlessera schlessera mentioned this issue Feb 4, 2017
@websupporter websupporter added this to the 2.8.0 milestone Feb 5, 2017
@krafit krafit modified the milestones: 2.8.0, 2.9 Apr 20, 2018
@websupporter websupporter modified the milestones: 2.9.0, Future Release Jul 20, 2018
@websupporter
Copy link
Contributor

Was also suggested as a feature in the support forums: https://wordpress.org/support/topic/spambox-alert-possible/

@Zodiac1978 Zodiac1978 self-assigned this Apr 17, 2020
@Zodiac1978 Zodiac1978 modified the milestones: Future Release, 2.9.3 Apr 17, 2020
@Zodiac1978
Copy link
Member

I would like to try to code this feature.

@websupporter
Copy link
Contributor

That would be cool. I would like to move it to 2.10 though, since I would like to ship 2.9.3 quickly, if thats ok.

@websupporter websupporter modified the milestones: 2.9.3, 2.10 Apr 18, 2020
@Zodiac1978
Copy link
Member

This (paid) addon to "Better Notifications" does provide such a feature and if I understand correctly it is just collecting the mails and concatenates them:
https://betternotificationsforwp.com/documentation/add-ons/digest/

@Zodiac1978
Copy link
Member

wearerequired/digest#1 is still open, so at the moment #26 is closed but has no real solution, because the Digest plugin from required never supported ASB.

Added here for completeness and documentation of possible solutions.

@Zodiac1978
Copy link
Member

My idea so far:

If Mark as spam, do not deleteis enabled and Spam-Notification by email is enabled too we could add a new line:
Send notification email(s): Immediately | Daily | Weekly | Bi-weekly | Monthly
Time: xx:yy (if not set to Immediately)
Or maybe even more: Input field [x] and dropdown with hours/days/weeks/months

Or alternatively once email number goes past a specified threshold?

I don't like this idea from the forum, because false positives could lay around too long with such a configuration.

Mails could contain the info from the comment page (Tab: Spam) in a table:

+-----------------+----------+----------------+--------------------------------------------+
| Author (Mail) - IP | Comment | In Response To | Submitted On (with link to page/edit page?) | Spam Reason (if the setting is true!)
+-----------------+----------+----------------+--------------------------------------------+
| Actions Links: Not Spam | Delete Permanently
+----------------+----------+----------------+--------------------------------------------+

Link to the comment page (Tab: Spam) for doing bulk edits or other actions.

The recipient mail should use our filters:
antispam_bee_notification_recipients and antispam_bee_notification_subject

Possible problems and questions:

  • What happens if the digest schedule is bigger than the automatic deletion of the spam comments? (e.g.: Deletion after 6 days, weekly digest mail notifications) I think we shouldn't allow this.
  • I am not sure how to handle the first bulk. We could send all spam comments in the first mail after enabling the feature and then use the correct schedule to limit to the new spam, but this could result in a very big first mail. Or we could ditch the first check (assuming it is already done) and start with the first scheduled mail after the choosen timeframe.

I excited to hear your thoughts on this feature!

@websupporter
Copy link
Contributor

What happens if the digest schedule is bigger than the automatic deletion of the spam comments? (e.g.: Deletion after 6 days, weekly digest mail notifications) I think we shouldn't allow this.

👍

[...] how to handle the first bulk. [...] we could ditch the first check

I like this path.

The idea of the digest is to reduce the amount of "spam"-emails. When you are under a huge spam wave this could result in a lot of mails. I fear, now it will result in a huge email (potentially couple of MB) if we give detailed information about all spams. I would suggest to only show the last X (10?) recent comments.

I would suggest something like

Hi,
here is your recent spam digest.

# Statistics
==============================================
Type      | Amount | Link
----------+--------+--------------------------
Honeypot  | 1000   | to honeypot comments
----------+--------+--------------------------
Language  |  100   | to language comments
----------+--------+--------------------------
Regex     |  100   | to regex comments
----------+--------+--------------------------
Total     | 1200   | to all spam comments
==============================================

# Most recent spam comments
[overview over the last comments]

This way the digest would remain compact and still would you give easy access to all necessary information imo.

@planetahuevo
Copy link

Hi,
I would like to add one thing.
A lot of times, the admin of the website in settings is not the real admin of the website, but the owner.
It would be good to have a "to:" field so we can tell the plugin where do we want this emails to be sent to.

Thanks

@Zodiac1978
Copy link
Member

@planetahuevo We already have this feature as a filter:
https://antispambee.pluginkollektiv.org/documentation/#notification-by-email

antispam_bee_notification_recipients and antispam_bee_notification_subject

@planetahuevo
Copy link

I missed that this morning when reading the docs.
I ignored that option because it do not say anything on the gui, and I was reading all the other options.

It would be good to have an option on the gui or at least a text with a link on the settings page under the email check so we know this can be done.
As you were talking to add the option for daily/weekly/monthly, you could add the field for the email to?

@Zodiac1978 Zodiac1978 modified the milestones: 2.10, 2.11 Jun 24, 2021
@Zodiac1978 Zodiac1978 removed their assignment Oct 7, 2021
@pfefferle pfefferle self-assigned this Feb 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants