search

callstack / react-native-brownfield

Set of helpers to make your brownfield integration smooth and easy.
MIT License
225 stars 14 forks source link

Memory Leak in Latest React Native #80

Open satheshrgs-rksv opened 1 week ago

satheshrgs-rksv commented 1 week ago

Environment

Hi With the latest react native brownfield, I'm facing memory leak. Attaching the screenshot for reference. Can be repro'ed using the master code

Screenshot 2024-06-27 at 20 41 04

harsh-rksv commented 1 week ago

I am facing a similar issue after the Upgrade. @okwasniewski can you please help here? We are using React Native 0.74.2

okwasniewski commented 1 week ago

Hey, thanks for opening the issue. I will take a look at this 👍🏻

harsh-rksv commented 1 week ago

To add more context our on issue faced: To support Android 14, we had to upgrade React to a version above 0.73. We were using the Brownfield project as a reference for our integration (thanks for this excellent brownfield repo 🤝 ). However, after the upgrade, our automation has detected memory leaks throughout our application.

Could you please provide a fix for this issue at the earliest? The Android 14 deadline is approaching quickly, and a prompt resolution would be greatly appreciated.

satheshrgs-rksv commented 1 day ago

@okwasniewski https://github.com/facebook/react-native/issues/45217#issuecomment-2208877868