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

CommentFileSizeAuthor
#9 1871726-contextual-links-user-2.patch1.08 KBCountPacMan
#5 1871726.5.contextual_links.patch971 bytesHaza
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

antpre’s picture

antpre CreditAttribution: antpre commented
Version: 7.x-2.7 » 7.x-2.10

Hi 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

bountyla’s picture

bountyla CreditAttribution: bountyla commented

Yes, would deffenetly be nice with a better solution for this problem...

Jody Lynn’s picture

Jody Lynn
she/her
Primary language English
CreditAttribution: Jody Lynn commented

I was able to use hook_contextual_links_view_alter for my use case.

filipdebruyne’s picture

filipdebruyne CreditAttribution: filipdebruyne commented
Issue summary: View changes

Also 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.

Haza’s picture

Haza
Primary language French
CreditAttribution: Haza commented
Version: 7.x-2.10 » 7.x-2.x-dev
Status: Active » Needs review
FileSize
1871726.5.contextual_links.patch971 bytes

Here is a patch that forces drupal to render the node as an anonymous should be able to see them.

Aptalops’s picture

Aptalops CreditAttribution: Aptalops commented

subscribe

ashleysman’s picture

ashleysman CreditAttribution: ashleysman commented

Verified that the patch in #5 solves this problem, thanks Haza.

Aptalops’s picture

Aptalops CreditAttribution: Aptalops commented

Can also verify that the patch in #5 solves this problem, thanks Haza.

CountPacMan’s picture

CountPacMan CreditAttribution: CountPacMan at ThinkShout commented
Version: 7.x-2.x-dev » 7.x-3.x-dev
Status: Needs review » Reviewed & tested by the community
FileSize
1871726-contextual-links-user-2.patch1.08 KB

Patch works with minor changes. Updated patch for 7.x-3.x-dev and kept code to remove contextual links.

jami’s picture

jami CreditAttribution: jami at ThinkShout for Mailchimp commented
Status: Reviewed & tested by the community » Fixed

Patch pushed in 7.x-3.4: http://drupalcode.org/project/mailchimp.git/commit/053d35d

Thanks for everyone's work patching and reviewing!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.