-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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): 「みつける」のなかにミュートしたユーザが現れてしまう問題を修正 #12559
fix (backend): 「みつける」のなかにミュートしたユーザが現れてしまう問題を修正 #12559
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## develop #12559 +/- ##
===========================================
+ Coverage 78.42% 78.77% +0.35%
===========================================
Files 180 956 +776
Lines 23857 104015 +80158
Branches 495 8351 +7856
===========================================
+ Hits 18709 81943 +63234
- Misses 5148 22072 +16924 ☔ View full report in Codecov by Sentry. |
このPRによるapi.jsonの差分 差分はこちら |
] = me ? await Promise.all([ | ||
this.cacheService.userMutingsCache.fetch(me.id), | ||
this.cacheService.userBlockedCache.fetch(me.id), | ||
]) : [new Set<string>(), new Set<string>(), new Set<string>()]; |
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.
数があってないわね
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.
2つにしました
凡ミス申し訳ないです🙇♂️
🙏🙏 |
* fix (backend): 「みつける」のなかにミュートしたユーザが現れてしまう問題を修正 * fix
What
#12383 の対処です。
この対応により、自分がミュートしたユーザがハイライトの中に表示されることがなくなります。
また、同様にハイライトに載っているユーザにブロックされている場合、同じくハイライトの中に表示されることがなくなります。
Why
fix #12383
Additional info (optional)
以下をローカルで確認しました
Checklist