clevertap-react-native icon indicating copy to clipboard operation
clevertap-react-native copied to clipboard

Support RN new architecture

Open shubhamdeol opened this issue 2 years ago • 12 comments

This is an issue requesting support for Turbo Module architecture. Let me know how can we get started. I am open to contribute.

shubhamdeol avatar Dec 01 '22 18:12 shubhamdeol

@shubhamdeol Could you let us know why the current architecture is not suited to your needs?

darshanclevertap avatar Dec 02 '22 11:12 darshanclevertap

@darshanclevertap React Native community is migrating to new architecture it has some benefits like improved performance and start up time and much more. To be on new architecture it is required that all the dependencies must support new architecture. Following is the list of libraries has support for new RN architecture.

https://github.com/reactwg/react-native-new-architecture/discussions/categories/libraries

Suppose in future we plan to migrate to new Architecture as well. CleverTap dependency can be blocker for us. So I just want to confirm if you have any plans to support new architure near future.

shubhamdeol avatar Dec 12 '22 11:12 shubhamdeol

@shubhamdeol We try and keep our SDK up to date with the new features of React Native. So, we will be doing our research on this.

To be on new architecture it is required that all the dependencies must support the new architecture.

The biggest problem I see with this is backward compatibility. If we migrate our SDK to the new architecture, all our clients may or may not be able to use it. And it is very difficult to maintain 2 versions of the SDK.

Having said that, we will research this internally and decide whether it makes sense for us to migrate to the new architecture at this point in time.

I will keep this thread open and will respond as soon as we have more information

darshanclevertap avatar Dec 13 '22 05:12 darshanclevertap

@darshanclevertap okay. You will get a flag: newArchEnabled if user is on new Architecture or not. Most libraries are adding support for new architecture using that flag. You won't need to have 2 versions. In single version you can maintain the code using that flag. Currently very few forks are on new architecture. Soon this might become the business requirement for you as well.

shubhamdeol avatar Dec 14 '22 12:12 shubhamdeol

any updates on this ..? when it will be out ..? really need this

maulik54e avatar Apr 26 '23 16:04 maulik54e

@maulik54e This is on our roadmap for the current quarter. I will provide you an update as soon as we have something

darshanclevertap avatar Apr 27 '23 05:04 darshanclevertap

@darshanclevertap Any further update on this?

ajeetfancode avatar Jul 25 '23 12:07 ajeetfancode

@darshanclevertap the react native community uses this sheet to track library support for the new architecture (on by default in the next React Native release 0.74) and CleverTap isn't on this list. is this something that is being prioritised?

nickvgn avatar Apr 10 '24 00:04 nickvgn

@nickvgn

Yes, this has been prioritized by our team. We have started initial research on how we can adopt the new RN architecture without breaking anything from a backward compatibility POV. However, at this time, I won't be able to give you an ETA on this. I need to discuss this internally and I will provide an ETA by next week.

The Google Sheet shared by you is quite helpful to see how other libraries are adopting this new architecture. Can we know who is maintaining this sheet? We would want CleverTap to be added there so that developers know the status and eventual ETA for the new RN architecture changes.

darshanclevertap avatar Apr 10 '24 05:04 darshanclevertap

@darshanclevertap Thank you for your prompt response and for taking this seriously. I'm not sure who is maintaining the architecture support spreadsheet but this is where the discussion is taking place. There are plenty of resources there as well as other library authors trying to do the same. Hope that helped. Excited to see some updates on this soon.

nickvgn avatar Apr 10 '24 06:04 nickvgn

@darshanclevertap Do we have any update on this? We are trying to enable new architecture in our app and this is a blocker for us

ayushnathani avatar Jul 23 '24 11:07 ayushnathani

Same here, we are also looking to enable New Architecture but are stuck since CleverTap doesn't support New Architecture yet :(

udasi-jew-peter avatar Jul 24 '24 05:07 udasi-jew-peter