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.

comment_notify-migration_support.patch2.21 KBmikeryan
PASSED: [[SimpleTest]]: [MySQL] 46 pass(es).
[ View ]


greggles’s picture

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

mikeryan’s picture

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

Status:Needs review» Fixed

Status:Fixed» Closed (fixed)

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