Skip to content

Update link-tracking note for removal of legacy mautic:disable-tracking attribute in 8.0 - #920

Open
promptless-for-oss wants to merge 3 commits into
mautic:8.0from
Promptless:promptless/pr-17059-remove-disable-tracking-attribute-7.2
Open

Update link-tracking note for removal of legacy mautic:disable-tracking attribute in 8.0#920
promptless-for-oss wants to merge 3 commits into
mautic:8.0from
Promptless:promptless/pr-17059-remove-disable-tracking-attribute-7.2

Conversation

@promptless-for-oss

Copy link
Copy Markdown
Contributor

Open in Promptless

Mautic 8.0 removes support for the legacy mautic:disable-tracking HTML attribute used to opt an Email or Landing Page link out of click tracking; only data-mautic-disable-tracking="true" continues to work. The note in the "Tracking links in Emails" section of docs/channels/emails.rst previously described the legacy attribute as merely "deprecated," which understates the impact and would leave users of the legacy attribute unaware their link-tracking opt-out stops working after 8.0.

This updates that note to state that Mautic deprecated the legacy attribute and removes it in 8.0, that once removed the legacy attribute no longer disables link tracking, and that any Segment or Template Emails still using it should switch to data-mautic-disable-tracking="true". It also corrects the legacy attribute's spelling in the note (previously written mautic:disable:tracking; the actual attribute is mautic:disable-tracking).

Trigger Events

Comment thread docs/channels/emails.rst
Comment thread docs/channels/emails.rst
@adiati98 adiati98 added this to the 8.0 milestone Sep 1, 2026
@promptless-for-oss
promptless-for-oss marked this pull request as ready for review September 2, 2026 18:08
@TomasVotruba

Copy link
Copy Markdown

I cannot review this one, as I'm not enough front-end experienced

@adiati98

adiati98 commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

I cannot review this one, as I'm not enough front-end experienced

/aside @TomasVotruba gotcha. 👍🏼

@escopecz can you please review this, or who'd you recommended to review? TIA! ✨

escopecz
escopecz previously approved these changes Sep 4, 2026

@escopecz escopecz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is fine 👍

@adiati98

Copy link
Copy Markdown
Contributor

@promptless-for-oss please rebase to 8.0 branch.

@promptless-for-oss
promptless-for-oss changed the base branch from 7.2 to 8.0 September 10, 2026 13:00
@promptless-for-oss
promptless-for-oss dismissed escopecz’s stale review September 10, 2026 13:00

The base branch was changed.

@promptless-for-oss

Copy link
Copy Markdown
Contributor Author

In reply to this discussion

Done — PR #920 now targets the 8.0 branch as requested, which is the correct target under the branch rules (the source change, mautic/mautic #17059, is an 8.0 removal). The PR is open and mergeable against 8.0 with the same single-line note update to docs/channels/emails.rst; no content changed. No force-push was needed since the branch already sat on top of 8.0's history. Ready for re-review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants