enhance(frontend): チャンネル投稿をユーザーページと前後ノートに表示する #238
Conversation
| @@ -47,7 +47,7 @@ const pagination = computed(() => tab.value === 'featured' ? { | |||
| userId: props.user.id, | |||
| withRenotes: tab.value === 'all', | |||
| withReplies: tab.value === 'all', | |||
| withChannelNotes: tab.value === 'all', | |||
| withChannelNotes: true, | |||
There was a problem hiding this comment.
tab.value == null (ノート)のときにもincludeするべきかは少し議論の余地がある気がします。
There was a problem hiding this comment.
私としては、"すべて"と"ファイル付き"でチャンネル投稿も含めば良いかなという気持ちです
There was a problem hiding this comment.
個人的には「ノート」にもチャンネル投稿を含めていいと思っていました。(ノートと全ての違いは返信を含むか否かだと考えたため)
There was a problem hiding this comment.
ノート欄でchannelが見えないのはupstreamの実装で明示的なものなので、見えるようにするのは私は反対です。(upstreamの議論を経由したいです)
There was a problem hiding this comment.
こっちもチャンネル投稿を常に含めるべきかは議論の余地があると思います。(場合によってはupstreamに投げることも考えて。)
例えば対象のノートがチャンネル投稿ならincludeChannelNotesを有効にするとかが考えられるかなと思います。
There was a problem hiding this comment.
私としては、一律で含むつもりでした。
が…現状他の人へのリプとかってここに入っていましたっけ? チャンネルかどうかは気にしないということで良いかも。
There was a problem hiding this comment.
確認してきました
ノートの前後表示は、ユーザーページの"ノート"に相当するようで、返信は入っていませんでした。(ついでに、チャンネル投稿は"ノート"には表示されませんでした)
う～む…
There was a problem hiding this comment.
"ノート"にチャンネル投稿を含むかどうか
ちょっと分かります
どちらが良いかというと難しいですね…(とりあえず後で本家にissueはしましょうかね)
What
Why
#237
Additional info (optional)
Checklist