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

住所や会員情報を国際標準へ対応 #2598 #3234

Merged

Conversation

hoand-vn
Copy link
Contributor

概要(Overview・Refs Issue)

  • PullRequestの目的、関連するIssue番号など
    受け入れ条件

電話番号(tel)のフィールドを1カラムにする

郵便番号(zip)のフィールドを1カラムにする

FAXは廃止する

郵便番号の自動入力が動作すること

unit test/codeception

やること

zip01-03

postal_codeに統合

db, entity, form type, twig

コントローラ等で使用している変数名

DBの列はstringで保持

バリデーション

-と数値

8文字まで入力可

-は入力後切り捨てる(KanaListenerを参考に)→DBにはハイフン無しの数値を保持

tel01-03

phone_numberに統合

db, entity, form type, twig

コントローラ等で使用している変数名

** バリデーション

-と数値

14文字まで入力可

-は、入力後切り捨てる

fax

廃止

対象テーブルdtb_customerdtb_baseinfodtb_shippingdtb_order

参考#2598

方針(Policy)

  • このPullRequestを作るにあたって考慮したものや除外した内容
     - none

実装に関する補足(Appendix)

  • コードだけではわかりづらい点など、実装するにあたってレビューアに追加で伝えておきたいこと

テスト(Test)

  • phpUnit
  • codeCeption

相談(Discussion)

  • min length

@okazy okazy added this to the 3.n.0 milestone Jun 27, 2018
@chihiro-adachi chihiro-adachi merged commit 8c1b1af into EC-CUBE:experimental/sf Jun 28, 2018
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.

None yet

6 participants