-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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(backend): fix creating reactions bugs #13901
fix(backend): fix creating reactions bugs #13901
Conversation
このPRによるapi.jsonの差分 差分はこちら--- base
+++ head
@@ -56450,6 +56450,15 @@
}
}
},
+ "CANNOT_REACT_TO_RENOTE": {
+ "value": {
+ "error": {
+ "message": "You cannot react to Renote.",
+ "code": "CANNOT_REACT_TO_RENOTE",
+ "id": "eaccdc08-ddef-43fe-908f-d108faad57f5"
+ }
+ }
+ },
"INVALID_PARAM": {
"value": {
"error": { |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #13901 +/- ##
============================================
- Coverage 77.74% 64.95% -12.79%
============================================
Files 183 989 +806
Lines 25373 112861 +87488
Branches 487 5816 +5329
============================================
+ Hits 19726 73313 +53587
- Misses 5640 38106 +32466
- Partials 7 1442 +1435 ☔ View full report in Codecov by Sentry. |
単にエラー吐いて止まるのとハートつけるのとどっちが良いんだろって悩んでいた |
自分も迷ったんですが、普通の適当な絵文字でない文字列でもフォールバックされるのでその仕様に揃える方が合理的と思いましたがどうでしょう |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
コード見た感じ良さそう
#13998 によると、異体字セレクタ単体を選択してリアクションが出来てしまうようです。 こちらについても |
に引っかからないようにしたい気がする というかtwemojiから持ってきてるからmfm的な話も合わせるとtwemoji側のバグな気もしてきた |
これをパッと見た感じでは意図的な気はしますね 追記: テストもありました |
じゃあ除外するべきなのかなるほど |
🙏🏻 |
What
以下のことを行いました。
\u2764
にフォールバックされるようにWhy
Closes #13804
Closes #13896
Additional info (optional)
Checklist