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.

Sign up for GitHub

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

Jump to bottom

Post sanitation process does not remove emoticons that prevent db entry #874

Closed
AramZS opened this issue Dec 11, 2016 · 5 comments
Closed

Post sanitation process does not remove emoticons that prevent db entry #874

AramZS opened this issue Dec 11, 2016 · 5 comments
Labels
Drop-to-next-milestone-candidate Test
Milestone
5.1

Comments

@AramZS
Copy link
Member

AramZS commented Dec 11, 2016 •
edited
Loading

Emojis in article bodies or content via Nominate This.
@AramZS
Copy link
Member Author

AramZS commented Dec 11, 2016

http://stackoverflow.com/questions/12807176/php-writing-a-simple-removeemoji-function

@AramZS AramZS added this to the 4.4.x milestone Mar 21, 2017
@AramZS AramZS modified the milestones: 4.4.x, 5.1 Oct 17, 2017
@AramZS
Copy link
Member Author

AramZS commented Jan 19, 2018

Also check scottmac/opengraph#39

AramZS added a commit that referenced this issue Feb 5, 2018
@AramZS
Adjusting encoding processing. #874
16a10e7
@AramZS AramZS added the Test label Feb 5, 2018
@jccatalano
Copy link

To be clear on this issue, we need to test posts that have emojis in the body or urls or both?

@AramZS
Copy link
Member Author

AramZS commented Feb 20, 2018

Just the body!

@jccatalano
Copy link

This appears to be working (for the emojis tested).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Drop-to-next-milestone-candidate Test
Projects
None yet
Milestone
5.1
Development

No branches or pull requests
3 participants
@AramZS @regan008 @jccatalano