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.

Sign up for GitHub

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

Jump to bottom

Updated array element in mobile banking charge method. #413

Merged
merged 1 commit into from Nov 2, 2023
Merged

Updated array element in mobile banking charge method. #413

merged 1 commit into from Nov 2, 2023

Conversation

aashishgurung
Copy link
Contributor

@aashishgurung aashishgurung commented Nov 2, 2023 •
edited

1. Objective

Fix charge issue with mobile banking payment.

Related issue: #412

2. Description of change

  • Fixed the discrepancy on mobile banking code when setting platform_type

@aashishgurung aashishgurung requested review from danfowler and a team November 2, 2023 05:06
@danfowler danfowler removed the request for review from ajzkk November 2, 2023 05:33
@aashishgurung aashishgurung merged commit dc581d0 into develop Nov 2, 2023
4 checks passed
@aashishgurung aashishgurung deleted the fix_charge_with_mobile_banking branch November 2, 2023 07:11
This was referenced Nov 2, 2023
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@danfowler danfowler danfowler approved these changes

@AnasNaouchi AnasNaouchi AnasNaouchi approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@aashishgurung @danfowler @AnasNaouchi