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

ハイライト(みつける)のアンケート欄にて、(センシティブ)チャンネルのものが含まれてしまう #13833

Closed
1 task
Sayamame-beans opened this issue May 19, 2024 · 2 comments · Fixed by #13838
Labels
⚠️bug? This might be a bug

Comments

@Sayamame-beans
Copy link
Member

💡 Summary

タイトル通りです。
ハイライト(みつける)のノート欄ではセンシティブチャンネルの投稿が含まれなかったと思うので、アンケートの方もそうである方が良いと思いました。
(センシティブでなくとも、そもそもチャンネルである時点でハイライト(みつける)に載らなかったような気もします…?)

🥰 Expected Behavior

(センシティブ)チャンネルの投稿はハイライト(みつける)のアンケート欄に表示されない。

🤬 Actual Behavior

(センシティブ)チャンネルの投稿がハイライト(みつける)のアンケート欄に表示される。

📝 Steps to Reproduce

No response

💻 Frontend Environment

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

🛰 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 Sayamame-beans added the ⚠️bug? This might be a bug label May 19, 2024
@Sayamame-beans
Copy link
Member Author

Sayamame-beans commented May 19, 2024

みつける(ノート): api/notes/featured
みつける(アンケート): api/notes/polls/recommendation

https://github.com/misskey-dev/misskey/blob/ba62b7378bb13b384bd9db27acb0013fb90b53b3/packages/backend/src/server/api/endpoints/notes/polls/recommendation.ts

@Sayamame-beans
Copy link
Member Author

api/notes/polls/recommendation にはチャンネル投稿を除外する処理が無さそうなので、除外するようにすれば解決しそうです。(互換性の観点では、チャンネルのものでも除外しない用のオプションをエンドポイント側に作る必要があるかもしれませんが)

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
None yet
Development

Successfully merging a pull request may close this issue.

1 participant