Skip to content

Dev/vaddy 4.2 - #8

Closed
dotani1111 wants to merge 22 commits into
4.2from
dev/vaddy-4.2
Closed

Dev/vaddy 4.2#8
dotani1111 wants to merge 22 commits into
4.2from
dev/vaddy-4.2

Conversation

@dotani1111

Copy link
Copy Markdown
Owner

概要(Overview・Refs Issue)

方針(Policy)

実装に関する補足(Appendix)

テスト(Test)

相談(Discussion)

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

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

レビュワー確認項目

  • 動作確認
  • コードレビュー
  • E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
  • 互換性が保持されているか
  • セキュリティ上の問題がないか
    • 権限を超えた操作が可能にならないか
    • 不要なファイルアップロードがないか
    • 外部へ公開されるファイルや機能の追加ではないか
    • テンプレートでのエスケープ漏れがないか

@dotani1111
dotani1111 deleted the branch 4.2 April 3, 2026 01:46
@dotani1111 dotani1111 closed this Apr 3, 2026
dotani1111 added a commit that referenced this pull request May 19, 2026
CodeRabbit review (#8) の指摘に対応する.

`pond.on('initfile')` ハンドラが `product_image_error` という存在しない ID を
参照しており, アップロード前のエラーメッセージリセットが no-op になっていた.
同テンプレート内の `error` ハンドラで使われている正しい ID
`payment_image_error` に揃える.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
dotani1111 pushed a commit that referenced this pull request Jul 13, 2026
nanasess のレビュー指摘8件に対応。

- GA無条件ロード(OFF時)/同意撤回の非即時/GA越境移転/同意ログの個人データ該当を
  制限事項としてコード・ひな型に明記 (#1-#4)
- 更新APIの consent_status を all() で受け、配列など非スカラー入力でも
  BadRequestException でなくクリーンな 400 JSON へ落ちるよう堅牢化 (#6)
- source / previous_status を getConsentStatus() と同じ allowlist 方式で正規化 (#5)
- 更新APIのルート path を /cookie-consent/update に統一(表示系とハイフンで揃える。
  ルート名は不変、JS フォールバック・E2E の直叩きも追従)(#7)
- 拒否ボタンをグレー塗りにし同意ボタンと視覚的重みを対等化 (#8, EDPB 03/2022)

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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.

2 participants