Skip to content

docs: Document save-time link validation for Emails#820

Merged
adiati98 merged 3 commits into
mautic:7.2from
Promptless:promptless/email-malformed-link-validation-7.2
Jul 4, 2026
Merged

docs: Document save-time link validation for Emails#820
adiati98 merged 3 commits into
mautic:7.2from
Promptless:promptless/email-malformed-link-validation-7.2

Conversation

@promptless-for-oss

Copy link
Copy Markdown
Contributor

Open this suggestion in Promptless to view citations and reasoning process

Adds a 'Link validation' note to the Tracking links in Emails section of channels/emails.rst, explaining that Mautic now blocks saving an Email that contains a malformed link href and shows an 'invalid URL' error, and what counts as a valid link. Based on mautic/mautic PR #16256.

Trigger Events


Tip: Configure how Promptless handles changelogs in Agent Settings 📋

Comment thread docs/channels/emails.rst Outdated
Comment thread docs/channels/emails.rst Outdated
Wrap the link validation note in vale off/on to allow the generic
'email' term in the error string, remove parentheses, and reword
'interrupting the send' to 'disrupting email delivery'.
@promptless-for-oss

Copy link
Copy Markdown
Contributor Author

Thanks @adiati98! I've applied your suggestion:

  • Wrapped the note in .. vale off / .. vale on so the generic "email" in the error string no longer trips Vale.
  • Removed the parentheses.
  • Reworded "interrupting the send" to "disrupting email delivery".

Pushed to the same branch.

@Thecesar85 Thecesar85 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Reviewed against the merged behavior in mautic/mautic#16256.

The new Link validation section accurately explains that malformed href values are rejected when an Email is saved, includes the same representative malformed value covered by the regression tests, and lists the supported URL forms documented by the implementation tests. The wording also correctly describes the send-time protection as defense against broken delivery.

All documentation checks are green. LGTM from my side.

@adiati98 adiati98 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the review, @Thecesar85! 🚀

@adiati98 adiati98 merged commit bd5272f into mautic:7.2 Jul 4, 2026
3 checks passed
@promptless-for-oss promptless-for-oss deleted the promptless/email-malformed-link-validation-7.2 branch July 4, 2026 05:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants