Repository navigation
Charge cancellation fees - #8
illia-romanenko wants to merge 3 commits into
Conversation
There was a problem hiding this comment.
🔴 Critical risk · Payments
Charges cards and issues refunds. Read every changed line, run it
locally, and check the tests cover declined, partial and repeated payments.
Standards: PCI DSS v4.0.1 Req. 6.2.3 (custom code reviewed before
release) and 6.2.3.1 (reviewer is not the author)
Docs: Payments review checklist ·
Card data flow
Approvals: 2 needed · Who can approve: anyone with write access
Resolve this thread or reply /approve once you're done.
Why this file is governed
lib/payments/cancellation_fee.dart matched path /lib/payments/ (level payments). 40 lines changed.
Coance review0 of 2 files approved
Risk levelspayments: 2 approvals, not counting the author's own 🔴 Critical risk · PaymentsCharges cards and issues refunds. Read every changed line, run it Standards: PCI DSS v4.0.1 Req. 6.2.3 (custom code reviewed before patient-records: 2 approvals, not counting the author's own 🔴 Critical risk · Patient recordsReads or writes health records. Check every access is logged, every Standards: HIPAA Security Rule 45 CFR 164.312(b) audit controls, auth: 1 approval, not counting the author's own 🟠 High risk · Sign-in and permissionsCheck every new endpoint enforces authorization, and tokens and Standards: OWASP ASVS 5.0 V6–V8; SOC 2 CC6.1 infrastructure: 1 approval, not counting the author's own 🟠 High risk · Infrastructure and migrationsRuns against production. Read the plan output, check nothing new is Standards: ISO/IEC 27001:2022 A.8.9, A.8.32; SOC 2 CC8.1 app: 1 approval 🟡 Standard · App codeRead the file as a whole. Does the change make sense? Lines changed: 56 in total · payments 40 · patient-records 0 · auth 0 · infrastructure 0 · app 16 Policy: |
There was a problem hiding this comment.
🟡 Standard · App code
Read the file as a whole. Does the change make sense?
Approvals: 1 needed · Who can approve: anyone with write access
Resolve this thread or reply /approve once you're done.
Why this file is governed
test/payments/cancellation_fee_test.dart matched path *.dart (level app). 16 lines changed.
Second layer: charges a fee for late cancellations and no-shows, using the classification from the layer below.