おさむのひと comments

Results 212 comments of


                                            おさむのひと

CHANGELOGが毎度conflictするのが煩わしい

もしこれが採択された場合、CHANGELOG.mdの記入箇所チェックが役割を終えるので消しておきたい - https://github.com/misskey-dev/misskey/blob/develop/.github/workflows/changelog-check.yml - https://github.com/misskey-dev/misskey/tree/develop/scripts/changelog-checker ああいや結局記入位置を間違える問題はついて回りそうかな…蒸気は最終的な形次第で

フォロー中チャンネル一覧取得API（channels/followed）のuntilIdを設定すると想定通りの動作をしない

以下の実装が噛み合っていないものと推測しています。 - [channels/followed.ts](https://github.com/samunohito/misskey/blob/6cf466e5d1d423aec8ec06f65644c73c7e9a8ecd/packages/backend/src/server/api/endpoints/channels/followed.ts#L33)に渡されるsinceId/untilIdの種別 →ここで渡ってくるのはchannel.id - [channel_following向けのクエリビルダ](https://github.com/samunohito/misskey/blob/6cf466e5d1d423aec8ec06f65644c73c7e9a8ecd/packages/backend/src/server/api/endpoints/channels/followed.ts#L51) - [QueryService.makePaginationQuery](https://github.com/samunohito/misskey/blob/e6c54de814c4d43785cc3c26378479bd053d9b63/packages/backend/src/core/QueryService.ts#L43)の実装 →sinceId/untilIdで絞り込む対象のカラムはid固定上記から、channel_following.idに対しsinceId/untilIdを使って絞り込みを測るも、channel.idとchannel_following.idは別物であるため意図せぬ動作をしているものと考えました。なので、絞り込み対象のカラムをchannel.idと同種であるchannel_following.followeeIdにすれば期待する動作となるかと思います。

フォロー中チャンネル一覧取得API（channels/followed）のuntilIdを設定すると想定通りの動作をしない

以下のようにすればいけるかなと思ったのですが、如何でしょうか。方針的によさそうであれば、プルリクを作ろうかと思います。 ```ts // 絞り込み対象とするカラムを外から設定できるようにする // （他箇所に影響がないように元々固定で設定されていたidカラムをデフォルト値とする） public makePaginationQuery( q: SelectQueryBuilder, sinceId?: string | null, untilId?: string | null, sinceDate?: number | null, untilDate?: number | null, columnName = 'id',...

フォロー中チャンネル一覧取得API（channels/followed）のuntilIdを設定すると想定通りの動作をしない

> ChannelFollowing を返さないとダメそうとなると…channelEntityService.pack()の流用はできなくなるので、channels/followed専用のものを新たに作成してそれを使うようにするイメージでしょうか（idはchannel_following.idを返すようにして、別にchannelIdみたいなパラメータを増やして返す）上記方針だと[フロント側](https://github.com/samunohito/misskey/blob/b0f6c44f36f3173f5d232d1adb02fc09dfdcc003/packages/frontend/src/components/MkChannelPreview.vue#L7)やサードパーティアプリの該当機能も修正する必要が出てきますね…