リストタイムラインでミュートが貫通してしまう問題に対処 #12534
Merged
リストタイムラインでミュートが貫通してしまう問題に対処 #12534
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
このPRによるapi.jsonの差分 差分はこちら |
samunohito
commented
Dec 1, 2023
| @@ -135,8 +138,7 @@ export default class extends Endpoint<typeof meta, typeof paramDef> { // eslint- | |||
| .leftJoinAndSelect('note.reply', 'reply') | |||
| .leftJoinAndSelect('note.renote', 'renote') | |||
| .leftJoinAndSelect('reply.user', 'replyUser') | |||
| .leftJoinAndSelect('renote.user', 'renoteUser') | |||
| .leftJoinAndSelect('note.channel', 'channel'); | |||
| .leftJoinAndSelect('renote.user', 'renoteUser'); | |||
There was a problem hiding this comment.
チャンネル投稿はそもそもuserListTimelineに配信されていないように見受けられるので省きました
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #12534 +/- ##
===========================================
+ Coverage 78.73% 78.77% +0.03%
===========================================
Files 952 952
Lines 103602 103619 +17
Branches 8328 8345 +17
===========================================
+ Hits 81573 81621 +48
+ Misses 22029 21998 -31 ☔ View full report in Codecov by Sentry. |
samunohito
commented
Dec 1, 2023
| @@ -69,6 +70,7 @@ export default class Connection { | |||
| this.userIdsWhoMeMuting = userIdsWhoMeMuting; | |||
| this.userIdsWhoBlockingMe = userIdsWhoBlockingMe; | |||
| this.userIdsWhoMeMutingRenotes = userIdsWhoMeMutingRenotes; | |||
| this.userMutedInstances = new Set(userProfile.mutedInstances); | |||
There was a problem hiding this comment.
userProfileのキャッシュは独立して存在していますが、ストリーミングの頻度で毎度Setを作るのも無駄な気がしたのでSetを作成してキャッシュしています
|
ストリーミング部分のテストも追加してもらえるといいかもしれません🙏 |
# Conflicts: # packages/backend/src/server/api/endpoints/notes/user-list-timeline.ts
|
👍👍👍 |
camilla-ett
pushed a commit
to kaseiski/misskey
that referenced
this pull request
Jan 2, 2024
* ユーザリストTL系の各種動作を修正・統一 * fix * fix CHANGELOG.md * テスト追加
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What
#10443 の対処です。
この対応により、リストタイムラインにてユーザミュート、サーバミュートが機能するようになり、
チャンネル投稿がストリーミングに流れなくなります。
もとのIssueでは
…とまとめていますが、チャンネル投稿は元々リロード時やFTT有効時の読み込み対象としない意図が明示されている実装がありましたため、チャンネル投稿は含めない形としています。
（なのでストリーミング側でチャンネル投稿が漏れるのを塞ぐ方針で対応）
Why
fix #10443
Additional info (optional)
ユーザミュートとチャンネル投稿についてはローカルで確認しました。
サーバミュートについては、連合に参加している環境が手元になく確認できていません。
Checklist