Open IuliiaHerets opened 2 weeks ago
Triggered auto assignment to @muttmuure (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.
@muttmuure 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
@IuliiaHerets The video in OP doesn't match the step. Can you please update the video?
No overdue
Uploaded a correct video, sorry for it
Job added to Upwork: https://www.upwork.com/jobs/~021837165313806139268
Triggered auto assignment to Contributor-plus team member for initial proposal review - @akinwale (External
)
@akinwale, @muttmuure Whoops! This issue is 2 days overdue. Let's get this updated quick!
Tapping thread reply of expense hold offline, hmm not here displayed briefly.
childVisibleActionCount
of parent report action then when we delete the money request, the iouReport
is marked as pending delete in offline and it's cleared in success data hereWhen we hold a transaction here, we should update parentReportAction in optimistic data by using getOptimisticDataForParentReportAction
function as we do in other places
const optimisticParentReportActions = ReportUtils.getOptimisticDataForParentReportAction(reportID, currentTime, 'add');
optimisticParentReportActions.forEach((optimisticParentReportAction) => {
if (!optimisticParentReportAction) {
return;
}
optimisticData.push(optimisticParentReportAction);
});
We also have another problem after doing that, the expense report still displays as the combined report. The RCA is we don't check the singleAction
is deleted or not. To fix that we should return early if the singleAction
is deleted
if (((originalMessage?.deleted ?? '') !== '' || isDeletedAction(singleAction)) && isMoneyRequestAction(singleAction)) {
return;
}
https://github.com/user-attachments/assets/650d6e11-f34a-4d70-92aa-a9ec9cb16729
@akinwale proposal above for you
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸
@akinwale, @muttmuure Eep! 4 days overdue now. Issues have feelings 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!
Version Number: V9. 0.35-0 Reproducible in staging?: Y Reproducible in production?: Y Issue reported by: Applause Internal Team
Action Performed:
Expected Result:
Tapping thread reply of expense hold offline, hmm not here must not be displayed briefly.
Actual Result:
Tapping thread reply of expense hold offline, hmm not here displayed briefly.
Workaround:
Unknown
Platforms:
Screenshots/Videos
https://github.com/user-attachments/assets/4ad47104-3a16-4692-9976-cad4b1a72bcd
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @akinwale