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

add "related:12345" powertag which email notifies author of related post #646

Open
jywarren opened this issue Aug 2, 2016 · 10 comments
Open
Labels
enhancement explains that the issue is to improve upon one of our existing features help wanted requires help by anyone willing to contribute Ruby

Comments

@jywarren
Copy link
Member

jywarren commented Aug 2, 2016

Also, response:12345 should also notify the author of the original post. See this page for more on Power Tags, but basically adding the tag response:12345 where 12345 is the nid of a node, makes it appear in the sidebar of both nodes. Examples: https://publiclab.org/tag/response:*

This proposes a simpler version called related: which just adds the author of the orig. post to the list of people who get notified on creation. Not on additional commenting, though.

Could be done in DrupalTag:subscribers (here) or in the SubscriptionMailer: https://github.com/publiclab/plots2/blob/master/app/mailers/subscription_mailer.rb

This relates to the "related content" feature in the new Editor project: publiclab/PublicLab.Editor#26

screenshot 2016-08-02 at 11 31 03 am

@jywarren jywarren added enhancement explains that the issue is to improve upon one of our existing features help wanted requires help by anyone willing to contribute Ruby labels Aug 2, 2016
@ananyo2012
Copy link
Member

Ok this should be an easy fix. So you wan't an email notification on note creation if that tag is present? A new mailer view and action should do it.

@jywarren
Copy link
Member Author

jywarren commented Aug 19, 2016

Yep! And we should consider how to display related notes. Maybe same place
as "response" notes?

@ananyo2012
Copy link
Member

Yes we can just make one more section for displaying it probably we can call it "Mentioned notes"

@jywarren
Copy link
Member Author

jywarren commented Aug 19, 2016

Well, the related tag could be in the body of the post or just in the tag
field, so maybe we should just say "related notes".

@ananyo2012
Copy link
Member

But there is already a term called "Related notes" for notes tagged with same tags.

@jywarren
Copy link
Member Author

jywarren commented Aug 20, 2016

It's true -- but do you think related:12345 should then be referred to as
"mentioned" instead, somehow? OK how about we rename the current same-tags
list as "Similar notes" and the new section will be "Related notes"?

@ananyo2012
Copy link
Member

@jywarren Yes we can definitely do that.

@grvsachdeva
Copy link
Member

@jywarren should we re-open this as a new issue? Thanks!

@jywarren
Copy link
Member Author

jywarren commented Mar 25, 2019 via email

@NitinBhasneria
Copy link
Collaborator

@jywarren Is this issue still pending.

@stale stale bot added the stale label Oct 7, 2020
@publiclab publiclab deleted a comment from stale bot Oct 8, 2020
@stale stale bot removed the stale label Oct 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement explains that the issue is to improve upon one of our existing features help wanted requires help by anyone willing to contribute Ruby
Projects
None yet
Development

No branches or pull requests

4 participants