Open m-natarajan opened 2 weeks ago
Triggered auto assignment to @sakluger (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.
Payment type is not mentioned in the details page.
We do not have a field for payment type. We only have fields for 'amount', 'comment', 'merchant', 'date', 'category' and 'tag'.
Add a new field similar to the one below for 'PaymentType' info: https://github.com/Expensify/App/blob/951c7b38e0ec83a876c5e1d24af81e9e6543fc5d/src/components/ReportActionItem/MoneyRequestView.tsx#L373-L386
PaymentType info is available in ParentReportAction
array that's passed to this component.
https://github.com/Expensify/App/blob/951c7b38e0ec83a876c5e1d24af81e9e6543fc5d/src/components/ReportActionItem/MoneyRequestView.tsx#L90
@sakluger Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!
Job added to Upwork: https://www.upwork.com/jobs/~014f39f2dacd55f2a4
Triggered auto assignment to Contributor-plus team member for initial proposal review - @ahmedGaber93 (External
)
@jainilparikh Thanks for the proposal.
I don't think we need to add a new field for payment type, we need to edit the header and IOU preview to be xyz paid $50.00 using Expensify
instead of xyz paid $50.00
@ahmedGaber93 , the reportAction does not have any information about how payment is done (Via Expensify or via Elsewhere). This is what we get:
avatarUrl: undefined
chatReportID: "8115357106458061"
chatType: ""
currency: "INR"
description: ""
errorFields: []
fieldList: undefined
invoiceReceiver: undefined
iouReportID: undefined
isDeletedParentAction: undefined
isOptimisticReport: undefined
isOwnPolicyExpenseChat: false
isPinned: false
isPolicyExpenseChat: undefined
isWaitingOnBankAccount: false
lastMentionedTime: undefined
lastReadTime: "2024-05-12 05:03:17.881"
lastVisibleActionCreated: "2024-05-12 05:03:17.881"
managerID: 16778387
nonReimbursableTotal: 0
notificationPreference: "always"
oldPolicyName: ""
ownerAccountID: 17102600
parentReportActionID: "189825184571791248"
parentReportID: "8115357106458061"
participantAccountIDs: (2) [16778387, 17102600]
pendingFields: undefined
permissions: (2) ['read', 'write']
policyID: "E1269E8555168358"
policyName: undefined
reportID: "7799079344254003"
reportName: "IOU"
stateNum: 2
Do we first need the BE to send this info in the parentReportActions
?
Not overdue, we're still discussing the expected behavior.
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸
@sakluger, @ahmedGaber93 Huh... This is 4 days overdue. Who can take care of this?
I checked it today, we already have the paymentType in FE but not with this reportAction, I will complete checking tomorrow to know if we can catch it from FE, or it requires BE changes
Triggered auto assignment to @sakluger (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.
Triggered auto assignment to @alexpensify (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.
Reassigning while I'm OOO for two weeks.
Current assignee @sakluger is eligible for the Bug assigner, not assigning anyone new.
Al will also be OOO for the same time, re-reassigning!
Triggered auto assignment to @johncschuster (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.
I don't think we need to BE changes as we already have paymentType
in FE, and we can use it.
@ahmedGaber93 , the reportAction does not have any information about how payment is done (Via Expensify or via Elsewhere). Do we first need the BE to send this info in the parentReportActions ?
@jainilparikh we already have it in another place, so no need to BE changes.
@trjExpensify can you please confirm is this the expected behavior?
or this https://github.com/Expensify/App/issues/41645#issuecomment-2102612537?
What you're showing in that screenshot is what's expected. The report is missing the payment reportAction.
@johncschuster @ahmedGaber93 this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!
I can't reproduce in latest main. Are I missed something? @trjExpensify
https://github.com/Expensify/App/assets/41129870/34036123-024d-44f4-91a8-1fe55500ddfd
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.70-2 Reproducible in staging?: Yes Reproducible in production?: Yes 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: @trjExpensify Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1714729020479179
Action Performed:
Prerequisite: Pay someone with either Expensify or elsewhere option
Expected Result:
Method of payment displayed in both DM and details page (Expensify or elsewhere)
Actual Result:
Only xyz paid displayed and method of payment not displayed
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @ahmedGaber93