Attached is migrate module integration for comment_notify. The main motivation was to make sure that by default comment_notify doesn't attempt to send notifications for migrated comments, but this also will support migrating data to comment_notify.

CommentFileSizeAuthor
comment_notify-migration_support.patch2.21 KBmikeryan

Comments

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented

Seems like a great idea. I'm unlikely to test this. In your opinion is it ready to commit?

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Pittsfield, MA, USA
commented

Well... I'm confident in it accomplishing its main point, preventing notifications. As to whether it will actually work for importing comment_notify data, it's untested, I don't have any to test with... We could just remove the fields() method so it doesn't advertise functionality we can't confirm works.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Needs review » Fixed

That's good enough for me... ;)

Thanks!

http://drupalcode.org/project/comment_notify.git/commit/dbc897f

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.