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

仮会員が無効の場合、会員登録完了後直ぐにブラウザが閉じられたり通信が途絶えたりすると、仮会員として作成されてしまう不具合修正 #4345

Closed
wants to merge 1 commit into from

Conversation

k-yamamura
Copy link
Contributor

@k-yamamura k-yamamura commented Oct 4, 2019

概要(Overview・Refs Issue)

仮会員が無効の場合、会員登録完了後直ぐにブラウザが閉じられたり通信が途絶えたりすると、
仮会員として作成されてしまう不具合修正

方針(Policy)

内部でリダイレクトされていたのが原因のためリダイレクトしないように変更する。

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

  • 既存機能の仕様変更
  • フックポイントの呼び出しタイミングの変更
  • フックポイントのパラメータの削除・データ型の変更
  • twigファイルに渡しているパラメータの削除・データ型の変更
  • Serviceクラスの公開関数の、引数の削除・データ型の変更
  • 入出力ファイル(CSVなど)のフォーマット変更

レビュワー確認項目

  • 動作確認
  • コードレビュー
  • E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
  • 互換性が保持されているか
  • セキュリティ上の問題がないか

@Yangsin
Copy link

Yangsin commented Jan 11, 2020

@k-yamamura Pull Requestありがとうございます。
確認に時間を要しておりますが、リポジトリ分割にともない以下のIssueへと引き継がせていただきました。
手数ですが、再度3系のリポジトリへPull Requestいただけますと幸いです。
EC-CUBE/ec-cube3#78

@Yangsin Yangsin closed this Jan 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants