misskey-dev / misskey

🌎 An interplanetary microblogging platform 🚀
https://misskey-hub.net/
GNU Affero General Public License v3.0
9.84k stars 1.33k forks source link

ノート検索画面に作成日時の絞り込みを追加する #14018

Open KisaragiEffective opened 3 months ago

KisaragiEffective commented 3 months ago

Summary

ノート検索画面に作成日時の絞り込みを追加する

Purpose

エンドポイントにはあるのに画面上で露出されてないのは嬉しくないから

https://misskey.io/notes/9ukgzvdvvr2b0b5x

Do you want to implement this feature yourself?

KisaragiEffective commented 3 months ago

インスタンスごとにID発番形式が異なるので、sinceIduntilIdをどうやってクライアントから与えるかが問題

KisaragiEffective commented 3 months ago

直接検索UIにIDの入力欄を露出してもいいが、混乱を招くのでやりたくない

anatawa12 commented 3 months ago

絞り込みを実装するなら普通に他のエンドポイントみたいにsinceDate/untilDateをはやしていい気がする。 現在の実装では内部的には結局どちらもssinceId/untilIdに変換されてる

https://github.com/misskey-dev/misskey/blob/a88579ca98a70115d6a61c74f26c36215f1f3daa/packages/backend/src/server/api/endpoints/roles/notes.ts#L49-L50

samunohito commented 3 months ago

①APから受信した情報をもとにノートが作成されるとき、IDの採番はサーバごとの設定による(受信したものが持つIDはそのまま使用されない) https://github.com/misskey-dev/misskey/blob/develop/packages/backend/src/core/NoteCreateService.ts#L405

②meilisearchも①のIDを使ってインデックスされている https://github.com/misskey-dev/misskey/blob/develop/packages/backend/src/core/SearchService.ts#L115

…ので、

インスタンスごとにID発番形式が異なる

については気にしなくてもよいのかもと思いました。 (なんか変なこと言ってたらスルーしてください)

anatawa12 commented 3 months ago

採番方法の話は日時 => id変換がフロントエンド側でやる場合に問題があるよねという話だと思います。(nodeinfoとかで表に出てるかどうかがわからない)

samunohito commented 3 months ago

ああ、なるほど・・・

KisaragiEffective commented 3 months ago

sinceDate/untilDateをはやしていい気がする。

あるのか…困惑