Email: Moderation email improvements #93

New Issue

Pablo Vazquez · 2024-04-29T13:00:54+02:00

Pablo Vazquez commented

2024-04-29 13:00:54 +02:00

Current

The current text for the email is lacking some important information:

Mention why user is receiving this email. (probably because the user is a moderator)
Mention how to stop receiving such notifications. (link to profile preferences)
Mention other notifications user may have missed. (link to notifications)

The information in it is also sorted in a way that important information can be lost when previewed:

Solution: Mention most important content first.

Current subject, body:

New Activity: {{ extension_author }} requested review {{ extension_type }} "{{ extension_name }}"

{{ extension_author }} requested review {{ extension_type }} "{{ extension_name }}": {{ extension_url }}

Proposal

Proposed subject, body:

{{ extension_type }} review request: "{{ extension_name }}"

{{ extension_author }} has requested a review for their {{ extension_type }} "{{ extension_name }}".
{{ extension_url }}

Read all notifications at https://extensions.blender.org/notifications/

You are receiving this email because you are a moderator subscribed to notification emails.
Unsubscribe by adjusting your preferences at https://extensions.blender.org/settings/profile/

For example, what used to be:

New Activity: Aiden McKown requested review Theme "Genuinely Disgusting"

Aiden McKown requested review Theme "Genuinely Disgusting": https://extensions.blender.org/approval-queue/genuinely-disgusting/

Would now be:

Theme review request: "Genuinely Disgusting"

Aiden McKown has requested a review for their theme "Genuinely Disgusting".
https://extensions.blender.org/approval-queue/genuinely-disgusting/

Read all notifications at https://extensions.blender.org/notifications/

You are receiving this email because you are a moderator subscribed to notification emails.
Unsubscribe by adjusting your preferences at https://extensions.blender.org/settings/profile/

### Current The current text for the email is lacking some important information: * Mention why user is receiving this email. (probably because the user is a moderator) * Mention how to stop receiving such notifications. (link to [profile preferences](https://extensions.blender.org/settings/profile/)) * Mention other notifications user may have missed. (link to [notifications](https://extensions.blender.org/notifications/)) The information in it is also sorted in a way that important information can be lost when previewed: ![gmail mobile](/attachments/aefaf201-d074-447a-a86a-4541c8ffeb4f) Solution: Mention most important content first. Current subject, body: > **New Activity: {{ extension_author }} requested review {{ extension_type }} "{{ extension_name }}"** > {{ extension_author }} requested review {{ extension_type }} "{{ extension_name }}": {{ extension_url }} ---- ### Proposal Proposed subject, body: > **{{ extension_type }} review request: "{{ extension_name }}"** > {{ extension_author }} has requested a review for their {{ extension_type }} "{{ extension_name }}". > {{ extension_url }} > > Read all notifications at https://extensions.blender.org/notifications/ > > You are receiving this email because you are a moderator subscribed to notification emails. > Unsubscribe by adjusting your preferences at https://extensions.blender.org/settings/profile/ For example, what used to be: ```plaintext New Activity: Aiden McKown requested review Theme "Genuinely Disgusting" Aiden McKown requested review Theme "Genuinely Disgusting": https://extensions.blender.org/approval-queue/genuinely-disgusting/ ``` Would now be: ```plaintext Theme review request: "Genuinely Disgusting" Aiden McKown has requested a review for their theme "Genuinely Disgusting". https://extensions.blender.org/approval-queue/genuinely-disgusting/ Read all notifications at https://extensions.blender.org/notifications/ You are receiving this email because you are a moderator subscribed to notification emails. Unsubscribe by adjusting your preferences at https://extensions.blender.org/settings/profile/ ```

image.png

73 KiB

Pablo Vazquez added the

Type

Enhancement

label 2024-04-29 13:00:54 +02:00

Anna Sirota referenced this issue

2024-04-29 19:08:39 +02:00

Basic email template for notifications #96

Anna Sirota referenced this issue from a commit

2024-05-02 14:04:24 +02:00

Basic email template for notifications (#96)

Márton Lente referenced this issue from a commit

2024-05-03 17:28:05 +02:00

Basic email template for notifications (#96)

Oleg-Komarov commented

2024-05-30 19:33:37 +02:00

Everything except the reason why a user is receiving the email was implemented.
This can be potentially done if we start propagating the Flag value from the follow relation to the Notification object, but I'm not sure if it is always possible to define unambiguously (e.g. there may be multiple reasons to receive the same notification - being a commenter and being an author).
Closing for now.

Everything except the reason why a user is receiving the email was implemented. This can be potentially done if we start propagating the Flag value from the follow relation to the Notification object, but I'm not sure if it is always possible to define unambiguously (e.g. there may be multiple reasons to receive the same notification - being a commenter and being an author). Closing for now.

👍 1

Oleg-Komarov closed this issue

2024-05-30 19:33:37 +02:00

Sign in to join this conversation.