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

[NavigationExperimental] fixes a bug where NavigationPropTypes.SceneRenderer was a plain object #7518

Closed
wants to merge 1 commit into from

Conversation

joenoon
Copy link
Contributor

@joenoon joenoon commented May 11, 2016

... used as both a shape and plain object.

this splits them out so both parts can be used as needed.

NavigationPropTypes.SceneRenderer is a PropTypes shape
NavigationPropTypes.SceneRendererProps is the plain object that makes up the shape.

@facebook-github-bot
Copy link
Contributor

By analyzing the blame information on this pull request, we identified @hedgerwang and @ericvicenti to be potential reviewers.

@facebook-github-bot facebook-github-bot added GH Review: review-needed CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. labels May 11, 2016
@joenoon joenoon force-pushed the jn-SceneRendererProps branch from f012de1 to 92b557b Compare May 13, 2016 02:31
@ghost
Copy link

ghost commented May 13, 2016

@joenoon updated the pull request.

@joenoon joenoon force-pushed the jn-SceneRendererProps branch from 92b557b to 431b9a5 Compare May 14, 2016 17:14
@ghost
Copy link

ghost commented May 14, 2016

@joenoon updated the pull request.

@joenoon joenoon force-pushed the jn-SceneRendererProps branch from f92c782 to 93f4499 Compare May 14, 2016 18:42
@ghost
Copy link

ghost commented May 14, 2016

@joenoon updated the pull request.

@joenoon
Copy link
Contributor Author

joenoon commented May 14, 2016

What is the preferred way to handle keeping the PR current with master?

So far, I've been periodically rebasing against master and force pushing my branch.

@joenoon joenoon force-pushed the jn-SceneRendererProps branch from 93f4499 to 83b03f3 Compare May 15, 2016 17:51
@ghost
Copy link

ghost commented May 15, 2016

@joenoon updated the pull request.

@joenoon joenoon changed the title fixes a bug where NavigationPropTypes.SceneRenderer was a plain object [NavigationExperimental] fixes a bug where NavigationPropTypes.SceneRenderer was a plain object May 15, 2016
used as both a shape and plain object.

this splits them out so both parts can be used as needed.

NavigationPropTypes.SceneRenderer is a PropTypes shape
NavigationPropTypes.SceneRendererProps is the plain object that makes up the shape.
@joenoon joenoon force-pushed the jn-SceneRendererProps branch from 83b03f3 to 127479d Compare May 17, 2016 19:15
@ghost
Copy link

ghost commented May 17, 2016

@joenoon updated the pull request.

@joenoon
Copy link
Contributor Author

joenoon commented May 17, 2016

@ericvicenti just checking to see if this will make 0.26? (which I believe happens today?) It seems sort of important to get in. I just rebased it again, hoping that helps.

@grabbou
Copy link
Contributor

grabbou commented May 18, 2016

@bestander @mkonicek what's the status here?

@ericvicenti
Copy link
Contributor

@facebook-github-bot shipit

@ericvicenti
Copy link
Contributor

Sorry about the delay on this one!

@ghost ghost added GH Review: accepted Import Started This pull request has been imported. This does not imply the PR has been approved. and removed GH Review: review-needed labels May 18, 2016
@ghost
Copy link

ghost commented May 18, 2016

Thanks for importing. If you are an FB employee go to Phabricator to review.

@ghost ghost closed this in 0e997c6 May 18, 2016
poppybank pushed a commit to poppybank/react-native that referenced this pull request May 19, 2016
Summary:
... used as both a shape and plain object.

this splits them out so both parts can be used as needed.

NavigationPropTypes.SceneRenderer is a PropTypes shape
NavigationPropTypes.SceneRendererProps is the plain object that makes up the shape.
Closes facebook#7518

Differential Revision: D3317322

Pulled By: ericvicenti

fbshipit-source-id: e8a31e05130e6647b63f68dbef31bc874550948c
zebulgar pushed a commit to nightingale/react-native that referenced this pull request Jun 18, 2016
Summary:
... used as both a shape and plain object.

this splits them out so both parts can be used as needed.

NavigationPropTypes.SceneRenderer is a PropTypes shape
NavigationPropTypes.SceneRendererProps is the plain object that makes up the shape.
Closes facebook#7518

Differential Revision: D3317322

Pulled By: ericvicenti

fbshipit-source-id: e8a31e05130e6647b63f68dbef31bc874550948c
bubblesunyum pushed a commit to iodine/react-native that referenced this pull request Aug 23, 2016
Summary:
... used as both a shape and plain object.

this splits them out so both parts can be used as needed.

NavigationPropTypes.SceneRenderer is a PropTypes shape
NavigationPropTypes.SceneRendererProps is the plain object that makes up the shape.
Closes facebook#7518

Differential Revision: D3317322

Pulled By: ericvicenti

fbshipit-source-id: e8a31e05130e6647b63f68dbef31bc874550948c
mpretty-cyro pushed a commit to HomePass/react-native that referenced this pull request Aug 25, 2016
Summary:
... used as both a shape and plain object.

this splits them out so both parts can be used as needed.

NavigationPropTypes.SceneRenderer is a PropTypes shape
NavigationPropTypes.SceneRendererProps is the plain object that makes up the shape.
Closes facebook#7518

Differential Revision: D3317322

Pulled By: ericvicenti

fbshipit-source-id: e8a31e05130e6647b63f68dbef31bc874550948c
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Import Started This pull request has been imported. This does not imply the PR has been approved.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants