Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

日報作成、編集フォームのプラクティス選択をChoices.jsに置き換える。 #4626

Conversation

NorifumiOgawa
Copy link
Contributor

@NorifumiOgawa NorifumiOgawa commented Apr 15, 2022

概要

  • 日報作成、編集フォームのプラクティス選択をselect2からChoices.jsに置き換える。
  • 既存のcompany-select.jschoices-ui.js にリネームし、複数選択可能なプルダウン用にjs-choices-multiple-selectというidを当てて対応する方針です。※詳細はIssueの町田さんからのコメントにあります。
  • テストもChoices.jsに合わせて修正しました。

変更前

Image from Gyazo

変更後

Image from Gyazo

確認方法

  1. ブランチ feature/replace-practice-selection-on-daily-report-editing-forms-with-choices-js をローカルに取り込む
  2. bin/rails sでサーバーを立ち上げる
    3.komagata でログインし、日報作成の画面を開く
  3. 「プラクティス」の項目をクリックし、複数のプラクティスが選択できることを確認する
  4. 「プラクティス」で選択されているプラクティスの「✗」ボタンクリックで削除できることを確認する
  5. 「プラクティス」の項目に文字railsを入力し、railsに該当するプラクティスが候補として表示されることを確認する

@NorifumiOgawa NorifumiOgawa changed the title 新規質問作成フォームのプラクティス選択をChoice.jsに置き換える。 日報作成、編集フォームのプラクティス選択をChoice.jsに置き換える。 Apr 15, 2022
@NorifumiOgawa NorifumiOgawa changed the title 日報作成、編集フォームのプラクティス選択をChoice.jsに置き換える。 日報作成、編集フォームのプラクティス選択をChoices.jsに置き換える。 Apr 15, 2022
@NorifumiOgawa NorifumiOgawa marked this pull request as ready for review April 15, 2022 08:26
@NorifumiOgawa
Copy link
Contributor Author

NorifumiOgawa commented Apr 15, 2022

@garammasala29 ガラムマサラさんが先日、管理者向けユーザー編集画面の「企業」欄で導入されたChoices.jsを、日報作成・編集フォームの「プラクティス」欄に適用するIssueの対応となります。お手すきの際にレビューをお願いします🙏

@machida ひと通り実装してみました。この段階でデザインをお願いして大丈夫でしょうか?よろしくお願いします🙏

@machida
Copy link
Member

machida commented Apr 15, 2022

@NorifumiOgawa デザイン了解です!

Copy link
Contributor

@garammasala29 garammasala29 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@NorifumiOgawa さん
お疲れ様です。レビュー依頼ありがとうございました!
複数選択できることを確認できました、LGTMです👍

@machida
Copy link
Member

machida commented Apr 20, 2022

@NorifumiOgawa デザイン入れましたー

image

@machida machida removed their assignment Apr 20, 2022
@NorifumiOgawa
Copy link
Contributor Author

@garammasala29 さん、レビューありがとうございました!
@machida さん、デザインありがとうございました!
@komagata さん、レビューよろしくお願いします🙏

@NorifumiOgawa NorifumiOgawa requested a review from komagata April 20, 2022 05:30
Copy link
Member

@komagata komagata left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

確認させて頂きました。OKです〜🙆‍♂️

@komagata komagata merged commit d3e0d9d into main Apr 20, 2022
@komagata komagata deleted the feature/replace-practice-selection-on-daily-report-editing-forms-with-choices-js branch April 20, 2022 05:55
@github-actions github-actions bot mentioned this pull request Apr 20, 2022
20 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants