Skip to content

AO3-7517 Disable phone number detection in emails on iOS#5925

Open
not-varram wants to merge 5 commits into
otwcode:masterfrom
not-varram:AO3-7517
Open

AO3-7517 Disable phone number detection in emails on iOS#5925
not-varram wants to merge 5 commits into
otwcode:masterfrom
not-varram:AO3-7517

Conversation

@not-varram

Copy link
Copy Markdown
Contributor

Pull Request Checklist

Issue

https://otwarchive.atlassian.net/browse/AO3-7517 (Please fill in issue number and remove this comment.)

Purpose

What does this PR do?

Prevent IOS from considering numerical strings of certain lengths as phone numbers via adding a certain meta tag to the email.

Credit

varram (he/him)

@pmonfort
pmonfort self-requested a review July 19, 2026 03:32

@pmonfort pmonfort 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 working on this! The update looks good, I left a couple of small comments

Comment thread app/views/layouts/mailer.html.erb
Comment thread spec/support/shared_examples/mailer_shared_examples.rb
Co-authored-by: Pablo Monfort <pmonfort@gmail.com>
Comment thread spec/support/shared_examples/mailer_shared_examples.rb Outdated
Co-authored-by: Pablo Monfort <pmonfort@gmail.com>

@pmonfort pmonfort 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.

Looks great, thanks for addressing the comments!

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.

2 participants