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

Experimental/sf #8

Open
wants to merge 4,796 commits into
base: master
Choose a base branch
from
Open

Experimental/sf #8

wants to merge 4,796 commits into from

Conversation

ryo-endo
Copy link
Owner

@ryo-endo ryo-endo commented Jun 5, 2018

概要(Overview・Refs Issue)

  • PullRequestの目的、関連するIssue番号など

方針(Policy)

  • このPullRequestを作るにあたって考慮したものや除外した内容
     - 例)Symfony のXXにならって作成、2系で同様の仕様があったため など

実装に関する補足(Appendix)

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

テスト(Test)

  • テストを行っている範囲など、レビューアが安心できるような情報

相談(Discussion)

  • 相談したいことや意見を求めたいこと

マイナーバージョン互換性保持のための制限事項チェックリスト

  • マイナーバージョンでは、機能・プラグイン・デザインテンプレート互換性を損なう変更は原則取り込みません。
  • 既存機能の仕様変更
  • フックポイントの呼び出しタイミングの変更
  • フックポイントのパラメータの削除・データ型の変更
  • twigファイルに渡しているパラメータの削除・データ型の変更
  • Serviceクラスの公開関数の、引数の削除・データ型の変更
  • 入出力ファイル(CSVなど)のフォーマット変更

okazy and others added 30 commits July 24, 2018 17:15
…in-multi-shipping

# Conflicts:
#	src/Eccube/Controller/Admin/Order/OrderController.php
…ipping

お届け先追加のテスト追加
…in-multi-shipping

# Conflicts:
#	src/Eccube/Resource/template/admin/Order/index.twig
MySQLだとDoctrineのDateTimeTzが使えなくて+9h表示されちゃうので
sai-kita and others added 29 commits August 3, 2018 17:27
Proxy クラスを利用すると、 Windows 環境で動作しない不具合の対応
EC-CUBE#50 【権限設定】拒否URLに部分一致のページはメニューに表示されるがアクセスできない
【管理画面サイドナビ】1,3階層目を拡張できるように修正、それに伴ってeccube_navの構造見直し
…de-items-processing

【管理画面/会員編集】購入履歴に購入処理中のものまで表示される
EC-CUBE#265 【カテゴリ管理】画面右のツリーで選択中の項目をわかるように表示する
EC-CUBE#69 【ページ管理】新規作成時点で初期値として最低限のテンプレートを用意しておきたい
EC-CUBE#51 【メンバー管理】フォームのエラーメッセージがわかりにくい
EC-CUBE#107 【商品管理/商品登録】画像の順番が不明
 EC-CUBE#257 商品規格の並び方が管理画面/フロントで異なる
【出荷登録画面】お届け先自体の削除機能追加
サンプルとして入っていたスマホ用のtwigファイルを削除
ryo-endo pushed a commit that referenced this pull request Sep 14, 2020
テストを追加して最新ブランチに追従
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.