Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
I find it very undescriptive when I recieve a message from Drupal with the subject "message from username".
This patch adds a "subject" field for the contact.module which the user can fill, and would tell you what they want right away.
Oh, and it helps group the 'conversation' on Gmail into something meaningful.
(Note, I have not tested this since I do not have a CVS installation at the moment. Appreciate if someone can test it).
Comment | File | Size | Author |
---|---|---|---|
#12 | Drupal.fix-contact-module-ws.patch | 1.42 KB | Gábor Hojtsy |
contact.module-subject.patch | 1.93 KB | kbahey | |
Comments
Comment #1
kbahey CreditAttribution: kbahey commentedDid this patch make it? Any objections to it?
Comment #2
Dries CreditAttribution: Dries commentedI'll commit this patch when there is more demand for it.
Comment #3
killes@www.drop.org CreditAttribution: killes@www.drop.org commentedI'd like to see this applied. An email got to have a subject.
Comment #4
kbahey CreditAttribution: kbahey commentedThis is badly needed.
Just today I got a message from walkah and Gmail grouped it in a previous conversation with someone else, just because the subject is "message from kbahey".
We have to get this to CVS.
Comment #5
Morbus Iff+1 from me.
Comment #6
tangent CreditAttribution: tangent commentedThere is an added newline in the foreach which seems unnecessary. Some lines also have tabs which disrupts the indentation.
Otherwise, +1 on the functionality.
Comment #7
(not verified) CreditAttribution: commented+1 from me too. Since this is a simple useability extension, I hope that it will get into 4.6.
Comment #8
Gábor HojtsyThe previous 'me' was me :)
Comment #9
Dries CreditAttribution: Dries commentedCommitted to HEAD.
Comment #10
kbahey CreditAttribution: kbahey commentedI think we missed something in this patch.
The form_textfield has an argument that the maximum is 8 characters.
This is too short. Should be 30 characters or more. Line 125 in HEAD.
Comment #11
Dries CreditAttribution: Dries commentedFixed in HEAD. Thanks.
Comment #12
Gábor HojtsyWhitespace fixes after the patch...
Comment #13
Steven CreditAttribution: Steven commentedSeems this last patch was also applied by Dries...
Comment #14
(not verified) CreditAttribution: commented