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.

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

Refund Payment Working from Admin #27

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

rdr6000
Copy link

@rdr6000 rdr6000 commented Sep 11, 2024

Refund from admin was not working due to fetch payment id block. I have updated the code and its currently working now.

@rdr6000
Copy link
Author

rdr6000 commented Sep 11, 2024

Working data in admin:
image

I am skipping all of this:

image

and replacing with this:

image

Using fetchPayments I get the payments block directly where as paymentIntent.payments was empty. This solves the refund issue
@SGFGOV

@SGFGOV
Copy link
Owner

SGFGOV commented Sep 11, 2024

Hey thanks. This good.. plz add tests

@SGFGOV
Copy link
Owner

SGFGOV commented Sep 11, 2024

Plz add tests also to validate the fix

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