Skip to content

Conversation

ArushKapoorJuspay
Copy link
Collaborator

Type of Change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring
  • Dependency updates
  • Documentation
  • CI/CD

Description

googlePay and applePay billing details not being passed in confirm call for saved methods

How did you test it?

Billing Details should be passed in the confirm body for ApplePay and GooglePay in Saved Payment Methods Flow

Checklist

  • I ran npm run re:build
  • I reviewed submitted code
  • I added unit tests for my changes where possible

Copy link
Contributor

@PritishBudhiraja PritishBudhiraja left a comment

Choose a reason for hiding this comment

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

Code wise looks good

@PritishBudhiraja PritishBudhiraja force-pushed the fix/wallet-billing-confirm branch from d8ef902 to e32916d Compare July 1, 2024 06:26
@PritishBudhiraja PritishBudhiraja merged commit 19db1d3 into main Jul 1, 2024
@PritishBudhiraja PritishBudhiraja deleted the fix/wallet-billing-confirm branch July 1, 2024 06:29
@github-actions github-actions bot added the Closed Label will be automatically added when the PR will get merged to main label Jul 1, 2024
akash-c-k pushed a commit that referenced this pull request Jul 1, 2024
## [0.71.8](v0.71.7...v0.71.8) (2024-07-01)

### Bug Fixes

* googlePay and applePay billing details not being passed in confirm call for saved methods ([#464](#464)) ([19db1d3](19db1d3))
rising-dragon401 added a commit to rising-dragon401/hyperswitch-web that referenced this pull request Jul 2, 2024
## [0.71.8](juspay/hyperswitch-web@v0.71.7...v0.71.8) (2024-07-01)

### Bug Fixes

* googlePay and applePay billing details not being passed in confirm call for saved methods ([#464](juspay/hyperswitch-web#464)) ([19db1d3](juspay/hyperswitch-web@19db1d3))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Closed Label will be automatically added when the PR will get merged to main
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants