-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
[HOLD for payment 2024-11-13] Enable Bridgeless mode #48163
Comments
Triggered auto assignment to @abekkala ( |
Current assignee @abekkala is eligible for the NewFeature assigner, not assigning anyone new. |
thanks for creating this @mountiny! |
This will take some time to enable, swm is working on this |
We will want to post a proper problem solution to the channel before we make more progress on this one @WoLewicki @j-piasecki I will try to see if I can help with that |
updated proposal was shared in slack and I think we are good to move forward https://expensify.slack.com/archives/C05LX9D6E07/p1727354565454709 |
@mountiny is there a PR for this one yet? I'm going ooo until Oct 20 - I'd like to assign a buddy to watch this one until I return. But I'd like to better understand the progress status of this one first. |
Here it is: #48160 |
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.54-11 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue: If no regressions arise, payment will be issued on 2024-11-05. 🎊 |
BugZero Checklist: The PR adding this new feature has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
|
PAYMENT SUMMARY FOR NOV 13
|
Skipping the payment summary for this issue since all the assignees are employees or vendors. If this is incorrect, please manually add the payment summary SO. |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.57-10 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue: If no regressions arise, payment will be issued on 2024-11-13. 🎊 |
BugZero Checklist: The PR adding this new feature has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
|
Skipping the payment summary for this issue since all the assignees are employees or vendors. If this is incorrect, please manually add the payment summary SO. |
No Payments needed:
No regressions, closing this issue |
@abekkala I was C+ on this PR : #48160 . Correct payment summary is #48163 (comment) |
Thanks, @fedirjh. That's what I thought - Melvin's post after me confused me a bit. In this case their post was not helpful> Back to my original payment summary lol PAYMENT SUMMARY FOR NOV 13
|
$250 approved for @fedirjh |
Problem
We have updated the app to the 0.75 RN version now which allows us to use bridgeless mode. You can read more about what this mode is here. We have not enabled it yet, though, so we are missing out on the benefits that this mode brings. The main benefit should be improved performance, although we still need to test and measure how much performance gains there are for us in the App.
Solution
Enable the Bridgeless mode in the App and compare the performance with the mode on and off
Issue Owner
Current Issue Owner: @abekkalaThe text was updated successfully, but these errors were encountered: