ユーザー登録のemailにバリデーションを追加

[a-kuroki-gs]

Issue

• ユーザー登録のemailにバリデーションを追加する #7444

概要

ユーザー登録のemailにバリデーションが何も設定されていなかったので設定した。
こちらのコメントで議論(検討)し、URI::MailTo::EMAIL_REGEXPを用いることにした。

• 利用可能なemailの例
  • [email protected]
  • abcd.AB-CD_12/[email protected]
  • [email protected]
• 利用不可能なemailの例
  • abcd ABCD [email protected]
  • (abcdABCD1234)@fjord.jp
  • abcd@ABCD@[email protected]
  • [email protected]
  • abcdABCD1234@fjord_fjord.jp
  • その他全角文字が入るもの

変更確認方法

1. feature/add-validation-to-user-emailをローカルに取り込む
2. http://localhost:3000/users/new にアクセスする
3. カード番号を入力する
   • 補足
     • ref: ユーザー登録画面でアンチハラスメントポリシーと利用規約のチェック状態をエラー時に残す #7440
     • カード番号に関しては他の必須項目とは別のバリデーションの仕組みになっており、この項目が未入力だとカード番号のバリデーションが先に動いてしまうため、この手順を入れている
     • ダミーの番号として、4242 4242 4242 4242が使用可能（有効期限は任意の将来の日付、セキュリティコードは任意の3桁の数字）
4. 概要欄の例に記載しているEmailを入力する
5. 規約の同意にチェックを入れ、参加するを押す
6. 正しい値であればユーザー登録が行われ、不正な値であれば「Emailは不正な値です」と表示されることを確認する

Screenshot

不正な値を入力し、ユーザー作成を行った場合

変更前

変更後

[a-kuroki-gs]

@SuzukaHori
お疲れ様です！
こちらのPRのレビューをお願いしてもよろしいでしょうか？
お時間厳しければ遠慮なくおっしゃってください🙌

ご確認よろしくお願いいたします🙇‍♀️

[SuzukaHori]

@a-kuroki-gs
お疲れさまです！
現在別のレビューに取り組んでおり、そちらをお待たせしている状況のため、今回は別の方に依頼をしていただいてもよろしいでしょうか🙇‍♀️

申し訳ありません💦💦

[a-kuroki-gs]

@SuzukaHori
すみません、見逃しておりました～😭
承知です！
ご連絡ありがとうございます！

[a-kuroki-gs]

@dowdiness
お疲れ様です！
こちらのPRのレビューをお願いしてもよろしいでしょうか？
お時間厳しければ遠慮なくおっしゃってください🙌

ご確認よろしくお願いいたします🙇‍♀️

[a-kuroki-gs]

@dowdiness
すみません、こちらご都合いかがでしょうか？
レビューの時間を取るのが難しそうであれば、ご連絡いただければ幸いです🙏

連絡がつかなさそうであれば
申し訳ありませんが、他の方にレビュー依頼させていただこうと思いますm(_ _)m

[dowdiness]

@a-kuroki-gs
すいません、通知に気付いてませんでした🙇‍♂️
今週中にはレビューしようと思いますので、少々お待ちください。

[dowdiness]

@a-kuroki-gs

お待たせしました！
手元の環境で利用可能・利用不可能なemailの例を全て試してみました。
（例を提示されているのがとても分かりやすかったです🙆‍♂️）
確かにIssueで議論されている仕様通りにバリデーションが動いていました。
実装とテストの両方とも問題無く出来ていると思うのでAproveします👍

[a-kuroki-gs]

@dowdiness
ご確認いただきありがとうございます🙌

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

[komagata]

👍

[komagata]

確認させていただきました。OKです～👌