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

GTLで過去の投稿を閲覧するとき「リノートを表示」オプションが機能していない #12233

Closed
ozelotdev opened this issue Nov 3, 2023 · 0 comments · Fixed by #12234
Labels
⚠️bug? This might be a bug

Comments

@ozelotdev
Copy link
Contributor

💡 Summary

GTLで過去の投稿を閲覧するとき、「リノートを表示」がオフでも、他人のリノートが表示されます。
コードを確認しましたが GTL の Endpoint で処理されていないようです。

Endpoint Stream
HTL
LTL
STL
GTL

memo:

  • 772d243 GTLに「リノートを表示」オプションを追加
  • Release 2023.9.2
  • 6277a55 GTLを削除
  • 0fe8c01 GTLを復活(が、リノート表示の処理が抜けてしまっている)
  • Release 2023.10.0

🥰 Expected Behavior

GTLで過去の投稿を閲覧するとき、「リノートを表示」がオフなら、他人のリノートが表示されない。

🤬 Actual Behavior

GTLで過去の投稿を閲覧するとき、「リノートを表示」がオフでも、他人のリノートが表示される。

📝 Steps to Reproduce

  1. GTLを開く
  2. 「リノートを表示」をオフにする
  3. 過去の投稿を閲覧する(下にスクロール)

💻 Frontend Environment

* Model and OS of the device(s): Windows
* Browser: Firefox
* Server URL: localhost:3000
* Misskey: 2023.11.0-beta.8

🛰 Backend Environment (for server admin)

* Installation Method or Hosting Service: development environment (devcontainer)
* Misskey: 2023.11.0-beta.8
* Node: 20
* PostgreSQL: 15
* Redis: 7
* OS and Architecture: Debian
@ozelotdev ozelotdev added the ⚠️bug? This might be a bug label Nov 3, 2023
syuilo added a commit that referenced this issue Nov 4, 2023
* fix(backend): GTLの「リノートを表示」オプションが機能しないのを修正 (#12233)

* docs: Update changelog

* Apply suggestions from code review

Co-authored-by: syuilo <Syuilotan@yahoo.co.jp>

---------

Co-authored-by: syuilo <Syuilotan@yahoo.co.jp>
kanarikanaru pushed a commit to kanarikanaru/misskey that referenced this issue Nov 9, 2023
* fix(backend): GTLの「リノートを表示」オプションが機能しないのを修正 (misskey-dev#12233)

* docs: Update changelog

* Apply suggestions from code review

Co-authored-by: syuilo <Syuilotan@yahoo.co.jp>

---------

Co-authored-by: syuilo <Syuilotan@yahoo.co.jp>
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
1 participant