[HOLD for payment 2023-07-20] Task Detailed View Redesign

[thienlnam]

Update the task detailed view to look like the above mockups
This issue is mostly focusing on the detailed view within the task. There will be another issue for the LHN / header title view as well as the deleted view, so no need to worry about those here

• Move the 'Mark as done' button to the header
• The task details float above the comments in the report and can be scrolled out of screen if there are enough comments
• The task title now contains a checkbox that can be used
• When there is no assignee or description, it should middle align the input

https://expensify.slack.com/archives/C04QEB4MJEQ/p1684396364608389

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~0184754c976c0dca40
• Upwork Job ID: 1666918061946351616
• Last Price Increase: 2023-06-15

[melvin-bot]

Triggered auto assignment to @JmillsExpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

[JmillsExpensify]

Oh cool, glad I randomly got assigned on this issue. I think those are good questions. Just a quick heads up that I'm out of office the remainder of this week.

[melvin-bot]

Triggered auto assignment to @twisterdotcom (NewFeature), see https://stackoverflowteams.com/c/expensify/questions/14418#:~:text=BugZero%20process%20steps%20for%20feature%20requests for more details.

[twisterdotcom]

I'm not super sure what we should do with this. Do we both need to pre-design how to handle all these questions?

[thienlnam]

Yup, looks like @JmillsExpensify and @shawnborton are leading up this new header view that works for tasks/money requests/threads

https://expensify.slack.com/archives/C04QEB4MJEQ/p1684777292283899

[thienlnam]

Reassigning since this is holding on that pre-design conversation

[shawnborton]

Addressing this in the threads.

[thienlnam]

Planning on making this external so we can get some movement on this issue - I've included the relevant mock-ups in the description - are we pretty set with this or are we still finalizing some elements? cc @JmillsExpensify @shawnborton

[JmillsExpensify]

Let's make sure that we don't include the icons! Those were removed

[JmillsExpensify]

Also, I still have some reservations here. Can we take this back to Slack?

[thienlnam]

This is ready now, updated the description with the latest convo from slack!

[melvin-bot]

Reviewing label has been removed, please complete the "BugZero Checklist".

[melvin-bot]

The solution for this issue has been 🚀 deployed to production 🚀 in version 1.3.39-11 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

• Implement Task Detailed View Redesign #20841

If no regressions arise, payment will be issued on 2023-07-20. 🎊

After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.

• External issue reporter
• Contributor that fixed the issue
• Contributor+ that helped on the issue and/or PR

As a reminder, here are the bonuses/penalties that should be applied for any External issue:

• Merged PR within 3 business days of assignment - 50% bonus
• Merged PR more than 9 business days after assignment - 50% penalty

[Christinadobrzyn]

Hired @fedirjh in Upwork

Internal job - https://www.upwork.com/ab/applicants/1666918061946351616/job-details
External job - https://www.upwork.com/jobs/~0184754c976c0dca40

@fedirjh or @jasperhuangg Should a regression test apply to this?

[Christinadobrzyn]

I'm going to be ooo until July 31st so going to unassign and assign a new teammate.

@JmillsExpensify At this time, we're reviewing PR and prepping for payment based on #19330 (comment)

I'll take this back if it's still open when I'm back

[melvin-bot]

Current assignee @JmillsExpensify is eligible for the Bug assigner, not assigning anyone new.

[jasperhuangg]

Hired @fedirjh in Upwork

Internal job - https://www.upwork.com/ab/applicants/1666918061946351616/job-details External job - https://www.upwork.com/jobs/~0184754c976c0dca40

@fedirjh or @jasperhuangg Should a regression test apply to this?

No need for a regression test, the existing regression tests should cover this flow

[jasperhuangg]

This is still on HOLD for payment not overdue

[jasperhuangg]

@fedirjh were you paid out?

[fedirjh]

were you paid out?

@jasperhuangg I was hired but not yet paid , I think @Christinadobrzyn is ooo till July 31st .

[jasperhuangg]

@JmillsExpensify Can you help make sure @fedirjh gets paid out? It looks like @Christinadobrzyn has already hired them on Upwork: #19330 (comment)

[jasperhuangg]

@JmillsExpensify friendly bump! Can we get @fedirjh paid out for reviewing the PR?

[JmillsExpensify]

Jumping in now! Thanks

[JmillsExpensify]

Alright, @fedirjh is all paid out. I didn't see any automation related to a an urgency bonus, or any other mention of it, so I've paid the $1k standard amount.