daichi_otani

Results 8 issues of daichi_otani

### 概要(Overview) ゲスト購入時にご注文手続き>お客様情報編集の際に、都道府県を編集しようとすると「更新に失敗しました。入力内容を確認してください。」というアラートが表示されます。 ### 再現手順(Procedure) 1. 以下の欄を編集します。 2.エラーになります。 ### 環境(Environment) EC-CUBE:4.2.3 PHP:7.4

bug
bug:Low
good-first-issue

### 概要(Overview) 今だと注文メールしか選択できないため条件を外してもらえればと思います。 https://github.com/EC-CUBE/ec-cube/blob/4.3/src/Eccube/Form/Type/Admin/OrderMailType.php#L56 https://github.com/EC-CUBE/ec-cube/pull/5836#issuecomment-1907624516 ### 期待する内容(Expect) or 要望(Requirement) 注文メール以外の項目が選択できるようになる。

improvement

### 概要(Overview) 現在、商品検索の入力フォームにはバリデーションが特にないようです。 https://github.com/EC-CUBE/ec-cube/blob/81b68e78b38e4e25d7d818eb65dedce3dc3b87b9/src/Eccube/Form/Type/Admin/SearchProductType.php#L75-L78 ### 期待する内容(Expect) or 要望(Requirement) フォームへのバリデーション追加 ・商品名・商品ID・商品コードで用いてるようなので、これらの制限をまとめて考える必要がある。

ご注文手続き画面から会員の住所を変更することはできません。 またE2Eを見たところ、一度マイページへ移動してから会員情報編集を行なっていました。 該当コードのCustomerAddressChangePageは、ヘッダーからマイページに遷移しておりました。 https://github.com/EC-CUBE/ec-cube/blob/81b68e78b38e4e25d7d818eb65dedce3dc3b87b9/codeception/acceptance/EF09ThrottlingCest.php#L698

## 概要(Overview・Refs Issue) https://github.com/EC-CUBE/ec-cube/pull/4912 ## 方針(Policy) 以下のフローを実装しています。 1. 管理画面 > 基本設定から、 GAタグフォームにタグを設定 2. フロントでタグが設定されているか確認 ## 実装に関する補足(Appendix) ## テスト(Test) ## 相談(Discussion) ## マイナーバージョン互換性保持のための制限事項チェックリスト - [x] 既存機能の仕様変更はありません - [x] フックポイントの呼び出しタイミングの変更はありません - [x]...

## 概要(Overview・Refs Issue) この辺りのページエラーを修正しました。 https://github.com/EC-CUBE/ec-cube/issues/6074 https://github.com/EC-CUBE/ec-cube/issues/6206 ## 方針(Policy) ## 実装に関する補足(Appendix) ## テスト(Test) http://127.0.0.1:8000/products/list?pageno=abc ## 相談(Discussion) ## マイナーバージョン互換性保持のための制限事項チェックリスト - [x] 既存機能の仕様変更はありません - [x] フックポイントの呼び出しタイミングの変更はありません - [x] フックポイントのパラメータの削除・データ型の変更はありません - [x] twigファイルに渡しているパラメータの削除・データ型の変更はありません -...

bug
bug:Low

## 概要(Overview・Refs Issue) https://github.com/EC-CUBE/ec-cube/issues/6266 ## 方針(Policy) ## 実装に関する補足(Appendix) ## テスト(Test) ## 相談(Discussion) ## マイナーバージョン互換性保持のための制限事項チェックリスト - [x] 既存機能の仕様変更はありません - [x] フックポイントの呼び出しタイミングの変更はありません - [x] フックポイントのパラメータの削除・データ型の変更はありません - [x] twigファイルに渡しているパラメータの削除・データ型の変更はありません - [x] Serviceクラスの公開関数の、引数の削除・データ型の変更はありません -...

improvement

### 概要(Overview) 2系にもあるようなAutoMerge機能が欲しいです。 ### 期待する内容(Expect) or 要望(Requirement) Approve2件とテストがクリアされていた場合に、自動的にマージされる。 背景 dependabotのテストが落ちてしまっているのを後から見てマージすることがあるため、自動でマージしてくれたら嬉しい。 また普通のPRでもより早くマージされる。

improvement