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.

Sign up for GitHub

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

Jump to bottom

ログインに失敗しても設定されている戻り先URLへ正常に遷移するように対応 #4500

Merged
merged 1 commit into from
Jul 22, 2020
Merged

ログインに失敗しても設定されている戻り先URLへ正常に遷移するように対応 #4500

merged 1 commit into from
Jul 22, 2020

Conversation

k-yamamura
Copy link
Contributor

概要(Overview・Refs Issue)

ログイン失敗後に戻り先URLが消えてしまうため正常に遷移するように対応

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

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

レビュワー確認項目

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

@okazy okazy added the bug:Low label Apr 7, 2020
@okazy okazy added this to the 4.0.x milestone Apr 7, 2020
@okazy
Copy link
Contributor

okazy commented Apr 7, 2020

おそらく以下のプルリクも同じ問題に対しての対策かと思います。
#4455

対応方針が全く違うので、どちらの方法が良いかご意見が欲しいです。

@kiy0taka kiy0taka mentioned this pull request Jul 17, 2020
Closed
11 tasks
@kiy0taka kiy0taka merged commit 4d2318e into EC-CUBE:4.0 Jul 22, 2020
@okazy okazy mentioned this pull request Jul 29, 2020
Open
@okazy okazy modified the milestones: 4.0.x, 4.0.5 Aug 21, 2020
@okazy okazy added the affected:template フロントテンプレートの変更 label Aug 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kiy0taka kiy0taka kiy0taka approved these changes

Assignees
No one assigned
Labels
affected:template フロントテンプレートの変更 bug:Low Status: ready-for-merge
Projects
None yet
Milestone
4.0.5
Development

Successfully merging this pull request may close these issues.
3 participants
@k-yamamura @okazy @kiy0taka