Closed kavimuru closed 1 year ago
Triggered auto assignment to @alexpensify (Bug
), see https://stackoverflow.com/c/expensify/questions/14418 for more details.
Platforms
in OP are ✅):wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash
deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:
Triggered auto assignment to @Li357 (Engineering
), see https://stackoverflow.com/c/expensify/questions/4319 for more details.
Is this a dupe of https://github.com/Expensify/App/issues/23568?
Hmm, seems not because this crash occurs right when the Split Bill button is clicked. Investigating
In case it helps... this may have the same root cause as: https://expensify.slack.com/archives/C049HHMV9SM/p1690832195690459
Not reproducible on main. Only staging.
I think it dup of/or same root cause https://github.com/Expensify/App/issues/23420
Seems like dupe. Then not sure why this is deploy blocker as https://github.com/Expensify/App/pull/17964 which caused this regression was deployed to production long time ago.
@situchan it's reproducible on staging only. and i don't think the linked PR is the cause no?
Agree, it seems to be the same root cause as https://github.com/Expensify/App/issues/23420 since currency
is undefined (because default props are not applied) when loading.
Shouldn't this bug be reproducible on production too if it has the same root cause as this pr https://github.com/Expensify/App/issues/23420?
True, I suspect there was some backend change that was merged recently that caused this intermediate case where currency
had no default. I'm going to search for it.
Hey, this is very likely a regression introduced by my PR! I am looking into it:
After looking into this, it seems it's a combo of changes that caused this issue. This PR will fix this error as well:
⚠️ Looks like this issue was linked to a Deploy Blocker here
If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.
If a regression has occurred and you are the assigned CM follow the instructions here.
If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.
⚠️ Looks like this issue was linked to a Deploy Blocker here
If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.
If a regression has occurred and you are the assigned CM follow the instructions here.
If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.
⚠️ Looks like this issue was linked to a Deploy Blocker here
If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.
If a regression has occurred and you are the assigned CM follow the instructions here.
If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.
⚠️ Looks like this issue was linked to a Deploy Blocker here
If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.
If a regression has occurred and you are the assigned CM follow the instructions here.
If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.
Should be fixed now, as of https://github.com/Expensify/App/pull/24003
Confirmed fix on staging!
Reviewing
label has been removed, please complete the "BugZero Checklist".
The solution for this issue has been :rocket: deployed to production :rocket: in version 1.3.48-5 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:
If no regressions arise, payment will be issued on 2023-08-08. :confetti_ball:
After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.
As a reminder, here are the bonuses/penalties that should be applied for any External 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:
The solution for this issue has been :rocket: deployed to production :rocket: in version 1.3.49-3 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:
If no regressions arise, payment will be issued on 2023-08-10. :confetti_ball:
After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.
As a reminder, here are the bonuses/penalties that should be applied for any External 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:
@Li357 - it appears the only payment here is to @Natnael-Guchima for reporting. Please confirm, so I can prepare for the upcoming pay period. Thanks!
@alexpensify Yep! That should be it.
Job added to Upwork: https://www.upwork.com/jobs/~01c58572b05d76428f
Triggered auto assignment to Contributor Plus for review of internal employee PR - @Santhosh-Sellavel (Internal
)
@Natnael-Guchima - to prepare for the upcoming pay period, I've sent a request in Upwork. Please accept and I can finalize the process on August 10. Thanks!
I have accepted the offer. Thanks @alexpensify
Here is the payment summary:
Upwork Job: https://www.upwork.com/jobs/~01c58572b05d76428f
*If applicable, the bonuses will be applied on the final payment
Extra Notes regarding payment: The payment has been paid in Upwork and the job has been closed there too.
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Action Performed:
Expected Result:
App shouldn't crash
Actual Result:
App crashes
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: 1.3.48-0 Reproducible in staging?: y Reproducible in production?: n If this was caught during regression testing, add the test name, ID and link from TestRail: Email or phone of affected tester (no customers): Logs: https://stackoverflow.com/c/expensify/questions/4856 Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/5237c6d0-ffc0-42d0-a7ab-f68f6dea40bb
https://github.com/Expensify/App/assets/43996225/19296699-e8cc-4660-b1aa-767d49677ab0
Expensify/Expensify Issue URL: Issue reported by: @natnael-guchima Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1690828817211769
View all open jobs on GitHub
Upwork Automation - Do Not Edit