I use the Nodecomment module on my site and I also use OG. In order to get all the groups subscription stuff, I need notifications. But Notifications doesn't work with Nodecomment. So I've got Comment Notify, which does work with Nodecomment, also installed. This makes for a messy UI for users because none of the thread subscriptions are listed in the Notifications section. I would like to just use Notifications for everything but I haven't the foggiest idea how to get it to work with Nodecomment.

I know this is a huge feature request but would benefit a lot of people. Once Advanced Forum 2.0 is out, I suspect many of its users will be switching to Nodecomment. Would be great to have this solved by then. :)

Thanks,

Michelle

Comments

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented

I've made some progress on this and I've got it sending the email. But the email looks like this:

Greetings Michelle,
Comment by [comment-author-name]: [comment-title]
[comment-body]
Read more [comment-url]

I've been grepping and searching and digging and I just can't figure out where those tokens are supposed to be replaced so I can put in some dsm's and figure out why they aren't.

Any hint?

Thanks,

Michelle

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented

Ok, now I've made enough progress to realize I'm going the wrong direction. Jumping into the comment handling isn't going to work because it seems to be re-loading the comment to get not only the info for the email but also to check if the user even has access to the comment.

Looks like I'll have to write a separate module for it that mimics, but doesn't latch onto, the comment handling.

Still welcome any help. :)

Michelle

shawnspqr’s picture

shawnspqr commented

Michelle,

I am trying to something very similar with the Nodecomment and Notifications modules. I need to be able to send out just a basic email stating that a comment has been posted to their original node and is waiting to be published. Is that achievable with what you have so far? If so, would mind writing out a quick description of your settings?

Thanks
Shawn

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented

Sorry, shawnspqr, I gave up and put this on hold until someone can help.

Michelle

jose reyero’s picture

jose reyero commented

There's some 'Comment Notify' replacement for the 4.x branch: Look into Notifications & Notifications Add-ons

Currently on beta testing, no plans to backport it though. (Testing and reporting would help)

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented

@Jose: Replacing Comment Notify isn't the same as working with Node Comments so I'm confused by your response here. Are you saying that there's code in the 4.x branch that will work with Node Comments?

Michelle

jose reyero’s picture

jose reyero commented

@Michelle, yes you're right, I see that won't work at all for Node Comments

I guess what we need here is some new plug-in defining this subscription type. I don't know too much about Node Comments though.

About your post on #1, that's because these are comment tokens, I guess what we need here it to use node tokens, and notify of node posts for that type.

It is possible, with 4.x to subscribe to new posts of a given node type and use a specific template for it. However, for the user to subscribe to the node the comment is for we'd need some more glue-code...

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented

I suggest setting this postponed for now unless someone else has time to work on it. I still would like to be able to use Notifications since I use OG but, for now, I'm just using Comment Notify. I don't know enough about what needs to be done here to be any help without investing a whole lot of time and that's just not something I can do right now.

Michelle

iva2k’s picture

iva2k commented

subscribing.

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented

Possible method in #871864: Node Comments Integration: Solution

Michelle

Taxoman’s picture

Taxoman commented

Subscribing.

YK85’s picture

YK85 commented

Hello,

I was wondering if there has been any further development in integration between Notifications module and Nodecomment module?

Thank you

rburgundy’s picture

rburgundy commented
Title: Nodecomment integration » Nodecomment Integration

+1 subscribing

The nodecomment module is great in allowing fields in the comment for images, videos etc.
I really hope that notifications could work with nodecomment module in the future.

Regards

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented
Status: Active » Closed (duplicate)

Even though my issue was first, I'm going to defer to the one that has code in it: #871860: Notifications Integrations: SOLUTION

I don't know if I'll ever get to writing anything. I'm not using Notifications anymore because of this incompatibility and may end up just rolling my own solution.

Michelle