-
Notifications
You must be signed in to change notification settings - Fork 136
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
Add prepare context #237
Add prepare context #237
Conversation
Codecov Report
@@ Coverage Diff @@
## master #237 +/- ##
=========================================
- Coverage 75.09% 74.8% -0.29%
=========================================
Files 13 13
Lines 261 262 +1
Branches 55 55
=========================================
Hits 196 196
- Misses 48 49 +1
Partials 17 17
Continue to review full report at Codecov.
|
2 similar comments
Codecov Report
@@ Coverage Diff @@
## master #237 +/- ##
=========================================
- Coverage 75.09% 74.8% -0.29%
=========================================
Files 13 13
Lines 261 262 +1
Branches 55 55
=========================================
Hits 196 196
- Misses 48 49 +1
Partials 17 17
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #237 +/- ##
=========================================
- Coverage 75.09% 74.8% -0.29%
=========================================
Files 13 13
Lines 261 262 +1
Branches 55 55
=========================================
Hits 196 196
- Misses 48 49 +1
Partials 17 17
Continue to review full report at Codecov.
|
@@ -80,6 +81,7 @@ export default function prepare(element: any) { | |||
} | |||
PrepareContextProvider.childContextTypes = { | |||
__PREPARE_STATE__: () => {}, | |||
__IS_PREPARE__: () => {}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be false?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, this is context types, which should be a prop type function. But rather than use that, a noop function suffices instead of using a real prop type.
Triggered Fusion.js build verification: https://buildkite.com/uberopensource/fusion-release-verification/builds/1656 |
This is needed by the react-router plugin now that client-side prepare is run again