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

Subscribe to comments posted in a page #40

Open
danirus opened this issue Jul 18, 2022 · 1 comment
Open

Subscribe to comments posted in a page #40

danirus opened this issue Jul 18, 2022 · 1 comment
Labels
enhancement New feature or request

Comments

@danirus
Copy link
Member

danirus commented Jul 18, 2022

Following on from this conversation in django-comments-xtd, a new feature is proposed to allow users to get notified on new comments posted on a page without having to post a comment themselves.

The implementation could consist of (this is a possible way to do it) writing a new model, say ObjectSubscription, with a content_type and object_pk fields and a ManyToMany point to ObjectSubscriber representing either a registered user or an external (verified) email address.

Then a function could be listening for the signal comment_was_posted and sending a notification to all the ObjectSubscribers whose ObjectSubscription (content_type and object_pk) was the subject of a new comment. More details can be found in the aforementioned conversation in django-comments-xtd.

@danirus danirus added the enhancement New feature or request label Jul 18, 2022
@jdbit
Copy link

jdbit commented Jul 20, 2022

Hi,
What if I left a comment and I don't want to get notifications about other comments on the page (not related to my comment), but I would like to get notifications about replies to my comments. I would make a subscription on comments replies automatic (no checkbox in the comment form needed), but instead add a link to the email notification template with ability to unsubscribe. Maybe we can use the same ObjectSubscription model for subscriptions on comment replies too. I hope there will be at least a posibilit to implement this via signals.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants