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

Prevent 🏢 in Outlook email <html> tag from getting converted to emoji #2706

Merged
merged 1 commit into from
Oct 12, 2016

Conversation

alanhartless
Copy link
Contributor

Q A
Bug fix? y
New feature? n
Related user documentation PR URL na
Related developer documentation PR URL na
Issues addressed (#s or URLs) #2440
BC breaks? n
Deprecations? n

Description:

Because :office was in an Outlook's email html tag, it was converted to the emoji equivalent of the short tag. This PR adds support to define exceptions which basically just reverts such cases that were converted to emoji's to begin with.

Steps to test this PR:

  1. Create/edit an email and update the source with <html xmlns="http://www.w3.org/1999/xhtml" xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office">
  2. Preview the email and view the source. :office: should remain intact.

Steps to reproduce the bug:

  1. Repeat the above only :office should have been converted to an office emoji

@alanhartless alanhartless added bug Issues or PR's relating to bugs ready-to-test PR's that are ready to test labels Oct 10, 2016
@alanhartless alanhartless added this to the 2.2.1 milestone Oct 10, 2016
@escopecz
Copy link
Sponsor Member

Yes, the :office: string in the html tag remains the same string after save 👍

@escopecz escopecz added the pending-test-confirmation PR's that require one test before they can be merged label Oct 11, 2016
Copy link
Contributor

@virlatinus virlatinus left a comment

Choose a reason for hiding this comment

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

Tested it the emoji icon was gone. It works!

@escopecz escopecz merged commit f19e7b9 into mautic:staging Oct 12, 2016
@alanhartless alanhartless deleted the bug-2440 branch October 24, 2016 01:53
MohHeader added a commit to MohHeader/mautic that referenced this pull request Dec 31, 2022
Building on
PR: mautic#2706 (comment)
issue: mautic#2440

Just added an extra case for exceptions : office : word
RCheesley added a commit that referenced this pull request May 31, 2023
* Add an extra Emoji exception for Outlook rendering

Building on
PR: #2706 (comment)
issue: #2440

Just added an extra case for exceptions : office : word

* add space to fix for CS Fixer

---------

Co-authored-by: Mohammad Abu Musa <m.abumusa@gmail.com>
Co-authored-by: Ruth Cheesley <ruth@ruthcheesley.co.uk>
mautibot pushed a commit to mautic/core-lib that referenced this pull request May 31, 2023
* Add an extra Emoji exception for Outlook rendering

Building on
PR: mautic/mautic#2706 (comment)
issue: mautic/mautic#2440

Just added an extra case for exceptions : office : word

* add space to fix for CS Fixer

---------

Co-authored-by: Mohammad Abu Musa <m.abumusa@gmail.com>
Co-authored-by: Ruth Cheesley <ruth@ruthcheesley.co.uk>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issues or PR's relating to bugs pending-test-confirmation PR's that require one test before they can be merged ready-to-test PR's that are ready to test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants