[HOLD for payment 2024-07-22] [HOLD for payment 2024-07-10] [LOW] [Performance] Improve the TTI for native apps

[mountiny]

Coming from here

Problem

The TTI time for Android app using the heavy account is 60s. For iOS App its 16s. This would deter any user from using the App as they would not wait that long for it to start being usable

Solution

Investigate how we can speed up the start up time for native devices

Issue Owner

Current Issue Owner: @JmillsExpensify

[melvin-bot]

Triggered auto assignment to Contributor-plus team member for initial proposal review - @abdulrahuman5196 (External)

[melvin-bot]

Current assignee @mountiny is eligible for the AutoAssignerNewDotQuality assigner, not assigning anyone new.

[melvin-bot]

Triggered auto assignment to @JmillsExpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

[hurali97]

@mountiny Please assign this to me 👋

[melvin-bot]

@JmillsExpensify, @mountiny, @abdulrahuman5196, @hurali97 Whoops! This issue is 2 days overdue. Let's get this updated quick!

[mountiny]

@hurali97 what are the next steps?

[JmillsExpensify]

Still waiting for clarity on next steps

[melvin-bot]

@JmillsExpensify, @mountiny, @abdulrahuman5196, @hurali97 Huh... This is 4 days overdue. Who can take care of this?

[mountiny]

@hurali97 is working on this one

[abdulrahuman5196]

Hi Team, I have limited availability to work on issues currently, so unassigning myself to avoid further delays and reproritize. Kindly add another C+.
@mountiny

[mountiny]

@hungvu193 is taking one

[melvin-bot]

Reviewing label has been removed, please complete the "BugZero Checklist".

[melvin-bot]

The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.3-7 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

• [NoQA] chore: bump onyx to 2.0.54 #44457

If no regressions arise, payment will be issued on 2024-07-10. 🎊

For reference, here are some details about the assignees on this issue:

• @hungvu193 requires payment through NewDot Manual Requests
• @hurali97 does not require payment (Contractor)

[melvin-bot]

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:

• [@hungvu193] The PR that introduced the bug has been identified. Link to the PR:
• [@hungvu193] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:
• [@hungvu193] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:
• [@hungvu193] Determine if we should create a regression test for this bug.
• [@hungvu193] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.
• [@JmillsExpensify] Link the GH issue for creating/updating the regression test once above steps have been agreed upon:

[hoangzinh]

@mountiny can you help assign me to this issue so that we can handle payment here when everything is done with @hungvu193 ? I helped to review the Onyx PR Expensify/react-native-onyx#563

[melvin-bot]

Payment Summary

Upwork Job

• ROLE: @hoangzinh paid $500 via Upwork
• Reviewer: @hungvu193 owed $500 via NewDot
• Contributor: @hurali97 is from an agency-contributor and not due payment

BugZero Checklist (@JmillsExpensify)

• I have verified the correct assignees and roles are listed above and updated the neccesary manual offers
• I have verified that there are no duplicate or incorrect contracts on Upwork for this job (https://www.upwork.com/ab/applicants//hired)
• I have paid out the Upwork contracts or cancelled the ones that are incorrect
• I have verified the payment summary above is correct

[mountiny]

@hoangzinh @hungvu193 what is the payment summary on this one?

[hoangzinh]

I recall the total is 1000$ https://expensify.slack.com/archives/C02NK2DQWUX/p1719302159058329?thread_ts=1719301209.052599&cid=C02NK2DQWUX. 500$ for each person, what do you think? 🤔

[JmillsExpensify]

@mountiny Can you provide a payment summary confirming that you agree?

[mountiny]

Yes that sounds good to me, thank you for you quick help on this tricky task

@hungvu193 $500
@hoangzinh $500

[melvin-bot]

@JmillsExpensify, @hoangzinh, @hungvu193, @mountiny, @hurali97 Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

[melvin-bot]

The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.6-8 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

• perf: improve native apps startup #44068

If no regressions arise, payment will be issued on 2024-07-22. 🎊

For reference, here are some details about the assignees on this issue:

• @hoangzinh requires payment (Needs manual offer from BZ)
• @hungvu193 requires payment through NewDot Manual Requests
• @hurali97 does not require payment (Contractor)

[melvin-bot]

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:

• [@hoangzinh / @hungvu193] The PR that introduced the bug has been identified. Link to the PR:
• [@hoangzinh / @hungvu193] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:
• [@hoangzinh / @hungvu193] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:
• [@hoangzinh / @hungvu193] Determine if we should create a regression test for this bug.
• [@hoangzinh / @hungvu193] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.
• [@JmillsExpensify] Link the GH issue for creating/updating the regression test once above steps have been agreed upon:

[JmillsExpensify]

@hoangzinh offer sent via Upwork. @hungvu193 feel free to submit a request via NewDot.

[hoangzinh]

Accepted. Thanks @JmillsExpensify

[hungvu193]

Thanks. I'll request on NewDot later.

[JmillsExpensify]

Awesome. I've paid out the Upwork contract so we can circle back on your payment when I process in New Expensify.

[JmillsExpensify]

$500 approved for @hungvu193