Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
I asked for Mail Editor module support here: #1878340: Privatemsg support but the developer told me:
Mail Editor is ready for every client module that wants to use it.
So I guess the support must be given by you. Any chances?
Comment | File | Size | Author |
---|---|---|---|
#12 | privatemsg-1878642-12.patch | 3.83 KB | mrded |
Comments
Comment #1
odizle CreditAttribution: odizle commentedany news about this? I would also like to have this option. My use case is that I need to make Hebrew templates with the RTL writing directions.
Comment #2
ptmkenny CreditAttribution: ptmkenny commentedWhat exactly is required to support Mail Editor? Is there something that Privatemsg does in a non-standard way that needs to be fixed? Otherwise, based on the Mail Editor maintainer's post, Privatemsg should be compatible.
Comment #3
warmth CreditAttribution: warmth commentedNo idea, I'm just reporting what I have been told. I hope you can figure it out because it will be a great addition.
Comment #4
salvisLook at the mail_edit/modules directory for examples.
Comment #5
ptmkenny CreditAttribution: ptmkenny commentedOk, here's a tentative patch that adds a new submodule offering integration based on the logintoboggan example. Let me know if it works for you.
Comment #6
ptmkenny CreditAttribution: ptmkenny commentedComment #8
ptmkenny CreditAttribution: ptmkenny commented#5: privatemsg-mail-edit-integration-submodule-1878642-5.patch queued for re-testing.
Comment #9
mrded CreditAttribution: mrded commentedI got the warning then I try to apply patch:
Comment #10
mrded CreditAttribution: mrded commentedWhy do you use hooks from another module? O_o
What is the reason to check module? This module is in dependencies.
privatemsg module has 'pm_email_notify' key only.
I guess that this patch will never works.
Comment #11
ptmkenny CreditAttribution: ptmkenny commentedOk, I don't really understand how to integrate with Mail Editor then. @mrded Please supply your own patch if you find a way to get it working.
Comment #12
mrded CreditAttribution: mrded commentedPlease, take a look my patch.
Comment #13
mrded CreditAttribution: mrded commentedAny progress, guys?
Comment #14
mrded CreditAttribution: mrded commentedComment #15
mrded CreditAttribution: mrded commented12: privatemsg-1878642-12.patch queued for re-testing.
Comment #16
ivnish CreditAttribution: ivnish commentedComment #17
andypostD7 is not yet outdated