Skip to content
This repository has been archived by the owner on Nov 19, 2024. It is now read-only.

Email dataset owner when new comment added #5

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Email dataset owner when new comment added #5

wants to merge 3 commits into from

Conversation

salsa-nathan
Copy link
Contributor

@salsa-nathan salsa-nathan commented Feb 6, 2020

Purpose

This pull request adds functionality to email dataset owners when a new comment has been added to their datasets.

Approach

Adds two new configuration options to the CKAN .ini file:

  1. to enable email notifications: ckan.comments.email_notifications_enabled (Defaults to False)
  2. to set who should receive the email notification: ckan.comments.email_notification_to (can be set to author, or maintainer. Defaults to author)

Emails the author or maintainer of the dataset based on the CKAN .ini file setting ckan.comments.email_notification_to, to notify them that a new comment has been added to their dataset.

Adds email notification subject and body templates to the extension in the /templates/emails/ directory.

Uses the CKAN job worker to send emails to avoid latency between comment submission and front-end display of updated comments.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant