Skip to content

Remove createJSModules @override marker - RN 0.47 compatibility#1628

Merged
guyca merged 2 commits into
wix:masterfrom
radko93:patch-1
Aug 3, 2017
Merged

Remove createJSModules @override marker - RN 0.47 compatibility#1628
guyca merged 2 commits into
wix:masterfrom
radko93:patch-1

Conversation

@radko93
Copy link
Copy Markdown
Contributor

@radko93 radko93 commented Aug 2, 2017

createJSModules is now not required on Android from RN 0.47. This is backwards compatible according to my tests.

@guyca
Copy link
Copy Markdown
Collaborator

guyca commented Aug 3, 2017

Thanks

@guyca guyca merged commit bef82c0 into wix:master Aug 3, 2017
@radko93 radko93 deleted the patch-1 branch August 3, 2017 08:10
timxyz pushed a commit to 3sidedcube/react-native-navigation that referenced this pull request Aug 3, 2017
* master: (95 commits)
  Update styling-the-navigator.md
  Support statusBarTextColorScheme on Android
  Remove depreciated createJSModules @OverRide marker (wix#1628)
  Fix crash with scrollViewDidEndDecelerating and RCTTextView (wix#1630)
  Center title when setting title dynamically
  update docs for iOS drawer animations in top-level-api section (wix#1619)
  walk-around for React-Native bug with multiple RCTRootView (wix#1618)
  Send initial screen appear event after it's displayed
  Send visibility events for TopTabs screens
  Update screen-api.md
  Push screen from drawer in example app
  Update top-level-api.md (Added forgotten styles) (wix#1600)
  Update readme for title Image on top bar (wix#1599)
  Update screen-api.md
  fix iOS crash on showController (wix#1588)
  Add show modal button to SideMenu
  Fix crash when SideMenu closes
  Add onKeyUp event to ActivityCallbacks
  Update screen-api.md
  Update top-level-api.md
  ...

# Conflicts:
#	android/app/src/main/java/com/reactnativenavigation/animation/VisibilityAnimator.java
#	ios/Helpers/RCTHelpers.h
#	ios/Helpers/RCTHelpers.m
#	ios/RCCViewController.m
timxyz pushed a commit to 3sidedcube/react-native-navigation that referenced this pull request Aug 3, 2017
* develop: (96 commits)
  Update styling-the-navigator.md
  Support statusBarTextColorScheme on Android
  Remove depreciated createJSModules @OverRide marker (wix#1628)
  Fix crash with scrollViewDidEndDecelerating and RCTTextView (wix#1630)
  Center title when setting title dynamically
  update docs for iOS drawer animations in top-level-api section (wix#1619)
  walk-around for React-Native bug with multiple RCTRootView (wix#1618)
  Send initial screen appear event after it's displayed
  Send visibility events for TopTabs screens
  Update screen-api.md
  Push screen from drawer in example app
  Update top-level-api.md (Added forgotten styles) (wix#1600)
  Update readme for title Image on top bar (wix#1599)
  Update screen-api.md
  fix iOS crash on showController (wix#1588)
  Add show modal button to SideMenu
  Fix crash when SideMenu closes
  Add onKeyUp event to ActivityCallbacks
  Update screen-api.md
  Update top-level-api.md
  ...
junedomingo pushed a commit to junedomingo/react-native-navigation that referenced this pull request Sep 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants