フォロー中のユーザが一定期間非アクティブである場合サジェストされなくなる

[samunohito]

💡 Summary

今現在、users/search-by-username-and-hostの動作は下記のようになっています。

①フォロー中のユーザのうちアクティブ¹なユーザを抽出
②1の結果がlimitに満たない場合、自分がフォローしていないかつupdatedAt != nullなユーザを抽出し、1と合体

上記の条件ですと「フォローしているが非アクティブなユーザ」が表示されなくなり、該当ユーザをメンションしたい時に若干不便な状態になってしまっています。

そこで、以下の優先順位でユーザを抽出するようにし、上記の課題を解決したいです。

①フォロー中のユーザのうちアクティブなユーザ
②フォロー中のユーザのうち非アクティブなユーザ
③未フォローのユーザのうちアクティブなユーザ
④未フォローのユーザのうちアクティブなユーザ

また、抽出されたユーザのソートも行われていますが、user.usernameLowerの昇順とuser.updatedAtの降順の組み合わせでまちまちなので、これらも一律user.usernameLowerしたいです。

🥰 Expected Behavior

フォローしている非アクティブユーザもサジェストに出る

🤬 Actual Behavior

出ない

📝 Steps to Reproduce

フォローしている非アクティブユーザめがけてメンションしようとする

💻 Frontend Environment

-

🛰 Backend Environment (for server admin)

-

Do you want to address this bug yourself?

• Yes, I will patch the bug myself and send a pull request

Footnotes

1. 過去30日の間でuser.updatedAtが更新されたユーザ ↩

[samunohito]

（あと、自分自身のIDを省くようにクエリが組まれているけれども…特殊な意図が無いようであれば、これも解除して自分自身のIDをサジェスト出来るようにもしたい）

[syuilo]

クライアント側でケアすれば良いんじゃないかしら

[samunohito]

というと…？

[syuilo]

自分の情報はクライアントが持っているからAPIからレスポンスする必要はないわね

[samunohito]

ああ、 #14149 (comment) の件…なるほど。

[samunohito]

ただ、「今入力しているキーワードが自分のアカウントと前方一致しているか」というのをクライアント側で常に判定しなければならなくなり、かつ検索結果のキャッシュにも入れておく必要があるので、もし自身を含めていいのであればサーバサイドのWHEREから消した方がシンプルに済みそうな気もしており

[syuilo]

ふぅ〜む

[syuilo]

ただ、「今入力しているキーワードが自分のアカウントと前方一致しているか」というのをクライアント側で常に判定しなければならなくなり

常に表示しておくのはダメかしら

[syuilo]

あーサジェストか