-
Notifications
You must be signed in to change notification settings - Fork 2.7k
RN 77 Upgrade #8018
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
Merged
Merged
RN 77 Upgrade #8018
Changes from 48 commits
Commits
Show all changes
51 commits
Select commit
Hold shift + click to select a range
3ceabfe
Merge branch 'master' into daniel/rn77-ios-new-arch
kochavi-daniel 5d717d0
auto renaming
kochavi-daniel 4928798
changes
kochavi-daniel 23510df
native fixes - passes build with manual change in RCTAppSetupUtils.h
kochavi-daniel 3b4187d
all works still with change in RCTAppSetupUtils
kochavi-daniel 54b6c73
loads the menu - playground works
kochavi-daniel d68c0c0
fixed event receivers
kochavi-daniel 0c99bf7
hermes falg
kochavi-daniel 4a6ac23
fix side menu pres test
kochavi-daniel 18f7953
test ot .mm
kochavi-daniel 4f6b723
comment out test that fail linkage
kochavi-daniel 2ad25c5
fix testSetRoot_withAnimation and testPush_shouldResolvePromiseAndSen…
kochavi-daniel 1027229
fix linking by a hack
kochavi-daniel f6d5ff7
remove invalidation
kochavi-daniel afa721c
fix Stub in Turbo Module
kochavi-daniel b1e2391
Fix StaticLifecycleEvents tests
yogevbd 73eedbc
Delay will appear and did appear when component not already mounted
yogevbd 5c00a8d
Update RNNReactView to use RCTSurfacePresenterStub and adjust compone…
yogevbd 3e2223a
Better implementation
yogevbd 4cfc980
Fix buttons
yogevbd f4be349
Fix button order test
yogevbd 54bef68
Another buttons fix
yogevbd 986a9b9
Refactor component lifecycle methods in RNNReactButtonView and RNNRea…
yogevbd d7286c6
fix snapshot c++ error
kochavi-daniel 41bbb15
fix snapshot refactor mm
kochavi-daniel e1f2a87
fix snapshot c code
kochavi-daniel b412677
view controller factory
kochavi-daniel 429450c
moved turbo module spec to async
kochavi-daniel e733ae0
codegen type
kochavi-daniel b206437
cleanup native sender
kochavi-daniel 7656151
Fixed types
gosha212 3ffb4fc
adjust interface
kochavi-daniel 92a182b
fix remaining e2e
kochavi-daniel 56152f3
another test bites the dust
kochavi-daniel 129a413
Disable failing E2E, failing snapshot and Turbo Modules on Android
52d5732
comment ios test out
fca1e8d
wrong test
bb8cae9
Fixed android screenshot
gosha212 a7e5cf2
Update package.json version to 8.1.0-alpha01 [buildkite skip]
mobileoss f5bd327
Update package.json version to 8.1.0-alpha [buildkite skip]
mobileoss 9930d6e
Merge branch 'feat/rn77-android-newarch' into daniel/rn77-ios-new-arch
gosha212 84913d7
Merge branch 'daniel/rn77-ios-new-arch' into daniel/rn77-preview
gosha212 27f2658
Android Turbo Modules (#7999)
gosha212 497ebd6
WIP
gosha212 11b1e59
Revert "WIP"
gosha212 4a3ae44
Feat/rn 76 to 77 upgrade (#8005)
gosha212 3120643
Fixed android turbo modules generation and autolinking (#8006)
gosha212 346c51e
iOS: RN 77 no lifecycle events for Top Bar Title component (#8015)
markdevocht d64c109
Better alignment of willAppear and didAppear events throughout the li…
markdevocht fac5413
Fixing pr
gosha212 88898ef
More fixes for PR
gosha212 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,10 +1,10 @@ | ||
module.exports = { | ||
root: true, | ||
extends: ['@react-native-community', 'prettier', 'prettier/@typescript-eslint', 'prettier/react'], | ||
extends: ['@react-native', 'prettier', 'prettier/@typescript-eslint', 'prettier/react'], | ||
parser: '@typescript-eslint/parser', | ||
plugins: ['@typescript-eslint'], | ||
env: { | ||
jest: true, | ||
'jest/globals': true, | ||
}, | ||
}; | ||
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,5 @@ | ||
import "setimmediate"; | ||
import { LogBox } from 'react-native'; | ||
LogBox.ignoreAllLogs(); | ||
|
||
LogBox.ignoreAllLogs(); | ||
require('./playground/index'); |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
40 changes: 27 additions & 13 deletions
40
lib/android/app/src/main/java/com/reactnativenavigation/react/NavigationPackage.kt
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,27 +1,41 @@ | ||
package com.reactnativenavigation.react | ||
|
||
import com.facebook.react.BaseReactPackage | ||
import com.facebook.react.ReactApplication | ||
import com.facebook.react.ReactHost | ||
import com.facebook.react.ReactPackage | ||
import com.facebook.react.bridge.NativeModule | ||
import com.facebook.react.bridge.ReactApplicationContext | ||
import com.facebook.react.module.model.ReactModuleInfo | ||
import com.facebook.react.module.model.ReactModuleInfoProvider | ||
import com.facebook.react.uimanager.ViewManager | ||
import com.reactnativenavigation.options.LayoutFactory | ||
import com.reactnativenavigation.react.modal.ModalViewManager | ||
|
||
class NavigationPackage() : ReactPackage { | ||
override fun createNativeModules(reactContext: ReactApplicationContext): List<NativeModule> { | ||
val reactApp = reactContext.applicationContext as ReactApplication | ||
return listOf( | ||
NavigationModule( | ||
reactContext, | ||
LayoutFactory(reactApp.reactHost) | ||
) | ||
) | ||
class NavigationPackage() : BaseReactPackage() { | ||
|
||
override fun getModule(name: String, context: ReactApplicationContext): NativeModule? { | ||
val reactApp = context.applicationContext as ReactApplication | ||
return when (name) { | ||
NavigationTurboModule.NAME -> { | ||
NavigationTurboModule(context, LayoutFactory(reactApp.reactHost)) | ||
} | ||
else -> { | ||
null | ||
} | ||
} | ||
} | ||
|
||
override fun createViewManagers(reactContext: ReactApplicationContext): List<ViewManager<*, *>> { | ||
override fun getReactModuleInfoProvider() = ReactModuleInfoProvider { | ||
mapOf(NavigationTurboModule.NAME to ReactModuleInfo( | ||
name = NavigationTurboModule.NAME, | ||
className = NavigationTurboModule.NAME, | ||
canOverrideExistingModule = false, | ||
needsEagerInit = false, | ||
isCxxModule = false, | ||
isTurboModule = true | ||
)) | ||
} | ||
|
||
return listOf(ModalViewManager(reactContext)) | ||
override fun createViewManagers(reactContext: ReactApplicationContext): List<ViewManager<*, *>> { | ||
return mutableListOf(ModalViewManager(reactContext)) | ||
} | ||
} |
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.