improve, fix: more reactive custom emoji update
Fix #9618
What
- api/emojisをgetで取得できる/するようにし、キャッシュを1分に緩和。setIntervalで10分おきに読みに行くように
- customEmojisをshallowRefに
Why
絵文字の更新頻度を上げたい
Additional info (optional)
Codecov Report
Merging #9619 (26fbb3a) into develop (80a400a) will decrease coverage by
0.02%. The diff coverage is5.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.
絵文字追加イベントsubscrubeするのはどうじゃろ
- api/emojisをgetで取得できる/するように
- customEmojisをshallowRefに
- 絵文字追加イベント/更新イベント(これは未実装だったかも)をsubscribeしてcustomEmojisを更新するように
で良いんじゃないかな
broadcaststreamにemojiaddedあるの知らなかった
broadcaststream復活させるの面倒で放置してた
面倒というかどうすんの(ストリーム系のコード、Redisへ入って出るんだけどその部分が分断されていてコードを追いにくい)
普通にemojiAdded飛んできてるわ
categoryがnullか'null'があってどうなんだという感じがある
とりあえず'null' → nullと変換するようにしてnullが必ず一番最後になるようにした?
(絵文字のカテゴリって並べ替えしたほうがいいんじゃないだろうかと思ったんだが)
これ私がミスってマージした?
全misskey 13.2.3で動作中