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

fix(frontend): Unicode絵文字とカスタム絵文字の名前が重複したときにカスタム絵文字がオートコンプリートにサジェストされない問題を修正 #13817

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

takaion
Copy link

@takaion takaion commented May 10, 2024

What

  • Fix Unicode絵文字のとカスタム絵文字の名前が重複すると、カスタム絵文字側がサジェストされなくなる #13815
  • マッチした絵文字のセットのキーとして絵文字のnamealiasOf (vsnew) ではなくemoji (🆚🆕:vs::new:のようにそれぞれの絵文字を示す文字列) を使用するようにしました。
    • Unicode絵文字と同じ名前を持つカスタム絵文字が存在すると重複した絵文字として判定されてしまっていました。
    • Unicode絵文字がカスタム絵文字の後に判定されるため Set の要素を上書きしてしまい、カスタム絵文字がサジェストされない状態でした。

Why

  • Unicode絵文字とカスタム絵文字を区別してオートコンプリートにサジェストさせるため。

Additional info (optional)

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label May 10, 2024
Copy link

codecov bot commented May 10, 2024

Codecov Report

Attention: Patch coverage is 90.90909% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 77.93%. Comparing base (f6af6d9) to head (6719524).

Files Patch % Lines
packages/frontend/src/scripts/search-emoji.ts 90.90% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##           develop   #13817       +/-   ##
============================================
+ Coverage    65.16%   77.93%   +12.77%     
============================================
  Files          985      185      -800     
  Lines       112139    25537    -86602     
  Branches      5742      484     -5258     
============================================
- Hits         73070    19903    -53167     
+ Misses       37635     5627    -32008     
+ Partials      1434        7     -1427     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/frontend Client side specific issue/PR
Projects
None yet
1 participant