Skip to content
This repository has been archived by the owner on Feb 22, 2022. It is now read-only.

Handle both themes and plugins email notifications #36

Merged
merged 1 commit into from Mar 27, 2020

Conversation

audrasjb
Copy link
Contributor

Let's update wp_autoupdates_automatic_updates_complete_notification and wp_autoupdates_send_email_notification functions to handle both plugin and theme email notifications.

Example of an email notification with both themes and plugins updates:

Capture d’écran 2020-03-27 à 11 05 55

Fixes: #16

@audrasjb audrasjb self-assigned this Mar 27, 2020
@audrasjb audrasjb requested a review from pbiron March 27, 2020 10:06
@audrasjb audrasjb added the enhancement New feature or request label Mar 27, 2020
@audrasjb audrasjb added this to the 0.4.0 milestone Mar 27, 2020
@audrasjb
Copy link
Contributor Author

Example of email notification when only themes were updated:

Capture d’écran 2020-03-27 à 11 16 50

Copy link
Contributor

@pbiron pbiron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. I like the idea of only sending 1 email for both kinds of updates. Avoids email overload and makes it more likely that admins will actually read them.

@audrasjb
Copy link
Contributor Author

Great. Let's merge this one 👍

@audrasjb audrasjb merged commit b6090a2 into master Mar 27, 2020
@audrasjb audrasjb deleted the add/theme-updates-email-notification branch March 27, 2020 14:52
@pbiron pbiron mentioned this pull request May 18, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Email notifications for themes
2 participants