-
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-05-06] [$250] [Held requests] Hold request link for the requestee shows "undefined" #37554
Comments
Triggered auto assignment to @greg-schroeder ( |
@greg-schroeder 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.Hold request reason link includes undefined What is the root cause of that problem?We are passing policy type here instead of the iouType and for a non-active policy it will be App/src/components/MoneyRequestHeader.tsx Line 98 in 9e40a11
Lines 301 to 304 in bae502e
What changes do you think we should make in order to solve the problem?Pass |
@greg-schroeder Whoops! This issue is 2 days overdue. Let's get this updated quick! |
@greg-schroeder Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
@greg-schroeder Huh... This is 4 days overdue. Who can take care of this? |
Job added to Upwork: https://www.upwork.com/jobs/~0166aa9b176af4b313 |
Setting to External per team discussion |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @sobitneupane ( |
Upwork job price has been updated to $250 |
ProposalPlease re-state the problem that we are trying to solve in this issue.Hold request reason link includes undefined What is the root cause of that problem?
What changes do you think we should make in order to solve the problem?I think we need to pass Navigation.navigate(ROUTES.MONEY_REQUEST_HOLD_REASON.getRoute(policy?.type ?? CONST.POLICY.TYPE.PERSONAL, parentReportAction?.originalMessage?.IOUTransactionID ?? '', report?.reportID, activeRoute)); Because currently, we only have 4 type of policies, other type What alternative solutions did you explore? (Optional)N/A |
@sobitneupane, @greg-schroeder Whoops! This issue is 2 days overdue. Let's get this updated quick! |
In latest main, when I try to reproduce the issue, the app crashes. Is it related to this issue? Or is there any other open issue to resolve it? Screen.Recording.2024-03-13.at.14.47.59.mov |
@bernhardoj Do you mean #38345 issue and #37554 issue are same? |
I would say they are the same and @Pujan92 solution will solve it. |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
Triggered auto assignment to @dangrous, see https://stackoverflow.com/c/expensify/questions/7972 for more details. |
Hi! I think this sounds good, but I'm going to take a little more time to confirm, since I just came back from leave and the Hold Request flow didn't exist when I left - so I have to get more familiar with it! |
Thanks @dangrous! Let us know if you need more eyes 👍 |
Okay, I think this makes sense to me! Thanks for your patience. I may have a couple questions along the way but let's move forward. |
❌ There was an error making the offer to @GandalfGwaihir for the Contributor role. The BZ member will need to manually hire the contributor. |
PR Update: waiting on @dangrous to merge the PR, I Think he's OoO :) |
Yep! I was out, looking at this shortly. |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.67-7 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-05-06. 🎊 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:
|
Yes
|
Regression Test Proposal
Do we agree 👍 or 👎 |
Processing |
$250 - C+ - @sobitneupane - you can make a ND manual request for payment I'll now file the regression test. |
Have accepted the offer, thanks :) |
$250 approved for @sobitneupane |
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.45-2
Reproducible in staging?: Y
Reproducible in production?: Y
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
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:
Expected Result:
There will be no "undefined" in Hold request link
Actual Result:
Hold request link for the requestee shows "undefined"
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6397422_1709230911407.20240301_004031.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: