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

ユーザのノートのみ表示時にDBへフォールバックするとリノートを含んでしまうのを修正 #12321

Merged
merged 2 commits into from
Nov 13, 2023

Conversation

samunohito
Copy link
Member

What

#12306 の対応です。
この対応により、ユーザのプロフィール画面→ノートタブ→ノートから参照できるTLにリノートが混ざらなくなります。
(Redisにキャッシュがあるときは発生せず、無いときだけ発生する)

※ひとまずusers/notes.tsのみ対応しましたが、以下のファイルにも同様のパターンが見られました。

  • notes/hybrid-timeline.ts
  • notes/timeline.ts
  • notes/user-list-timeline.ts

users/notes.tsにもあったincludeMyRenotesのほかに、includeRenotedMyNotes、includeLocalRenotesもありましたが、いずれもフロントエンド側から設定されるような実装は無く…

Why

  • 画面の表示通りの動きになっておらず、混乱を招いてしまう
  • キャッシュの有無により挙動が異なり、混乱を招いてしまう

Additional info (optional)

ローカルにてRedisを全削除→該当TLを表示でリノートが混ざり込まなくなることを確認
(対応前は混ざり込むことも見ているので手順はOKなはず)

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

@github-actions github-actions bot added the packages/backend Server side specific issue/PR label Nov 13, 2023
@syuilo syuilo merged commit e1cc95d into misskey-dev:develop Nov 13, 2023
17 of 18 checks passed
@syuilo
Copy link
Member

syuilo commented Nov 13, 2023

👍🏻

Copy link
Contributor

このPRによるapi.jsonの差分

差分はこちら
--- base
+++ head
@@ -56275,10 +56275,6 @@
                   "untilDate": {
                     "type": "integer"
                   },
-                  "includeMyRenotes": {
-                    "type": "boolean",
-                    "default": true
-                  },
                   "withFiles": {
                     "type": "boolean",
                     "default": false

Get diff files from Workflow Page

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/backend Server side specific issue/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants