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

ノート詳細画面から上矢印を押した時の挙動に違和感がある #14000

Closed
1 task
setaria430 opened this issue Jun 14, 2024 · 1 comment · Fixed by #14061
Closed
1 task
Assignees
Labels
🐛Bug Unexpected behavior packages/backend Server side specific issue/PR

Comments

@setaria430
Copy link
Contributor

💡 Summary

古いノートの詳細画面から上矢印を押すと、直後のノートが取得できない
(ノートの前後関係を把握したい時に、その目的が達成されない)

例えば以下のスクリーンショットは1年前のノートから上矢印を押した時だが、期待に反して1週間前のノートが直後のノートとして表示される
image

参考にしたノート
https://misskey.io/notes/9dmog35771
関係があるかもしれない過去のIssue
#13837

🥰 Expected Behavior

ノート詳細画面から上矢印を押すと、その直後のノートが表示される

🤬 Actual Behavior

ある程度最近のノートが、直後のノートとして表示される

📝 Steps to Reproduce

  1. たくさんノートしてそうな人を探す
  2. 昔のノートの詳細画面から上矢印を押す

💻 Frontend Environment

* Model and OS of the device(s): Windows 10 pro
* Browser: Chrome バージョン: 125.0.6422.176
* Server URL: misskey.io
* Misskey: 2024.5.0-io.1a

🛰 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
@setaria430 setaria430 added the ⚠️bug? This might be a bug label Jun 14, 2024
@zyoshoka
Copy link
Contributor

基準とするノート(A)が、FTT のキャッシュに入っている最古のノート(B)よりも古い場合、A と B の間に投稿されたノートが帰ってこなくなっていてこの問題が生じているように見えます。

shouldFallbackToDb = shouldFallbackToDb || (noteIds.length === 0);

具体的にはこのロジックを書き換えて、キャッシュから取ってきた noteIds にキャッシュ内で最古のノートが含まれていたら DB にフォールバックするようにすれば良さそうと思いました

Related to #12842

@zyoshoka zyoshoka added 🐛Bug Unexpected behavior packages/backend Server side specific issue/PR and removed ⚠️bug? This might be a bug labels Jun 15, 2024
@zyoshoka zyoshoka self-assigned this Jun 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛Bug Unexpected behavior packages/backend Server side specific issue/PR
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants