It would be nice to integrate with rules so that some custom rules can be created for notification emails. For example I'm working on a site that would like to be able to notify the asker when the question is read by a moderator. I also imagine having to mark some questions as duplicates, which would not be published, but by adding a new field to mark it as a duplicate and linking to the duplicate issue, which should get emailed to the asker.

CommentFileSizeAuthor
#1 faq_ask-add-rules-integration-2237129-1.patch2.94 KBjojonaloha
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jojonaloha’s picture

jojonaloha CreditAttribution: jojonaloha commented
Status: Active » Needs review
FileSize
faq_ask-add-rules-integration-2237129-1.patch2.94 KB

Attached in an initial patch. It adds a Rules condition to check that a valid notification email is provided for a question and a Rules action to load the notification email address for a question, which can be used in subsequent actions. You can use the "Send mail" action to actually send any notifications.

stenjo’s picture

stenjo CreditAttribution: stenjo commented

Thank you for the patch. I can see the test is green, but there is no test added for the feature. Would you be able to add some code to the faq_ask.test to verify the functionality you've just added?
I really appreciate your effort, and will be reviewing this patch as soon as possible.