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

fix: paze express checkout for multiple iframe #789

Merged
merged 4 commits into from
Nov 13, 2024
Merged

fix: paze express checkout for multiple iframe #789

merged 4 commits into from
Nov 13, 2024

Conversation

PritishBudhiraja
Copy link
Collaborator

Type of Change

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

Description

So need to fix one issue in the express checkout. It was throwing this error when I tried to combine two functions - handleApplePayIframePostMessage & handlePazeIframePostMessage

caml_option.js:24 Uncaught SecurityError: An attempt was made to break through the security policy of the user agent. at Module.handlePazeIframePostMessage (VM26169 Utils.bs.js:1367:170) at onPazeCallback (Elements.bs.js:118:7)

How did you test it?

Reverted back and made some changes started working.

Checklist

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

@PritishBudhiraja PritishBudhiraja added the Ready for Review PR with label Ready for Review should only be reviewed. label Nov 12, 2024
@PritishBudhiraja PritishBudhiraja self-assigned this Nov 12, 2024
@semanticdiff-com SemanticDiff.com
Copy link

Review changes with  SemanticDiff

sakksham7
sakksham7 previously approved these changes Nov 13, 2024
View reviewed changes
@PritishBudhiraja PritishBudhiraja merged commit 7af9e96 into main Nov 13, 2024
4 checks passed
@PritishBudhiraja PritishBudhiraja deleted the issue-express-checkout branch November 13, 2024 08:41
@github-actions github-actions bot added Closed Label will be automatically added when the PR will get merged to main and removed Ready for Review PR with label Ready for Review should only be reviewed. labels Nov 13, 2024
shubhamchasing pushed a commit to shubhamchasing/hyperswitch-web that referenced this pull request Nov 14, 2024
@semantic-release-bot
chore(release): 0.102.1 [skip ci]
324b556 
## [0.102.1](juspay/hyperswitch-web@v0.102.0...v0.102.1) (2024-11-13)

### Bug Fixes

* paze express checkout for multiple iframe ([juspay#789](juspay#789)) ([7af9e96](juspay@7af9e96))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ArushKapoorJuspay ArushKapoorJuspay ArushKapoorJuspay approved these changes

@sakksham7 sakksham7 sakksham7 left review comments

@seekshiva seekshiva Awaiting requested review from seekshiva seekshiva is a code owner

Assignees

@PritishBudhiraja PritishBudhiraja

Labels
Closed Label will be automatically added when the PR will get merged to main
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@PritishBudhiraja @ArushKapoorJuspay @sakksham7