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

ユーザー登録画面でアンチハラスメントポリシーと利用規約のチェック状態をエラー時に残す #7440

Conversation

naokinaokiboo
Copy link
Contributor

@naokinaokiboo naokinaokiboo commented Feb 26, 2024

Issue

概要

  • 名前が未入力などの入力エラー時に、以下のチェックボックスのチェック状態を維持するようにしました
    • アンチハラスメントポリシーに同意
    • 利用規約に同意

変更確認方法

  1. feature/maintain-states-of-code-of-conduct-and-terms-of-service-in-user-registrationをローカルに取り込む
  2. http://localhost:3000/users/newにアクセスする
  3. カード番号を入力する
    • 補足
      • カード番号に関しては他の必須項目とは別のバリデーションの仕組みになっており、この項目が未入力だとカード番号のバリデーションが先に動き本Issueの目的であるチェックボックスが外れた状態に遷移しないためこの手順を入れています
      • ダミーの番号として、4242 4242 4242 4242が使用可能(有効期限は任意の将来の日付、セキュリティコードは任意の3桁の数字)
  4. 他の入力必須項目を未入力のまま、以下のチェックボックスにチェックを入れ、参加するボタンをクリックする
    • アンチハラスメントポリシーに同意
    • 利用規約に同意
  5. 以下の項目を確認する
    • アンチハラスメントポリシーに同意のチェックボックスがチェック状態のままであること
    • 利用規約に同意のチェックボックスがチェック状態のままであること
    • 参加するボタンが有効(クリックできる状態)であること

Screenshot

変更前

Image from Gyazo

変更後

Image from Gyazo

@naokinaokiboo naokinaokiboo force-pushed the feature/maintain-states-of-code-of-conduct-and-terms-of-service-in-user-registration branch from a3d4fb8 to 9e84b47 Compare February 26, 2024 12:53
@naokinaokiboo naokinaokiboo self-assigned this Feb 27, 2024
@naokinaokiboo naokinaokiboo changed the title アンチハラスメントポリシーと利用規約のcheckboxの値をparamsから取得するように変更 ユーザー登録画面でアンチハラスメントポリシーと利用規約のチェック状態をエラー時に残す Feb 27, 2024
@naokinaokiboo naokinaokiboo force-pushed the feature/maintain-states-of-code-of-conduct-and-terms-of-service-in-user-registration branch 2 times, most recently from ed8bc81 to 0437510 Compare February 28, 2024 09:03
@naokinaokiboo naokinaokiboo force-pushed the feature/maintain-states-of-code-of-conduct-and-terms-of-service-in-user-registration branch from 0437510 to 18f39b3 Compare February 29, 2024 16:13
@naokinaokiboo naokinaokiboo marked this pull request as ready for review March 1, 2024 06:03
@naokinaokiboo
Copy link
Contributor Author

@a-terumoto-gs
お疲れ様です。
こちら、ご都合の良いタイミングでレビューをお願いできますでしょうか🙏(全く急ぎではありません)
もしご都合悪いなどありましたら、その旨、仰っていただけたらと思います。
よろしくお願い致します。🙇‍♂️

@a-terumoto-gs
Copy link
Contributor

@naokinaokiboo
お疲れ様です。レビュー依頼ありがとうございます!

来週中には確認できるようにしたいと思います。
少々お待ちいただくことになるかもしれませんが、よろしくお願いいたしますm(__)m

Copy link
Contributor

@a-terumoto-gs a-terumoto-gs left a comment

Choose a reason for hiding this comment

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

おつかれさまです!
チェックさせていただきました!
変更方法が細かく丁寧に書いてあって確認がしやすかったです(*'▽')
わたしからはApproveとさせていただきます。

@naokinaokiboo
Copy link
Contributor Author

@a-terumoto-gs
お忙しい中、レビューありがとうございました😊🙏

@naokinaokiboo
Copy link
Contributor Author

@komagata
チームメンバーにApproveをいただいたので、レビューをお願いいたします🙏

@naokinaokiboo naokinaokiboo requested a review from komagata March 6, 2024 07:45
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 c280eaa into main Mar 9, 2024
5 checks passed
@komagata komagata deleted the feature/maintain-states-of-code-of-conduct-and-terms-of-service-in-user-registration branch March 9, 2024 17:03
@github-actions github-actions bot mentioned this pull request Mar 9, 2024
29 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.

3 participants