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

[Emojis in Markdown] #1653

Merged

Conversation

juanmanuelramallo
Copy link
Contributor

@juanmanuelramallo juanmanuelramallo commented Jan 25, 2019

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Documentation Update

Description

  • Adds the ability to add emojis using colons :joy: :tada:
  • The emoji conversion is added to the markdown parser labor
  • When no emoji is found with the alias, same text is returned
  • No assets added (no emoji images), so this will only be supported when the browser supports emojis (https://blog.getemoji.com/post/57054354336/which-browsers-support-emoji)
  • Emoji conversion logic is out of markdown parser labor
  • Adds its own spec

Related Tickets & Documents

#203
#849

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

Not applicable

Added to documentation?

  • docs.dev.to
  • readme
  • no documentation needed

@pr-triage pr-triage bot added the PR: unreviewed bot applied label for PR's with no review label Jan 25, 2019
- Adds the ability to add emojis using colons 😂 🎉
- The emoji conversion is added to the markdown parser labor
- When no emoji is found with the alias, same text is returned
- No assets added (no emoji images), so this will only be supported when the browser supports emojis (https://blog.getemoji.com/post/57054354336/which-browsers-support-emoji)
- Emoji conversion logic is out of markdown parser labor
- Adds its own spec
@juanmanuelramallo
Copy link
Contributor Author

@benhalpern Thoughts on this?

@pr-triage pr-triage bot added PR: reviewed-approved bot applied label for PR's where reviewer approves changes and removed PR: unreviewed bot applied label for PR's with no review labels Jan 28, 2019
@benhalpern benhalpern merged commit 49c9b5b into forem:master Jan 28, 2019
@pr-triage pr-triage bot added PR: merged bot applied label for PR's that are merged and removed PR: reviewed-approved bot applied label for PR's where reviewer approves changes labels Jan 28, 2019
@benhalpern
Copy link
Contributor

Congrats! Feel free to write a changelog post (tagged changelog and meta) 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: merged bot applied label for PR's that are merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants