-
Notifications
You must be signed in to change notification settings - Fork 404
0.62.0 Discussion #179
Comments
From all of us! Heartily Thanks to all the contributors who made this release possible in the time of Global Pandemic. Also, by this comment I would request all to 'Stay At Home' and follow your Government Policies. |
Thank you for the amazing work getting this out in such difficult times. Would it be possible to get facebook/react-native#27574 backported into 0.62.x? commit -> facebook/react-native@7a4753d edited by @kelset to add link to commit |
From the bottom of my heart (high and vulnerable) i want to wish all of you to be safe, healthy and close with other fellow humans. We're in this together across the world. Thank you and lots of love. The world will be bright again, more tollerant and accepting! |
Hi guys! We need to cherry-pick this commit facebook/react-native@cfcf5eb to prevent crash in release build on Android with hermes engine and
ADDED. Look like it is actual for jsc too. Enabling proguard cause crash while app starting. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Hi! please pick this commit facebook/react-native@d3658bc |
This comment has been minimized.
This comment has been minimized.
@kesha-antonov @soroushm please open issues in the main repo to report your problems (first ofc check if someone already reported them), and link them back here. Also, if you think your issue may be related to upgrading, please refer to the new Upgrade Support repository. |
This comment has been minimized.
This comment has been minimized.
Can we pick facebook/react-native@405200e |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
to cherry pick too: facebook/react-native@227aa96 |
This comment has been minimized.
This comment has been minimized.
Hi 👋 |
This commit facebook/react-native@a5353c0 , I don't know why it's included in 0.61 but not in 0.62. Maybe because it's not added by PR so not on master? Any way this commit is very important as it fixed this Modal issue facebook/react-native#26473 @kelset Could we cherry pick the commit and make sure it will not be absent again in the next big version like 0.63.0? |
@xiao99xiao Those |
If someone encounters errors while building the ios app, try updating cocoapods. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
hey folks, quick update, we will try to get a 0.62.1 release out later today. For this patch release we will (likely*):
* as always, it depends on the cherry picking process & our local testing to go well 🤞 LMK if I forgot anything |
@kelset However this PR facebook/react-native#26490 is not a valid fix compared with the revert. |
@xiao99xiao Can you please elaborate on exactly why not and what you’d suggest we should be doing instead? |
@alloy OK. The key difference about the revert commit and the PR's solution is, the definition about dismiss on iOS. On iOS, when we say a Modal is dismissed, it means the view is not visible and its view controller is removed from the parent view controller. And another important thing is, you could only present a new Modal after the earlier one's view controller is removed too. The PR only watches on the visibility of the Modal view. However when the Modal view is not visible, its view controller could be not removed yet. Merging the PR will break one common logic that you need to present a Modal right after the previous one is dismissed. So that's why I insist using the revert commit: The PR and its former commit facebook/react-native@bd2b7d6 are not right about dismiss and break too much. |
Hey @xiao99xiao thank you a lot for your explanation 🙏 We've reached out to the author of the PR and he agrees with your assesment, so we will consider an alternative approach to solve the Modal issue (probably we will do the revert as you suggested in the first place). I'll write an update here once we have done what above |
This comment has been minimized.
This comment has been minimized.
Hey everybody, we just released v0.62.1: https://github.com/facebook/react-native/releases/tag/v0.62.1 🚀 You can see in @kelset’s comment what changes we applied or go directly to the CHANGELOG. Regarding the As for other changes that were requested but not yet landed on Thanks for all of your assistance and looking forward to continuing our collaborations on the v0.62.1 issue! |
Conversations on this thread are limited to 0.62 releases major issues and backport (cherry-pick) requests from commits that are already on master.
An example of a good such request is a bug fix for a serious issue that has been merged into master but did not make the 0.62.0 cut.
In other words, if you cannot point to a particular commit on master, then your request likely belongs as a new issue in http://github.com/facebook/react-native/issues.
We have released this version to respect the work of hundreds of contributors who made this release possible and to prevent the release from falling too far behind master. Please be mindful of the reduced capacity of contributors to help with issues and prepare to delay upgrading if necessary.
The text was updated successfully, but these errors were encountered: