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

fix: revert changes for hyper.res #548

Merged
merged 1 commit into from
Aug 7, 2024
Merged

fix: revert changes for hyper.res #548

merged 1 commit into from
Aug 7, 2024

Conversation

sakksham7
Copy link
Contributor

@sakksham7 sakksham7 commented Aug 7, 2024

Type of Change

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

Description

In an event listener, in a switch case we were handling all the events and in case of None we simply ignore the event but after removing the switch, the none case was not handled which was causing the issue, so this change is reverted in here and switch case is added.

How did you test it?

By rendering the SDK and checking confirm payment call and checking is the event being handled correctly or not

Checklist

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

@sakksham7 sakksham7 changed the title revert: revert changes for hyper.res fix: revert changes for hyper.res Aug 7, 2024
@PritishBudhiraja PritishBudhiraja merged commit 373cd40 into main Aug 7, 2024
5 checks passed
@PritishBudhiraja PritishBudhiraja deleted the revert-changes branch August 7, 2024 13:25
@github-actions github-actions bot added the Closed Label will be automatically added when the PR will get merged to main label Aug 7, 2024
akash-c-k pushed a commit that referenced this pull request Aug 7, 2024
## [0.82.1](v0.82.0...v0.82.1) (2024-08-07)

### Bug Fixes

* revert changes for hyper.res ([#548](#548)) ([373cd40](373cd40))
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.

3 participants