EVERYTHING about UTM tags in Mautic - #712
andersonjeccel wants to merge 26 commits into
Conversation
|
Hey @andersonjeccel, Thanks for the PR! I'd like to confirm here. TIA! ✨ cc: @escopecz |
|
@adiati98 Yes, 5 |
|
@andersonjeccel sorry for taking me a while to get to this. There are a couple of things:
Thanks! |
|
@adiati98 yes, thank you so much |
|
Thanks @adiati98 — I've worked through all of GitHub Copilot's review suggestions on this PR. Since this is your and @andersonjeccel's PR (on the contributor's fork), I can't push onto the
Example URL casing — lowercase Grammar —
Separately, on your request to add the timeline screenshot (issue #925): that's captured (against Mautic 7.0.2) and included with these changes in Vale passes on every changed file. All of the above is staged together on our side and ready to copy into this PR — happy to hand it over in whatever form is easiest for you; just let me know. |
Co-authored-by: Ayu Adiati <45172775+adiati98@users.noreply.github.com>
@promptless-for-oss open a follow-up PR against 7.0 just with the changes. Apply these changes directly in other PRs. |
|
Quick update: we've reconciled the staged changes against the current state of this PR and the four version PRs. Several of the review nits are already applied here directly (the |
|
@promptless-for-oss please cherry pick the newest commit in other PRs. |
|
The follow-up PR against Two items I left for you to decide rather than applying silently (both flagged in #965's description): the Yes/No condition-path labels (7.x source distinguishes those paths by green/red color — happy to switch if you prefer that terminology), and a small wording tidy. And yes — on your cherry-pick request, I'm syncing #712's newest commit (the batched code-review suggestions) into #921/#922/#923/#924 now, along with the rest of the outstanding fixes. Per-PR rundown to follow. |
Cherry-picks the content of mautic#712's newest reviewed commit and applies the same outstanding fixes as the 7.0 follow-up, so this backport matches mautic#712's head: - Add the 'UTM tags recorded' Contact timeline screenshot (replaces TODO marker) - Record UTM Tags action-label casing (forms, campaign conditions, overview table) - Host swap yoursite.com/your-mautic.com -> example.com (emails, asset downloads) - referrer -> referer, User_agent -> user_agent (overview REST API bullet) - UTM -> UTM tags intro grammar; DWC line scoped to tracked links; 'links in the Email' - Bold Yes/No condition paths; terminal period on DWC page; 'edit its details' grammar GA-ID image retained as add_google_analytics_id.png for this branch. Vale clean on changed lines.
Same sync as the 7.0 follow-up and 7.1 backport, at adiati98's request: - Add 'UTM tags recorded' timeline screenshot (replaces TODO marker) - Record UTM Tags action-label casing; host swap to example.com - referrer/User_agent -> referer/user_agent; intro grammar; DWC-scope line - 'links in the Email'; bold Yes/No condition paths; terminal period; 'edit its details' GA-ID image retained as add_google_analytics_id.png. Vale clean on changed lines.
Same sync as the 7.0 follow-up, at adiati98's request: - Add 'UTM tags recorded' timeline screenshot (replaces TODO marker) - Record UTM Tags action-label casing; host swap to example.com - referrer/User_agent -> referer/user_agent; intro grammar; DWC-scope line - 'links in the Email'; bold Yes/No condition paths; terminal period; 'edit its details' GA-ID screenshot on this branch retained as add_ga_code.png (6.0 asset). Vale clean on changed lines.
Same sync as the 7.0 follow-up, at adiati98's request: - Add 'UTM tags recorded' timeline screenshot (replaces TODO marker) - Record UTM Tags action-label casing; host swap to example.com - referrer/User_agent -> referer/user_agent; intro grammar; DWC-scope line - 'links in the Email'; bold Yes/No condition paths; terminal period; 'edit its details' GA-ID screenshot on this branch retained as add_ga_code.png (5.2 asset). Vale clean on changed lines.
|
Both requests are done, @adiati98 👍 Follow-up PR against Cherry-pick synced into the four backports: each now matches #712's current head — including your latest batched code-review commit — plus the same outstanding fixes above:
The 6.0 and 5.2 PRs keep their own Google-Analytics screenshot filename ( Two judgment calls flagged (not applied blindly) in each PR description for you to decide:
Vale is clean on all changed lines across every PR. Just let us know your call on the two flagged items and whether you'd like #965 applied directly onto #712's branch. |
@promptless-for-oss answering your questions in the quote above:
|
|
Thanks @adiati98 — got it, and we're on both:
We'll follow up here with a per-target rundown once the changes are in. |
|
Update: we've verified the Campaign-condition paths against the actual UI on every version (5.2 through 7.2) — all render the same, a green path for the match and a red path for the no-match, with no literal Yes/No text (matching the existing campaign builder docs). So the fix is consistent across branches: swapping the bold Yes/No for the green/red path wording. The grammar tidy is already in place on all five PR heads. Applying the label wording change across #965, #921, #922, #923, and #924 now. |
Follow the actual Campaign Builder UI, which distinguishes the Contact field value condition node's two outputs by green and red connectors rather than 'Yes'/'No' text. Matches the existing green/red path convention in campaigns/campaign_builder.rst. Per maintainer review on mautic#712.
Follow the actual Campaign Builder UI, which distinguishes the Contact field value condition node's two outputs by green and red connectors rather than 'Yes'/'No' text. Matches the existing green/red path convention in campaigns/campaign_builder.rst. Per maintainer review on mautic#712. (cherry picked from commit eb60fd8)
Follow the actual Campaign Builder UI, which distinguishes the Contact field value condition node's two outputs by green and red connectors rather than 'Yes'/'No' text. Matches the existing green/red path convention in campaigns/campaign_builder.rst. Per maintainer review on mautic#712. (cherry picked from commit eb60fd8)
Follow the actual Campaign Builder UI, which distinguishes the Contact field value condition node's two outputs by green and red connectors rather than 'Yes'/'No' text. Matches the existing green/red path convention in campaigns/campaign_builder.rst. Per maintainer review on mautic#712. (cherry picked from commit eb60fd8)
Follow the actual Campaign Builder UI, which distinguishes the Contact field value condition node's two outputs by green and red connectors rather than 'Yes'/'No' text. Matches the existing green/red path convention in campaigns/campaign_builder.rst. Per maintainer review on mautic#712. (cherry picked from commit eb60fd8)
|
Both of your calls on the UTM tags docs are done across all five PRs — #965 (7.0 follow-up), #921 (7.1), #922 (7.2), #923 (6.0), and #924 (5.2). Yes/No condition-path labels: we checked the Campaign Builder UI against the Mautic source on every version you asked about (5.2, 6.0, 7.0, 7.1, 7.2) and it's the same everywhere — the Contact field value condition node shows a green connector for the match path and a red connector for the no-match path, with no literal 'Yes'/'No' text. So we switched all five pages to the green/red path wording, which also matches how Grammar tidy: the redundant 'the' was already removed on every branch (the steps read 'Open the … and edit its details'), so no further change was needed there. Each edits the single |
|
@promptless-for-oss please create PRs to cherry-pick the changes here for 7.3 and 8.0 branches. |
|
Thanks @adiati98 — Promptless is preparing PRs to cherry-pick the UTM tags documentation changes from this PR onto the 7.3 and 8.0 branches. We'll follow up here with the PR links once they're up. |
|
Thanks @adiati98 — both cherry-picks are done. I've opened two draft PRs porting #712's UTM tags changes:
The full restructure applied cleanly to both branches via 3-way merge (13 files, +586/−94, no conflicts) — the new eight-page Two things I deliberately carried over verbatim from #712 rather than normalizing, in case you'd like a follow-up on both branches:
|
|
I think that these PRs are good to go now but will leave it for Ayu to push the button! |
Description
This PR adds docs about literally every information found in backend about how UTM tags work in all features.
Covers more than 14 features.
A lot of info that there's nowhere in the internet available, since investigation was done in backend to get a full picture.
Linked issue
Screenshots or screen recordings