-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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-08-01] [LOW] [Performance] Improve the performance of getDisplayNameForParticipant #45925
Comments
Triggered auto assignment to @johncschuster ( |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.11-5 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-08-01. 🎊 For reference, here are some details about the assignees on this issue:
|
BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
|
Payment Summary:Contributor+: @allgandalf owed $250 via NewDot |
I am paid via Upwork @johncschuster |
Nice so no checklist for this one and @allgandalf is paid on Newdot and we got a summary so we can close, thanks everyone! |
Problem
In the logs and profile traces from Jason's iOS app use, we can see that the getDisplayNameForParticipant method is taking a lot of processing time, forming a bottleneck.
Solution
Work on improving the performance of this method getDisplayNameForParticipant. The proposed solution from Callstack is to cache the phone number formatting
Issue Owner
Current Issue Owner: @johncschusterThe text was updated successfully, but these errors were encountered: