-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Bump react-native-onyx
to next version
#6195
Conversation
Please bump me on this once the PR is merged (if I forget) |
Cool. I left a comment there. |
This can come off hold now 👍 |
react-native-onyx
to next version
Gonna but a hold on this for 11/19. This PR will just add some logging for a weird edge case I am tracking, but it is not urgent and I will be OOO for the next week anyway. |
react-native-onyx
to next versionreact-native-onyx
to next version
react-native-onyx
to next versionreact-native-onyx
to next version
Taking off hold now that I am back from OOO and can keep an eye on this one :) |
✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release. |
🚀 Deployed to staging by @Julesssss in version: 1.1.16-11 🚀
|
🚀 Deployed to production by @roryabraham in version: 1.1.17-7 🚀
|
Details
See linked issue
Fixed Issues
$ Expensify/react-native-onyx#110
Tests
clitools generator:chatreport
) and trigger storage eviction (it will go faster if you alter the limit here) and you should see logs like thisQA Steps
We are just adding some additional logs here so there's not much to test besides obvious regressions.
Tested On