Skip to content

feat: return result of android dialog dismissal #636

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

Merged
merged 1 commit into from
Jul 25, 2022
Merged

Conversation

vonovak
Copy link
Member

@vonovak vonovak commented Jul 25, 2022

Summary

On the native side, the android dismiss call returns a promise that the JS side does not return to the call site.

This fixes that.

Test Plan

tested locally

What's required for testing (prerequisites)?

What are the steps to reproduce (after prerequisites)?

Compatibility

OS Implemented
iOS
Android

Checklist

  • I have tested this on a device and a simulator
  • I added the documentation in README.md
  • I updated the typed files (TS and Flow)
  • I added a sample use of the API in the example project (example/App.js)
  • I have added automated tests, either in JS or e2e tests, as applicable

@vonovak vonovak merged commit 98335f2 into master Jul 25, 2022
@vonovak vonovak deleted the feat/dismiss-result branch July 25, 2022 21:55
vonovak pushed a commit that referenced this pull request Jul 25, 2022
## [6.2.2](v6.2.1...v6.2.2) (2022-07-25)

### Bug Fixes

* return result of android dialog dismissal to JS ([#636](#636)) ([98335f2](98335f2))
@vonovak
Copy link
Member Author

vonovak commented Jul 25, 2022

🎉 This issue has been resolved in version 6.2.2 🎉

If this package helps you, consider sponsoring us! 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant