Skip to content
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

リアクション一覧画面でミュートした/ブロックされたユーザーのノートが見えてしまう #14070

Closed
1 task done
Sayamame-beans opened this issue Jun 22, 2024 · 5 comments · Fixed by #14100
Labels
⚠️bug? This might be a bug

Comments

@Sayamame-beans
Copy link
Collaborator

Sayamame-beans commented Jun 22, 2024

💡 Summary

ユーザーページにあるリアクションしたノートの一覧画面において、ミュート対象ユーザーのノートが見えてしまうようです。
関連: #12396

🥰 Expected Behavior

見えない

🤬 Actual Behavior

見える

📝 Steps to Reproduce

  1. 誰かのノートにリアクションをする
  2. そのユーザーをミュートする
  3. 自身のリアクション一覧画面を閲覧する

  1. 誰かのリアクション一覧画面を閲覧する
  2. そこに見えるノートのユーザーをミュートする
  3. リアクション一覧画面を再読み込みする

💻 Frontend Environment

* Model and OS of the device(s): any
* Browser: any
* Server URL: https://misskey.niri.la/
* Misskey: 2024.5.0-kinel.3

🛰 Backend Environment (for server admin)

* Installation Method or Hosting Service:
* Misskey:
* Node:
* PostgreSQL:
* Redis:
* OS and Architecture:

Do you want to address this bug yourself?

  • Yes, I will patch the bug myself and send a pull request
@Sayamame-beans
Copy link
Collaborator Author

reactionsの応答の中には直接note情報が入っているため、実装出来そう

@Sayamame-beans
Copy link
Collaborator Author

ブロックも考慮されていなさそう

@Sayamame-beans
Copy link
Collaborator Author

#7734 の結論が不明なため実装中断

@Sayamame-beans
Copy link
Collaborator Author

Sayamame-beans commented Jun 26, 2024

メモ: MiNoteReaction.noteはnullになる可能性があるっぽい? (型の上ではそう)
isUserRelatedの引数noteの型がanyになってるので、そのまま渡してisUserRelatedの中でnullチェック入れると楽そう

@Sayamame-beans Sayamame-beans changed the title リアクション一覧画面でミュートしたユーザーのノートが見えてしまう リアクション一覧画面でミュートした/ブロックされたユーザーのノートが見えてしまう Jun 27, 2024
@Sayamame-beans
Copy link
Collaborator Author

とりあえず実装したのでPRにします

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚠️bug? This might be a bug
Projects
1 participant