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.
Campaigns include inappropriate links like "Edit" linking to the node/edit page. And if you are embedding a view, then the link "Edit view" is included in the email.
Note that the preview page does not have the problem.
Turning off the Contextual Links module entirely solved the problem. However, it would be better for MailChimp Campaigns to play nicely with that module, especially since it is in core and is so handy.
Thanks for all the great progress on this module.
Shai Gluskin
Comment | File | Size | Author |
---|---|---|---|
#9 | 1871726-contextual-links-user-2.patch | 1.08 KB | CountPacMan |
#5 | 1871726.5.contextual_links.patch | 971 bytes | Haza |
Comments
Comment #1
antpre CreditAttribution: antpre commentedHi there,
Thanks for the good job done with this module.
However I having the same issue as mentioned above.
In the campaign when I insert content from a drupal node using the macro the contextuel links show up on the rendered news lettre...
Disabling contextual link is a work around but certainly not a satisfactory solution.
For me the key selling point of this module is being able to insert content from drupal node.
Are you working on this issue ? Will a fix be available soon ?
Thanks
PS I am runing mailchimp with durpal core v 7.22
Comment #2
bountyla CreditAttribution: bountyla commentedYes, would deffenetly be nice with a better solution for this problem...
Comment #3
Jody LynnI was able to use hook_contextual_links_view_alter for my use case.
Comment #4
filipdebruyne CreditAttribution: filipdebruyne commentedAlso waiting for a solution for this issue. I can't send newsletters with 'edit this content' / 'remove this content' to my registered users ...
And the module contextual links is required for a bunch of Commerce modules, so I can't disable it.
Comment #5
HazaHere is a patch that forces drupal to render the node as an anonymous should be able to see them.
Comment #6
Aptalops CreditAttribution: Aptalops commentedsubscribe
Comment #7
ashleysman CreditAttribution: ashleysman commentedVerified that the patch in #5 solves this problem, thanks Haza.
Comment #8
Aptalops CreditAttribution: Aptalops commentedCan also verify that the patch in #5 solves this problem, thanks Haza.
Comment #9
CountPacMan CreditAttribution: CountPacMan at ThinkShout commentedPatch works with minor changes. Updated patch for 7.x-3.x-dev and kept code to remove contextual links.
Comment #10
jami CreditAttribution: jami at ThinkShout for Mailchimp commentedPatch pushed in 7.x-3.4: http://drupalcode.org/project/mailchimp.git/commit/053d35d
Thanks for everyone's work patching and reviewing!