Closed ghost closed 1 year ago
Thank you for your contribution, this will be a huge enhancement :-) I am going to review and test it next week :-)
I can't run the android/iOS app because the gradle/pod builds are failing. I think migrating to reat-native-test-app and yarn would simply the whole setup. Am i allowed to push to this branch? Otherwise i would do the changes on origin and then you can do a rebase.
@SimonErm Either works for me! I did add you as a collaborator to my repo so feel free to push there 🙂
@SBS-FPATE-RINALDI I migrated to react-native-test-app and yarn. iOS and Android are working now. MacOS can be build but crashes on start, but i don't know if its because of "macOS monteray" or if i have to configure something in the lib like it's done in react-native-webview(https://github.com/react-native-webview/react-native-webview/tree/master/macos/RNCWebView.xcodeproj). Can you test if it's still working on windows (https://github.com/microsoft/react-native-test-app#windows)?
@SimonErm Windows is also having some issues now. It can't find the HRMClient or the AppRegistry module. I'll do some more debugging this week to see if I can fix it and get back to you.
Sorry still looking into this.
The windows version is working when imported via NPM in a separate project that I'm working on, however still just having issues getting it to work in the example project in this repo. I'll also take a look at the Mac side when I can. Thank you for your patience!
@SBS-FPATE-RINALDI is the react-native-windows version in the seperate project the same like in the example of this repo?
I'll also take a look at the Mac side when I can.
Awesome, thank you :-)
Updated React Native version dependency to be 0.64.2 to work with MacOS and Windows
Addition of Windows
Addition of MacOS
Changes to Example project
Important Notes
I did not test this on Android or iOS as I do not have an easy way of doing so. Recommend someone else can do this to make sure everything works the same. I did test on Windows 10 and MacOS Catalina, and both worked as expected.
I needed these changes for a project I was working on, so figured it would be good to put in a PR in case anything is useful 😊