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

[0.18] Things we should cherry-pick to 0.18 #5201

Closed
7 of 11 tasks
mkonicek opened this issue Jan 8, 2016 · 12 comments
Closed
7 of 11 tasks

[0.18] Things we should cherry-pick to 0.18 #5201

mkonicek opened this issue Jan 8, 2016 · 12 comments
Labels
Resolution: Locked This issue was locked by the bot.

Comments

@mkonicek
Copy link
Contributor

mkonicek commented Jan 8, 2016

Moving the release to Monday Jan 18 as there are quite a few things to get in plus we had an internal hackathon today on RN + Buck.

@satya164
Copy link
Contributor

satya164 commented Jan 8, 2016

Should also cherry pick 4f9086f , 7ecb693

And may be this, 5a3b272 (Flow gives lots of warnings due to this, and is almost unusable, though there are other warnings too, this will at least decrease them)

Also in https://github.com/facebook/react-native/blob/master/ReactAndroid/src/main/java/com/facebook/react/ReactActivity.java , it now extends Activity instead of FragmentActivity. It will be difficult for third-party libs that require the use of FragmentActivity (refer #4827 (comment)). It might also break the DialogModule as the commit message says that it requires FragmentActivity - 3a3af8a

PR here - #5206

@ide
Copy link
Contributor

ide commented Jan 10, 2016

180fc06 Fixes iOS 7

@mkonicek
Copy link
Contributor Author

Thanks a lot guys! Commented on the PR #5206.

@jsierles
Copy link
Contributor

Also should get this once it's merged. A number of people have run into it including myself:

#4481

@mkonicek
Copy link
Contributor Author

Thanks @jsierles, added.

@skevy
Copy link
Contributor

skevy commented Jan 13, 2016

0065986

and

#5294

@mkonicek
Copy link
Contributor Author

Thanks @skevy! Commented on the PR, thanks for the fix!

@Ehesp
Copy link
Contributor

Ehesp commented Jan 15, 2016

1843709

This would be awesome too.

@tomauty
Copy link
Contributor

tomauty commented Jan 15, 2016

^ yes please

@mkonicek
Copy link
Contributor Author

The Android Picker (1843709) is not 100% ready yet, we want to clean up the API to make it more consistent with iOS.

@mkonicek
Copy link
Contributor Author

Releasing now. I haven't been able to cherry-pick the fix for react-native-xcode.sh (7ecb693), see comment on #5156.

The Android textAlign fix (#4481) requires a change to internal tests. We're now figuring out a way to run the Android integration tests both internally and on CircleCI and then open sourcing all of them so Android PRs can be merged more easily.

@satya164
Copy link
Contributor

Closing as 0.18 is already released.

@facebook facebook locked as resolved and limited conversation to collaborators May 24, 2018
@react-native-bot react-native-bot added the Resolution: Locked This issue was locked by the bot. label Jul 20, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Resolution: Locked This issue was locked by the bot.
Projects
None yet
Development

No branches or pull requests

8 participants