| Project: | Feedback |
| Version: | 6.x-2.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (duplicate) |
Issue Summary
I wanted the ability to forward a specific feedback message to an email address in order to 'assign' the task of fixing it to someone specific within our organisation. The attached patch adds a new contrib module called feedback_forward which ads a new column on the feedback report, with a link to a forwarding page. On the forwarding page, the message is displayed in full and a textfield is provided to allow multiple email addresses to be entered. On submission, email addresses are validated, and then an html email message is sent out to the specified addresses. The email is driven by a themeable template.
A couple of small alterations were needed to feedback.module in order to allow me to hook into it properly. I made the theme_feedback_admin_view_form() function a little more dynamic to allow extra columns to be rendered. I also made a small change to feedback_load to allow me to use it as a dynamic wildcard loader argument for the feedback message detail page.
I just whipped this together in under 2 hours! A lot more testing needs to done with it before it could be bundled up with the main module, but I just wanted to get it out there for others to see/try/report bugs.
Things that still need doing:
* ensure forward link doesn't display if user does not have feedback forwarding permissions
* add some configuration options to allow more control over the variables made available to the email template
* tidy up email template
* tidy up code
* test, test and then test again!
enjoy :)
| Attachment | Size |
|---|---|
| feedback-forward.patch | 1.86 KB |
Comments
#1
oops... that patch doesn't contain the contrib module! Please use this one instead!
#2
Three other things for the todo list:
1) add a new field or the forwarding page which would allow you to include a short message with the email.
2) add an option to mark the feedback message as 'processed' after sending the email.
3) do it all in a nice ajax style popup rather than a new screen!
#3
This version of the patch adds a notes text area to allow you to email some notes along with the feedback message.
It also allows you to mark a message a processed once the email(s) have been sent.
#4
Sorry, this is too much overhead for a simple functionality like e-mail notification. Also, I do not want to duplicate functionality of other modules.
Marking as duplicate of #353548: Add trigger/action for e-mail notification
#5
This module does not provide that same functionality as the Email notification patch. I am subscribed to that thread and am aware of what it does. This module serves a purpose other than simply notifying an admin user of a new message. This module adds a 'feedback' link to each message in the report. Clicking on it brings up a screen from where you can forward the individual message to 1 or more specific email addresses. You can also include notes to go along with the email and choose to mark the message as processed after sending.
On our site, we have been finding this extremely useful as a way of processing the messages as we can forward individual messages to the most appropriate person to deal with the problem. It gives a much more fine grained control over who gets the emails, rather than just blanket sending the everyone.
Regarding extra overhead. All the code is provided in a separate module, so if you don't need the functionality, simply don't enable the module and there is no overhead. I ask you to try the patch before you pass it off as a duplicate - which it is not.
#6
I see. But I am sure there are other contrib modules that provide exactly this functionality, for example http://drupal.org/project/forward
Instead of duplicating those modules, I would accept a patch that integrates with one of them.
#7
well that module, again, provides different functionally and I don't think it would be suited to this task. We don't need any blocks, we don't need email tracking. It would be real overkill to ask people to install the forward module just so they can forward individual feedback messages to people. This module provides very simple functionality that is tightly liked to and customised for the feedback module
#8
That was an example. See http://drupal.org/search/apachesolr_search/forward?filters=type:project_... - there are plenty of duplicate efforts already (for this search term alone; you could also try "send" or "mail"). I really do not want to add another half-baked sub-module to this list.
#9
well, none of them really seem appropriately suited to this task either. The only thing that might be nice would be to integrate with http://drupal.org/project/messaging in order to be more flexible about how the messages get sent, but I see that as a possible future extension to this, rater than a different way of achieving the end goal.
#10
This can be handled with #353548: Add trigger/action for e-mail notification or #492094: Rules integration: rules event when submitting feedback