Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Links should not be converted to corner-bracket format when sending HTML email #220

Closed
boonebgorges opened this issue Mar 1, 2022 · 0 comments
Milestone

Comments

@boonebgorges
Copy link
Owner

When BPGES was first developed, BP emails were sent in plaintext. This meant that BPGES was forced to strip HTML tags from activity content. In the case of links, simple strip_tags() resulted in a loss of information, since the href attribute would go missing. So we introduced the ass_convert_links() utility, which took items of the form <a href="https://google.com">Google</a> to Google <https://google.com>.

This worked well for plaintext formatted emails. But when email is sent in HTML format, it causes problems in some clients, which interpret <https://google.com> as an invalid tag, and strip it from the content.

ass_convert_links() should only be applied when HTML email is disabled, or when generating the plaintext fallback.

boonebgorges added a commit that referenced this issue Jun 6, 2023
…mails.

We still do it in non-HTML emails, using the `ass_email_convert_html_to_plaintext()`
tool. In the case of plaintext fallback for HTML emails, the aforementioned
function already does the necessary conversion.

See #220.
@boonebgorges boonebgorges added this to the 4.0.5 milestone Jun 6, 2023
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

No branches or pull requests

1 participant