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

Request for unified 'react-native link` package setup for both iOS and Android #16

Closed
sunweiyang opened this issue Apr 11, 2018 · 12 comments
Labels
enhancement New feature or request WIP

Comments

@sunweiyang
Copy link

This opentok-react-native package has rather manual setup steps for iOS, and only uses react-native link for Android.

The common practice now is to use react-native link for efficient package setup, with optional manual steps to replace react-native link when problems arise.

@msach22
Copy link
Contributor

msach22 commented Apr 11, 2018

@sunweiyang Agreed! It's on our ToDo list to get it working for iOS as well!

@msach22 msach22 added the enhancement New feature or request label Apr 11, 2018
@ASerga
Copy link

ASerga commented Apr 24, 2018

Hello @msach22
Is it possible to install on iOS without using use_frameworks!?

Why

  1. Using use_frameworks! is slow and brings a lot of problems with some other packages like react-native-pdf, react-native-vector-icons...
  2. In cocoapods==1.5.0 already available Swift Static Libraries http://blog.cocoapods.org/CocoaPods-1.5.0/

Thanks!

@msach22
Copy link
Contributor

msach22 commented Apr 27, 2018

@ASerga Thanks for reporting this! I'll investigate further and see what we can do!

@msach22
Copy link
Contributor

msach22 commented May 11, 2018

@sunweiyang @ASerga Just wanted to update you folks that this is a WIP and should go out with 0.7.0

@msach22 msach22 added the WIP label May 17, 2018
@emin93
Copy link

emin93 commented May 30, 2018

Usually I'm not fanboying things, but this time: A BIG thank you! I was so close to giving up on integrating audio / video chat in my app before I stumbled upon OpenTok. Everything just worked flawlessly! Great documentation and clean integration.

As soon as the iOS setup works with react-native link as well, it will be the perfect module! 😃

P.S. GitHub needs an appreciation page for repositories 😄

@msach22
Copy link
Contributor

msach22 commented May 30, 2018

Thanks for the kind words @emin93!

I've opened a PR for the react-native link installation, but it still requires you to create an empty swift file in your iOS project to avoid a compiling error. If anyone else has a solution, please feel free to share!

@sunweiyang
Copy link
Author

sunweiyang commented May 30, 2018

@msach22 Thanks again! And perhaps this might give some insight:
https://github.com/rhdeck/react-native-swift
Also there's this stale issue about RN Swift built-in support: facebook/react-native#13030

@msach22
Copy link
Contributor

msach22 commented May 30, 2018

@sunweiyang Looks like this package is also creating an empty swift file: https://github.com/rhdeck/react-native-swift/blob/master/bin/postlink.js#L35

When you create a Swift file, XCode creates a Objective-C Generated Interface Header Name which is needed to import Swift into Objective-C. Looks like in this case also needed to be able to compile the Swift libraries.

@msach22
Copy link
Contributor

msach22 commented May 30, 2018

Also mentions the need to add an empty file:
https://github.com/rhdeck/react-native-swift/blob/master/README.md#how-it-works

@msach22
Copy link
Contributor

msach22 commented May 30, 2018

@sunweiyang In any case, I think I'm going to proceed with the empty swift file solution for now and improve as we can. I know it's not a optimal solution, but it is still a step forward because it allows you to use the library without having to use use_frameworks.

@mfranceschit
Copy link
Contributor

Thanks for this improvement @msach22 ! I've taken the desition to migrate to this library from react-native-opentok and this changes will be very useful to me because I'm having some problems with others libraries with the use_framework flag.

@msach22 msach22 mentioned this issue May 31, 2018
@msach22
Copy link
Contributor

msach22 commented May 31, 2018

Released with 0.7.0

@msach22 msach22 closed this as completed May 31, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request WIP
Projects
None yet
Development

No branches or pull requests

5 participants