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: kotlin compiler error inferred type Context? #2096

Merged

Conversation

muhamad-rizki
Copy link
Contributor

Description

this PR will fix android compile error Type mismatch: inferred type is Context? but Context was expected

Test plan

manual build on existing apps

@j-piasecki
Copy link
Member

On what version of React Native have you got this error?

@muhamad-rizki
Copy link
Contributor Author

muhamad-rizki commented Jun 15, 2022

On what version of React Native have you got this error?

  • RN v0.67.4
  • kotlin v1.6.10
  • gradle v7.4.2
  • gradle plugin v7.2.1
  • buildToolsVersion = "33.0.0"
  • compileSdkVersion = 33
  • targetSdkVersion = 33

@j-piasecki j-piasecki merged commit 8d70e87 into software-mansion:main Jun 15, 2022
gnprice added a commit to gnprice/zulip-mobile that referenced this pull request Nov 9, 2022
This is needed in order to bump compileSdkVersion to 33
without hitting this build error:
  software-mansion/react-native-gesture-handler#2096

The upgrade requires a change in our Android MainActivity class,
like so:
  https://docs.swmansion.com/react-native-gesture-handler/docs/guides/migrating-off-rnghenabledroot/
chrisbobbe pushed a commit to gnprice/zulip-mobile that referenced this pull request Nov 11, 2022
This is needed in order to bump compileSdkVersion to 33
without hitting this build error:
  software-mansion/react-native-gesture-handler#2096

The upgrade requires a change in our Android MainActivity class,
like so:
  https://docs.swmansion.com/react-native-gesture-handler/docs/guides/migrating-off-rnghenabledroot/
For the details of that change, we follow Expo's template app:
  expo/expo@a2dc4da94
Discussion here:
  zulip#5543 (comment)
BrandonNgoranNtam pushed a commit to BrandonNgoranNtam/zulip-mobile that referenced this pull request Nov 22, 2022
This is needed in order to bump compileSdkVersion to 33
without hitting this build error:
  software-mansion/react-native-gesture-handler#2096

The upgrade requires a change in our Android MainActivity class,
like so:
  https://docs.swmansion.com/react-native-gesture-handler/docs/guides/migrating-off-rnghenabledroot/
For the details of that change, we follow Expo's template app:
  expo/expo@a2dc4da94
Discussion here:
  zulip#5543 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants