Skip to content

Refund Logic Implementation - #41

Merged
AkibDa merged 2 commits into
GreenPlateByCodeNewbies:mainfrom
AkibDa:main
Jan 27, 2026
Merged

AkibDa merged 2 commits into
GreenPlateByCodeNewbies:mainfrom
AkibDa:main

Conversation

@AkibDa

@AkibDa AkibDa commented Jan 27, 2026

Copy link
Copy Markdown
Member

fix: correct Firestore batch update in cancel order flow

  • Fixed incorrect batch.update usage causing 500 error
  • Properly separate order and user updates in batch
  • Refund, resale, and staff payout logic now executes correctly

feat: add idempotency protection for payments and cancellations

  • Prevent double refunds on order cancellation
  • Make payment verification idempotent
  • Harden Razorpay webhook handlers against retries

@github-advanced-security github-advanced-security AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Prospector (reported by Codacy) found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

@github-advanced-security github-advanced-security AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pylintpython3 (reported by Codacy) found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

@github-advanced-security github-advanced-security AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pylint (reported by Codacy) found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

@AkibDa
AkibDa merged commit 19c3929 into GreenPlateByCodeNewbies:main Jan 27, 2026
9 checks passed
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