-
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-03-25] [$500] Distance confirmation page shows (none) briefly when the map is loading #37850
Comments
Triggered auto assignment to @abekkala ( |
👋 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 @cead22 ( |
@cead22 FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors. |
ProposalPlease re-state the problem that we are trying to solve in this issue.Distance confirmation page shows (none) briefly when the map is loading What is the root cause of that problem?We display merchant as title for distance App/src/components/MoneyTemporaryForRefactorRequestConfirmationList.js Lines 691 to 695 in 98129e6
So until we get the value of distance from What changes do you think we should make in order to solve the problem?Change it to display the empty string if data is empty What alternative solutions did you explore? (Optional)N/A |
This isn't a deploy blocker, the PR which made the title to |
ProposalPlease re-state the problem that we are trying to solve in this issue.Distance confirmation page shows (none) briefly when the map is loading What is the root cause of that problem?There is a delay when loading the App/src/components/MoneyTemporaryForRefactorRequestConfirmationList.js Lines 691 to 694 in 98129e6
What changes do you think we should make in order to solve the problem?We can check if the - title={iouMerchant}
+ title={isMerchantEmpty ? '' : iouMerchant} As we have for merchant: App/src/components/MoneyTemporaryForRefactorRequestConfirmationList.js Lines 709 to 714 in 98129e6
What alternative solutions did you explore? (Optional) |
@GandalfGwaihir to prove that an issue marked as a |
I don't think we should block deploys on this, so let's treat it as a regular bug |
Job added to Upwork: https://www.upwork.com/jobs/~0113e37a7ba3d78e1d |
Good catch! - Change it to display the title for distance
+ Change it to display the empty string if data is empty ProposalPolice™ 👮🏼 should be coming soon to detect these things |
from my POV: I was in proceed of updating this changed and the other contributors posted in the mean time fine with whatever the engineer decides |
The good news is that we have plenty of open bugs and work for contributors :) and in the interest of assigning whoever posted first and being consistent with previous assignments I've done, I'm going to assign @usman-ghani564 |
❌ There was an error making the offer to @aimane-chnaif for the Reviewer role. The BZ member will need to manually hire the contributor. |
📣 @usman-ghani564 You have been assigned to this job! |
@aimane-chnaif Please find the PR for this issue here: #38075 |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.53-2 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-03-25. 🎊 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:
|
@usman-ghani564 Can you send me your upwork profile link, I'm having a hard time finding you. |
@abekkala Please find my Upwork profile here: https://www.upwork.com/freelancers/~01b5c4ee99eace8e60 |
PAYMENT SUMMARY FOR MAR 25
|
@slafortune will be handling payments for me while I'm ooo |
Paid 👍 |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Version Number: 1.4.48-0
Reproducible in staging?: y
Reproducible in production?: n
Issue reported by: Applause internal team
Action Performed:
Expected Result:
The Distance row will not show (none) when the distance data is loading (production behavior).
Actual Result:
The Distance row shows (none) briefly when the distance data is loading.
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6404688_1709765507166.bandicam_2024-03-07_06-48-09-918.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: