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: RN 0.73 support by solving #2263: Kotlin compilation fails due to "'hasConstants' overrides nothing" at CameraDevicesManager.kt #2264

Merged
merged 2 commits into from
Dec 12, 2023

Conversation

parlet
Copy link
Contributor

@parlet parlet commented Dec 8, 2023

fix: RN 0.73 support by solving #2263: Kotlin compilation fails due to "'hasConstants' overrides nothing" at CameraDevicesManager.kt

…ls due to "'hasConstants' overrides nothing" at CameraDevicesManager.kt
Copy link

vercel bot commented Dec 8, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
react-native-vision-camera ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 11, 2023 8:37am

@4cc3ssX
Copy link

4cc3ssX commented Dec 8, 2023

LGTM!

Tested on Redmi Note 11T 5G

  • Android Version 13 (MIUI 14)

@luas10c
Copy link

luas10c commented Dec 11, 2023

<@mrousavy> this fixes a problem.

@mrousavy
Copy link
Owner

Hey - can you please test if this still works in RN 0.72/0.71?

I remember that there could be warnings in the console if you don't override this...

@rivaslive
Copy link

Please you could merge this change? I working in react 0.73.0 @mrousavy

@parlet
Copy link
Contributor Author

parlet commented Dec 12, 2023

Hey - can you please test if this still works in RN 0.72/0.71?

I remember that there could be warnings in the console if you don't override this...

Hey @mrousavy, 👋

Thank you for reviewing this!

I made sure to check for backward compatibility with RN 0.72.7 before raising the PR, and I didn't encounter any problems or build time warnings.

@mrousavy
Copy link
Owner

Awesome, thanks for testing man - LGTM!

@mrousavy mrousavy merged commit 9949184 into mrousavy:main Dec 12, 2023
4 checks passed
isaaccolson pushed a commit to isaaccolson/deliveries-mobile that referenced this pull request Oct 30, 2024
…vicesManager.kt (mrousavy#2264)

Co-authored-by: Przemysław Arlet <przemyslaw.arlet@lamel.pl>
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.

6 participants