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

ノートのホーム化をしてもVRTLからノートが非表示になっていない #186

Closed
Sayamame-beans opened this issue May 2, 2024 · 9 comments · Fixed by #189 or #191
Labels
⚠️Bug Something isn't working niri-la This issue or pull request is related to original features of misskey.niri.la

Comments

@Sayamame-beans
Copy link

💡 概要

PublicノートをHome化した際、LTLからは消えるようですが、その他のTLからは消えていないかもしれません。
(VRTLでは消えていないことを確認しました。GTLは密度的に消す処理なくても大丈夫そうですが既に消えてそう?でした。)

// remove from funout local timeline
const redisPipeline = this.redisForTimelines.pipeline();
this.fanoutTimelineService.remove('localTimeline', note.id, redisPipeline);
if (note.fileIds.length > 0) {
this.fanoutTimelineService.remove('localTimelineWithFiles', note.id, redisPipeline);
}
for (const renote of renotes) {
this.fanoutTimelineService.remove('localTimeline', renote.id, redisPipeline);
}
await redisPipeline.exec();
// remove from highlights
// since renotes are not included in featured, we don't need to remove them
await featuredService.removeNote(note);
});

🥰 想定される挙動

VRTLから消える

🤬 実際の挙動

VRTLに残る

📝 再現手順

  1. ぶいみみリレー内のリモートのノートをHome化する
  2. VRTLをリロードする
  3. 見える

💻 お使いの環境

* 機種とOS: any
* ブラウザ: any
@Sayamame-beans Sayamame-beans added ⚠️Bug Something isn't working niri-la This issue or pull request is related to original features of misskey.niri.la labels May 2, 2024
@Sayamame-beans
Copy link
Author

これってvmimiRelayTimelineとかに同じことすれば良いんですかね~…?

@Sayamame-beans
Copy link
Author

直ってないかも。
https://misskey.niri.la/notes/9tckfu8ur3

@Sayamame-beans
Copy link
Author

ローカルの投稿に対して適用したときが駄目かも。
リモートの投稿に対しては大丈夫かも。

@Sayamame-beans
Copy link
Author

両方処理入ってるように見えるのになんでだろう

@anatawa12
Copy link
Collaborator

#189 (comment) 完全に忘れてマージしたけどこれな気がする

@Sayamame-beans
Copy link
Author

なるほどね()

@Sayamame-beans
Copy link
Author

質問なんですが、LTL/VRTLやSTL/VSTLのLTL成分から、リアルタイムに当該ノートの削除走らせるって可能なんでしょうか?
STLは厳しい? (HTL成分と重複するから一緒に消えてしまう?)

@anatawa12
Copy link
Collaborator

一応がんばりゃできる

@Sayamame-beans
Copy link
Author

質問なんですが、LTL/VRTLやSTL/VSTLのLTL成分から、リアルタイムに当該ノートの削除走らせるって可能なんでしょうか?
STLは厳しい? (HTL成分と重複するから一緒に消えてしまう?)

一応がんばりゃできる

検討します

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚠️Bug Something isn't working niri-la This issue or pull request is related to original features of misskey.niri.la
Projects
None yet
2 participants