misskey icon indicating copy to clipboard operation
misskey copied to clipboard

improve, fix: more reactive custom emoji update

Open tamaina opened this issue 2 years ago • 4 comments

Fix #9618

What

  • api/emojisをgetで取得できる/するようにし、キャッシュを1分に緩和。setIntervalで10分おきに読みに行くように
  • customEmojisをshallowRefに

Why

絵文字の更新頻度を上げたい

Additional info (optional)

tamaina avatar Jan 16 '23 09:01 tamaina

Codecov Report

Merging #9619 (26fbb3a) into develop (80a400a) will decrease coverage by 0.02%. The diff coverage is 5.45%.

@@             Coverage Diff             @@
##           develop    #9619      +/-   ##
===========================================
- Coverage    22.71%   22.69%   -0.02%     
===========================================
  Files          734      734              
  Lines        68745    68821      +76     
  Branches      2028     2030       +2     
===========================================
+ Hits         15616    15622       +6     
- Misses       53129    53199      +70     
Impacted Files Coverage Δ
...es/backend/src/core/entities/EmojiEntityService.ts 53.06% <0.00%> (-2.26%) :arrow_down:
...rver/api/endpoints/admin/emoji/add-aliases-bulk.ts 0.00% <0.00%> (ø)
...ackend/src/server/api/endpoints/admin/emoji/add.ts 0.00% <0.00%> (ø)
...rc/server/api/endpoints/admin/emoji/delete-bulk.ts 0.00% <0.00%> (ø)
...end/src/server/api/endpoints/admin/emoji/delete.ts 0.00% <0.00%> (ø)
...rc/server/api/endpoints/admin/emoji/list-remote.ts 0.00% <0.00%> (ø)
...ckend/src/server/api/endpoints/admin/emoji/list.ts 0.00% <0.00%> (ø)
...r/api/endpoints/admin/emoji/remove-aliases-bulk.ts 0.00% <0.00%> (ø)
...rver/api/endpoints/admin/emoji/set-aliases-bulk.ts 0.00% <0.00%> (ø)
...ver/api/endpoints/admin/emoji/set-category-bulk.ts 0.00% <0.00%> (ø)
... and 5 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

codecov[bot] avatar Jan 16 '23 09:01 codecov[bot]

絵文字追加イベントsubscrubeするのはどうじゃろ

syuilo avatar Jan 17 '23 00:01 syuilo

  • api/emojisをgetで取得できる/するように
  • customEmojisをshallowRefに
  • 絵文字追加イベント/更新イベント(これは未実装だったかも)をsubscribeしてcustomEmojisを更新するように

で良いんじゃないかな

syuilo avatar Jan 17 '23 04:01 syuilo

broadcaststreamにemojiaddedあるの知らなかった

tamaina avatar Jan 17 '23 07:01 tamaina

broadcaststream復活させるの面倒で放置してた

tamaina avatar Jan 22 '23 12:01 tamaina

面倒というかどうすんの(ストリーム系のコード、Redisへ入って出るんだけどその部分が分断されていてコードを追いにくい)

tamaina avatar Jan 22 '23 12:01 tamaina

普通にemojiAdded飛んできてるわ

tamaina avatar Jan 22 '23 14:01 tamaina

categoryがnullか'null'があってどうなんだという感じがある

tamaina avatar Jan 22 '23 17:01 tamaina

とりあえず'null' → nullと変換するようにしてnullが必ず一番最後になるようにした?

tamaina avatar Jan 22 '23 17:01 tamaina

(絵文字のカテゴリって並べ替えしたほうがいいんじゃないだろうかと思ったんだが)

tamaina avatar Jan 22 '23 17:01 tamaina

これ私がミスってマージした?

tamaina avatar Jan 26 '23 06:01 tamaina

全misskey 13.2.3で動作中

syuilo avatar Jan 26 '23 06:01 syuilo