-
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
[$500] Web - Task -Deleted task message & user avatar not greyed out when deleted offline #33986
Comments
Job added to Upwork: https://www.upwork.com/jobs/~01de665dd153fc50f2 |
Triggered auto assignment to @zanyrenney ( |
Bug0 Triage Checklist (Main S/O)
|
Triggered auto assignment to Contributor-plus team member for initial proposal review - @rushatgabhane ( |
ProposalPlease re-state the problem that we are trying to solve in this issue.Deleted task message & user avatar not greyed out when deleted offline What is the root cause of that problem?Currently, the App/src/pages/home/report/ReportActionItem.js Lines 581 to 598 in 0f6fdd3
What changes do you think we should make in order to solve the problem?we need to embed it into return (
<OfflineWithFeedback pendingAction={props.action.pendingAction}>
<AnimatedEmptyStateBackground />
<View style={[StyleUtils.getReportWelcomeTopMarginStyle(props.isSmallScreenWidth)]}>
<ReportActionItemSingle
action={parentReportAction}
showHeader={_.isUndefined(props.draftMessage)}
report={props.report}
>
<RenderHTML html={`<comment>${props.translate('parentReportAction.deletedTask')}</comment>`} />
</ReportActionItemSingle>
<View style={styles.reportHorizontalRule} />
</View>
</OfflineWithFeedback>
); POC |
ProposalPlease re-state the problem that we are trying to solve in this issue.The "Deleted task" message and user's avatar are not greyed out What is the root cause of that problem?Delete task has no offline feedback App/src/pages/home/report/ReportActionItem.js Line 586 in 2d455bf
What changes do you think we should make in order to solve the problem?Wrap the delete task with
*Note: We display delete task base on
What alternative solutions did you explore? (Optional)NA |
there is a typo here #33986 (comment) I mistakenly posted |
@rushatgabhane please can you review the proposals? |
I like @dukenv0307's proposal #33986 (comment) because it's more accurate |
Triggered auto assignment to @marcaaron, see https://stackoverflow.com/c/expensify/questions/7972 for more details. |
📣 @dukenv0307 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app! Offer link |
Assignging @dukenv0307 to work on this, thanks! |
Isn't this already wrapped though? App/src/pages/home/report/ReportActionItem.js Lines 723 to 724 in 2d455bf
Maybe I did not understand the proposal. |
@marcaaron The deleted task is rendered here. We return it without App/src/pages/home/report/ReportActionItem.js Line 585 in 2d455bf
|
@rushatgabhane The PR is ready for review. |
@dukenv0307 even then, some items aren't wrapped with I'm thinking if we could refactor the component so that everything is wrapped with App/src/pages/home/report/ReportActionItem.js Line 632 in a2f5bd5
|
@dukenv0307 i agree with @marcaaron #33986 (comment), could you dig deeper into why App/src/pages/home/report/ReportActionItem.js Lines 723 to 724 in 2d455bf
|
@rushatgabhane Other
|
@rushatgabhane Is there any other confuse here? Should PR be ready for review.? |
This issue has not been updated in over 15 days. @rushatgabhane, @marcaaron, @zanyrenney, @dukenv0307 eroding to Monthly issue. P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do! |
The PR is merged and just deployed to production @zanyrenney Please help to add weekly label again. |
payment request here- https://staging.new.expensify.com/r/6779795954208025 |
@zanyrenney could you please attach payment summary. thanks 🙇 |
Awaiting payment summary from @zanyrenney |
Payment summary:
Upwork job: https://www.upwork.com/jobs/~01de665dd153fc50f2 |
$500 approved for @rushatgabhane based on summary. |
Thank you @garrettmknight for handling this one! I didn't see this bump before my OOO. |
Issue can be closed |
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.22-0
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:
Issue found when executing PR #29730
Action Performed:
and log in
Expected Result:
The "Deleted task" message and user avatar are greyed out
Actual Result:
The "Deleted task" message and user's avatar are not greyed out
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6332647_1704411433137.Recording__89.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: