fzngagan
(Faizaan Gagan)
1
クライアントの一人が、ステージング済みのユーザーを含むドロップダウンからユーザーを選択できる機能が必要としています。
コードベースを確認したところ、このコンポーネントは userSearch ライブラリメソッドを使用してサーバーへの API 呼び出しを行っています。現状では、そこに追加パラメータを渡す方法がありません。
さらに、Rails のコントローラアクション search_users にも、追加パラメータを受け付ける仕組みがありません。
そのため、includeStagedUsers というパラメータ(デフォルトは false)を user-chooser コンポーネントに渡せるようにする PR を作成しようと考えています。これにより、ドロップダウンにステージング済みのユーザーを表示できるようになります。ご意見をお聞かせください @j.jaffeux。
j.jaffeux
(Joffrey Jaffeux)
2
これについては特に意見はありません。コアに機能を追加するものですが、あまり活用されないでしょう。@zogstrip の考えは?
fzngagan
(Faizaan Gagan)
3
皆さん、何かニュースはありますか?
いくつかの追加の考え:
ここで重要なのは、このコードベースの一部が拡張不可能だということです。拡張可能にするための仕様を提案していただければ、それも役立ちます。その方針で PR を作成することも喜んで行います。
fzngagan
(Faizaan Gagan)
4
簡単なプルリクエストを作成しました。レビューとご提案をお願いいたします。