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

Add supports for RN 0.71 #26

Merged
merged 1 commit into from
Mar 4, 2023
Merged

Conversation

focux
Copy link
Contributor

@focux focux commented Jan 11, 2023

Starting from version 0.71, react native is not shipping the android folder (read more).

This library depends on the android folder to build correctly, so this PR adds support for 0.71 and above and it's not backward compatible with older versions. ⚠️ This is a breaking change.

Changes:

  • Uses RN core as prefabs on Android
  • Uses new buildscript

@focux focux marked this pull request as draft January 18, 2023 16:03
@teeserted
Copy link

teeserted commented Jan 22, 2023

Tried it on android RN 0.71, did not work, started receiving errors of different classes are defined multiple times. Tried cleaning and rebuilding project which did not help, after removing clusterer from deps, everything went back to normal

@JoniVR
Copy link
Contributor

JoniVR commented Feb 15, 2023

I can confirm that this fix still has some issues on my end as well. Though it's a good starting point. Currently this lib is blocking the upgrade for me.

@JoniVR
Copy link
Contributor

JoniVR commented Feb 15, 2023

@focux @teeserted @JiriHoffmann I made a PR in @focux 's fork that makes it work more consistently for me. Can you guys test/validate this?

focux#1

@teeserted
Copy link

@JoniVR Thanks to your work my android version is now building and clustering seems to work. (RN 71.1)

@focux focux marked this pull request as ready for review February 24, 2023 16:27
@JiriHoffmann JiriHoffmann merged commit a424e24 into JiriHoffmann:main Mar 4, 2023
@JiriHoffmann
Copy link
Owner

Merged. Thanks for the contributions!

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

4 participants