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

Fix scale transform on Android #9251

Closed
wants to merge 1 commit into from

Conversation

janicduplessis
Copy link
Contributor

#8892 broke scale transforms on Android as the new implementation did a Z scale for the scale transform instead of a XY scale.

cc @kmagiera @astreet

@ghost
Copy link

ghost commented Aug 5, 2016

By analyzing the blame information on this pull request, we identified @kmagiera to be a potential reviewer.

@ghost ghost 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 Aug 5, 2016
@astreet
Copy link
Contributor

astreet commented Aug 5, 2016

@facebook-github-bot shipit

Thanks for jumping on this so quickly!

@ghost ghost added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Aug 5, 2016
@janicduplessis
Copy link
Contributor Author

@astreet Looks like shipit bot didn't pick that up.

@ghost ghost added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Aug 5, 2016
@astreet
Copy link
Contributor

astreet commented Aug 5, 2016

@facebook-github-bot shipit

@astreet
Copy link
Contributor

astreet commented Aug 5, 2016

If that doesn't work I'll post internally

@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 Aug 5, 2016
@facebook-github-bot
Copy link
Contributor

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

@janicduplessis
Copy link
Contributor Author

I think the bot doesn't pickup 'shipit' + some text after. It does work with some text + 'shipit' after.

@ghost ghost added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Aug 5, 2016
@ghost ghost closed this in 12fb313 Aug 6, 2016
cmcewen pushed a commit to cmcewen/react-native that referenced this pull request Aug 15, 2016
Summary:
cc kmagiera astreet
Closes facebook#9251

Differential Revision: D3679466

Pulled By: astreet

fbshipit-source-id: bd4262169827151ed584f7f8d167f9e6b8d533af
mpretty-cyro pushed a commit to HomePass/react-native that referenced this pull request Aug 25, 2016
Summary:
cc kmagiera astreet
Closes facebook#9251

Differential Revision: D3679466

Pulled By: astreet

fbshipit-source-id: bd4262169827151ed584f7f8d167f9e6b8d533af
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.

3 participants