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

upgrade to RN0.42RC3 #24

Merged
merged 2 commits into from
Feb 16, 2017
Merged

upgrade to RN0.42RC3 #24

merged 2 commits into from
Feb 16, 2017

Conversation

nihgwu
Copy link
Contributor

@nihgwu nihgwu commented Feb 15, 2017

No description provided.

@SamSaffron
Copy link
Member

SamSaffron commented Feb 15, 2017

Cool, will confirm this all works shortly 👍

Can you email me at sam.saffron<at>gmail.com so I can sort out paypal

@SamSaffron
Copy link
Member

seem to be getting:

/Users/sam/Source/DiscourseMobile/node_modules/react-native/Libraries/ActionSheetIOS/RCTActionSheetManager.h:12:9: fatal error: 'React/RCTBridge.h' file not found
#import <React/RCTBridge.h>
        ^


** BUILD FAILED **


The following commands produced analyzer issues:
	Analyze RCTActionSheetManager.m
(1 command with analyzer issues)


The following build commands failed:
	CompileC /Users/sam/Source/DiscourseMobile/ios/build/Build/Intermediates/RCTSettings.build/Debug-iphonesimulator/RCTSettings.build/Objects-normal/x86_64/RCTSettingsManager.o RCTSettingsManager.m normal x86_64 objective-c com.apple.compilers.llvm.clang.1_0.compiler
(1 failure)

When I run

react-native run-ios

@SamSaffron
Copy link
Member

also totally failed when I try to compile in xcode :(

@nihgwu
Copy link
Contributor Author

nihgwu commented Feb 15, 2017

See the same errors when I download from zipped file, I'll try to figure out why, sorry for delay

@nihgwu
Copy link
Contributor Author

nihgwu commented Feb 15, 2017

Here is a trick, delete all the Libraries except React, build, then discard the change, now you can build successfully, but I still can't build from cli.

@nihgwu
Copy link
Contributor Author

nihgwu commented Feb 15, 2017

I need to sleep now, I think I've found the solution, will send a patch tomorrow, I'm from China 😄

@SamSaffron
Copy link
Member

SamSaffron commented Feb 15, 2017 via email

@nihgwu
Copy link
Contributor Author

nihgwu commented Feb 16, 2017

Hopefully I don't drop any configs

BTW:

  1. Why your target is 9.0+
  2. Orientation is supported by RN itself now, no need a 3rd-party library any more
  3. use useNativeDriver everywhere when available, will make Animation much more smooth

@SamSaffron
Copy link
Member

thanks, will merge soon, ping me via email!

@nihgwu
Copy link
Contributor Author

nihgwu commented Feb 16, 2017

can you build now? I'll wait until this be merged 😄

@SamSaffron SamSaffron merged commit cca251e into discourse:master Feb 16, 2017
@SamSaffron
Copy link
Member

Seems to be working find now ❤️ thanks so much

Why your target is 9.0+

I really did not want to support anything earlier, is there something else you would recommend here?

Orientation is supported by RN itself now, no need a 3rd-party library any more

Would love a PR to remove this dependency!

use useNativeDriver everywhere when available, will make Animation much more smooth

Would also love a PR for this.

@SamSaffron SamSaffron mentioned this pull request Feb 16, 2017
@jjaffeux
Copy link
Contributor

@nihgwu sorry to bother, do you know where the documentation is explaining Orientation please? Couldn't find anything.

@nihgwu
Copy link
Contributor Author

nihgwu commented Feb 17, 2017

@jjaffeux sorry it's my mistake, only Modal support Orientation

@jjaffeux
Copy link
Contributor

@nihgwu ok ;) thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants