fix:codecovのバージョンをあげる - #5
Closed
dotani1111 wants to merge 1 commit into
Closed
Conversation
dotani1111
added a commit
that referenced
this pull request
May 19, 2026
CodeRabbit review (#5) の指摘に対応する. `BulkSendMail.getPromises` は `forEach` で全 fetch を並列発行しており, 進捗バーが実態と乖離するうえ SMTP への同時接続が発生していた. 同ファイル内の `BulkStatusUpdate.getPromises` (ポイント・在庫の加算減算用に既に順次実行 パターンが組まれている) と同じ `urls.shift()` + `.then()` のコールバック チェーンに揃え, 1 件ずつ送信する形に変更する. 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>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
概要(Overview・Refs Issue)
方針(Policy)
実装に関する補足(Appendix)
テスト（Test)
相談（Discussion）
マイナーバージョン互換性保持のための制限事項チェックリスト
レビュワー確認項目