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: avoid app crash due to multiple reject invocation #2669

Merged
merged 1 commit into from
Jan 15, 2024

Conversation

khrulev
Copy link
Contributor

@khrulev khrulev commented Jan 9, 2024

Probably, someone forgot to delete the old Reject when adding the condition to the code. This leads to an error and crash of the application in the development environment.

Steps To Reproduce:
The error occurs when the user has canceled iOS system prompt for Sync with App Store using StoreKit2.

Error Message:
The error says: 'Illegal callback invocation from native module. This callback type only permits a single invocation from native code.'

Solution:
Delete redundant reject.

Copy link
Owner

@hyochan hyochan left a comment

Choose a reason for hiding this comment

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

Thanks for the fixes!

@hyochan hyochan added 📱 iOS Related to iOS 🛠 bugfix All kinds of bug fixes labels Jan 15, 2024
@hyochan hyochan merged commit 13eab87 into hyochan:main Jan 15, 2024
1 check passed
arthurgeron pushed a commit to arthurgeron/react-native-iap that referenced this pull request Apr 2, 2024
Probably, someone forgot to delete the old **Reject** when adding the
condition to the code. This leads to an error and crash of the
application in the development environment.

**Steps To Reproduce**:
The error occurs when the user has canceled iOS system prompt for Sync
with App Store using StoreKit2.

**Error Message:**
The error says: 'Illegal callback invocation from native module. This
callback type only permits a single invocation from native code.'

**Solution**:
Delete redundant reject.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🛠 bugfix All kinds of bug fixes 📱 iOS Related to iOS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants