daily-co / react-native-daily-js

https://docs.daily.co/reference/rn-daily-js

BSD 2-Clause "Simplified" License

36 stars 13 forks source link

BUG: iOS app fails to launch on iOS 15.4 #24

Closed arham-aiow closed 2 years ago

arham-aiow commented 2 years ago

Expected behavior

The app builds fine, but when I try launching it, it throws the following error and prevents the main view from loading. node_modules/@daily-co/daily-js/dist/daily-iframe.js: /Users/abcd/Data/abcd/mobile/node_modules/@daily-co/daily-js/dist/daily-iframe.js: Cannot read properties of undefined (reading 'env')

Screenshots

Simulator Screen Shot - iPhone 11 - 2022-03-31 at 20 05 42

System information

Building on Xcode 12 on Mac with M1 pro chip

* Device: iOS simulator (iPhone 11) * OS, version: iOS 15.4

filipi87 commented 2 years ago

Hi @arham-aiow ,

For validate the react-native-daily-js, we always use our playground app: https://github.com/daily-co/react-native-daily-js-playground

The current version of the playground is using:

react-native-daily-js 0.21.0
react-native-webrtc 1.94.1-daily.2

I have just tried to reproduce the issue here, but I have not been able to reproduce the issue that you are facing.

I have created an environment as you have described, just the Xcode that I am using a newer version: Xcode 13 on Mac with M1 pro chip. Device: iOS simulator (iPhone 11) OS, version: iOS 15.4

Are you able to reproduce the issue using our playground app ?
Can you share the versions of react-native-daily-js and react-native-webrtc that you are currently using ?
Are you doing something custom that could affect the process.env ?

arham-aiow commented 2 years ago

Hi @filipi87, Thanks for your response on this. I've finally managed to get it working on my project. Here's what didn't work:

Migrating the project to the same RN version as the playground app
Trying to build the daily-js lib in the node_modules folder to see if there might be any issues in the dist
Downgrading react-native-daily-js versions to various versions
Adding a .env file to the root, installing dotenv, updating the start script to add a NODE_ENV tag

After a tough ordeal of trying to make it all work and making all these changes on a mature project, what finally worked was commenting out the cwd entry frommodule-resolver inbabel.config.js

filipi87 commented 2 years ago

Hi @arham-aiow,

Great, glad you have made It work, and thanks for the feedback about your journey to make It work.

Let us know if you need any help!