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

[MOB-2012] BezierEmoji 구현 #78

Open
wants to merge 4 commits into
base: feature/redesign_bezier
Choose a base branch
from

Conversation

solchan87
Copy link
Contributor

@solchan87 solchan87 commented Sep 19, 2024

어떤 PR 인가요?

BezierEmoji 추가하는 PR 입니다.

왜 필요한건가요? (Optional)

작업 내용

  • Bezier Emoji를 축가합니다.

스크린샷 혹은 동영상

Reference

Figma

  • 문서는 아직 업데이트 전입니다.

Issue-number

Checklist

  • PR 제목을 라벨과 함께 명령형으로 작성했습니다.
  • 코딩 컨벤션 에 맞춰서 작성했습니다
  • 리뷰 리퀘스트 전에 더 이상 스스로 리뷰할게 없을 정도까지 셀프 리뷰를 진행했습니다
  • 변경사항에 대한 테스트코드를 추가했습니다. 또는, 테스트코드가 필요없는 이유가 있습니다(현재는 옵셔널입니다)

@solchan87 solchan87 self-assigned this Sep 19, 2024
@solchan87 solchan87 added the PR:NotYet Something isn't working label Sep 19, 2024
@solchan87 solchan87 changed the title [MOB-2012] Emoji 구현 [MOB-2012] BezierEmoji 구현 Sep 19, 2024
/// - Parameters:
/// - name: ch-asset 기반으로 emoji 의 file name 을 사용합니다.
/// - emojipedia 를 통해 name을 검색 할 수 있으며, Shortcodes/github 기준으로 사용합니다.
/// - ex) https://emojipedia.org/😄#technical
Copy link
Contributor Author

Choose a reason for hiding this comment

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

emoji 전체를 열거형으로 리스트업 하는 것보다 스트링을 통해서 가져오도록 논의가 되었습니다.

@solchan87 solchan87 added PR:Reviewable Improvements or additions to documentation and removed PR:NotYet Something isn't working labels Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR:Reviewable Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant