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

#139 【お問い合せ】会員登録とお問い合せのエラー文言の出し方が異なる #3448

Closed
wants to merge 5 commits into from

Conversation

sai-kita
Copy link
Contributor

@sai-kita sai-kita commented Aug 2, 2018

以下を参考にコメントを作成してください。

概要(Overview・Refs Issue)

会員登録のエラー文言の出し方を、お問い合せのエラー文言の出し方に寄せる。

方針(Policy)

会員登録画面
・フォーム要素のnovalidate属性を削除し、HTML5のバリデーションの表示を行う
・HTML5のバリデーション対応時に、お問い合わせ画面とエラー文言(チェック項目)
 が異なる場合は、お問い合わせのエラー文言表記に寄せる
 対象:メールアドレス形式のチェック

会員情報編集
・同じUIである為、会員登録画面と同様の対応を追加実施しております。

カスタムフォームフィールドの修正
 修正対象:RepeatedEmailType
 HTML5のメールアドレスバリデーションを実装するには、
 input要素内のtype属性に'email'を出力する必要があります。 
 オプションの指定は'entry_type'ではなく、'type'が正しい為、修正しております。

実装に関する補足(Appendix)

カスタムフォームフィールドの修正の影響範囲
 3箇所でRepeatedEmailType のフォームが使用されております。
  1. 会員登録画面
  2. 会員情報編集
  3. 購入フロー内、ゲスト購入のお客様情報の入力
 →3について、現在HTML5のバリデーション表示が実装されておりますが、
  メールアドレスかどうかのバリデーションチェックは未実装の状態でした。
  今回のカスタムフォームフィールドの修正で、
  新たにメールアドレスのチェックが追加された為、動作テストを実施しております。

テスト(Test)

テスト実施ブラウザ
 IE11
 Chrome 67
 Firefox 61

対象画面
 1. 会員登録画面
 2. 会員情報編集
 3. 購入フロー内、ゲスト購入のお客様情報の入力

テスト内容と期待結果
以下の場合、お問い合わせ画面同様に、HTML5のバリデーションエラーの表示を行う
 1. 入力項目に不備がある場合(必須チェック)
 2. 1の内、メールアドレス項目については、入力値がメールアドレス形式ではない場合

相談(Discussion)

なし

@ryo-endo ryo-endo added this to the 3.n.0 milestone Aug 2, 2018
@ryo-endo
Copy link
Contributor

ryo-endo commented Aug 5, 2018

@sai-kita ありがとうございます。

こちらですが、EC-CUBE3.nの方針としては、HTML5でのチェックではなく、submit後にエラーを表示するという方針で行きます。

そのためお手数おかけしますが「お問い合せ」を「会員登録」に合わせるという方針でご対応修正お願いします。

@ryo-endo ryo-endo changed the base branch from experimental/sf to 4.0 August 6, 2018 11:12
@okazy
Copy link
Contributor

okazy commented Sep 12, 2018

別の対応でまとめて対応しました。

@okazy okazy closed this Sep 12, 2018
4.0 BugHunt automation moved this from Fix-me (修正が必要) to Done Sep 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
4.0 BugHunt
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants