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(android): fix crash on startup with react-native 0.64 #1634

Merged
merged 1 commit into from
Oct 9, 2023
Merged

fix(android): fix crash on startup with react-native 0.64 #1634

merged 1 commit into from
Oct 9, 2023

Conversation

tido64
Copy link
Member

@tido64 tido64 commented Oct 9, 2023

Description

A bug in an older version of OkHttp (used by react-native 0.64) causes it to throw 'expected Android API level 21+ but was 33'.

Platforms affected

  • Android
  • iOS
  • macOS
  • Windows

Test plan

npm run set-react-version 0.64
yarn
# Switch to Node 16 otherwise Metro will fail
cd example
yarn android

# In a separate terminal
yarn start

@tido64 tido64 requested a review from kelset October 9, 2023 13:41
@github-actions github-actions bot added the platform: Android This affects Android label Oct 9, 2023
@tido64 tido64 enabled auto-merge (squash) October 9, 2023 13:49
@tido64 tido64 merged commit cbd5e05 into trunk Oct 9, 2023
27 checks passed
@tido64 tido64 deleted the tido/fix-0.64 branch October 9, 2023 14:55
@tido64 tido64 mentioned this pull request Oct 10, 2023
Closed
18 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kelset kelset kelset approved these changes

@arazabishov arazabishov Awaiting requested review from arazabishov

@sweggersen sweggersen Awaiting requested review from sweggersen

Assignees
No one assigned
Labels
platform: Android This affects Android
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@tido64 @kelset