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

feat: add emoji support #63

Merged
merged 5 commits into from Jul 24, 2023
Merged

Conversation

a0m0rajab
Copy link
Contributor

@a0m0rajab a0m0rajab commented Jul 22, 2023

Description

This PR added emoji support to the OpenGraph, at the beginning, it tried the proposed solution by @bdougie in using node-emoji, which it did not work, then it went to use the Satori emoji system, which solved the issue. The utils file were copied from vercel/satori repo without any change.

I kept the file without change and had all the API options to raise a discussion about which emojis to choose, right now we are using twemoji, but there are a couple of different options which you can try from here:

https://og-playground.vercel.app/

  • go to advanced
  • change emoji options.

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

  • πŸ• Feature
  • πŸ› Bug Fix
  • πŸ“ Documentation Update
  • 🎨 Style
  • πŸ§‘β€πŸ’» Code Refactor
  • πŸ”₯ Performance Improvements
  • βœ… Test
  • πŸ€– Build
  • πŸ” CI
  • πŸ“¦ Chore (Release)
  • ⏩ Revert

Related Tickets & Documents

fixes #50

Mobile & Desktop Screenshots/Recordings

Before:
161

after:

161

Added tests?

  • πŸ‘ yes
  • πŸ™… no, because they aren't needed
  • πŸ™‹ no, because I need help

Added to documentation?

  • πŸ“œ README.md
  • πŸ““ docs.opensauced.pizza
  • πŸ• dev.to/opensauced
  • πŸ“• storybook
  • πŸ™… no documentation needed

[optional] Are there any post-deployment tasks we need to perform?

[optional] What gif best describes this PR or how it makes you feel?

@a0m0rajab a0m0rajab changed the title Feat add emoji feat: add emoji support Jul 22, 2023
@a0m0rajab
Copy link
Contributor Author

Emoji Styles:
Twemoji:
image
Fluent:
image
Fluent Flat:
image
Noto:
image
Blob:
image
Open Moji:
image

@brandonroberts
Copy link
Contributor

Nice work @a0m0rajab!

Copy link
Member

@jpmcb jpmcb left a comment

Choose a reason for hiding this comment

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

πŸ‘πŸΌ Looks great!

src/social-card/templates/highlight-card.template.ts Outdated Show resolved Hide resolved
@brandonroberts brandonroberts merged commit 82c0a3a into open-sauced:beta Jul 24, 2023
5 checks passed
github-actions bot pushed a commit that referenced this pull request Jul 24, 2023
## [2.4.0-beta.1](v2.3.0...v2.4.0-beta.1) (2023-07-24)

### πŸ• Features

* add emoji support ([#63](#63)) ([82c0a3a](82c0a3a))
@github-actions
Copy link

πŸŽ‰ This PR is included in version 2.4.0-beta.1 πŸŽ‰

The release is available on:

Your semantic-release bot πŸ“¦πŸš€

github-actions bot pushed a commit that referenced this pull request Jul 24, 2023
github-actions bot pushed a commit that referenced this pull request Jul 24, 2023
## [2.4.0](v2.3.0...v2.4.0) (2023-07-24)

### πŸ• Features

* add emoji support ([#63](#63)) ([82c0a3a](82c0a3a))
@github-actions
Copy link

πŸŽ‰ This PR is included in version 2.4.0 πŸŽ‰

The release is available on:

Your semantic-release bot πŸ“¦πŸš€

@a0m0rajab a0m0rajab deleted the feat-add-emoji branch July 24, 2023 14:56
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.

Feature: Add node emoji to highlight cards
3 participants