[HOLD for payment 2023-07-26] Update Expense Report View

[grgia]

Part 1 of #20486 (comment)
Related to #20486

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~01809b70d955011b47
• Upwork Job ID: 1676899649993342976
• Last Price Increase: 2023-07-06

[melvin-bot]

Job added to Upwork: https://www.upwork.com/jobs/~01809b70d955011b47

[melvin-bot]

Triggered auto assignment to Contributor Plus for review of internal employee PR - @fedirjh (Internal)

[grgia]

@trjExpensify @shawnborton right now we don't show the three dots on the report view and only show it on the single transaction view. Is the new design with the three dots correct? What shows when you click them?

[shawnborton]

Yeah, I left a similar comment over in the other issue but I think we should stay consistent with the overflow menu on all of the various report type views (chat, thread, task, report, expense, etc). And then in the very least, it always has a pin option but it also might have:

• call
• delete
• etc

[melvin-bot]

⚠️ Looks like this issue was linked to a Deploy Blocker here

If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.

If a regression has occurred and you are the assigned CM follow the instructions here.

If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.

[melvin-bot]

⚠️ Looks like this issue was linked to a Deploy Blocker here

If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.

If a regression has occurred and you are the assigned CM follow the instructions here.

If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.

[melvin-bot]

⚠️ Looks like this issue was linked to a Deploy Blocker here

If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.

If a regression has occurred and you are the assigned CM follow the instructions here.

If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.

[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.42-26 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

• Update Expense Report View / Headers #22484

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

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

[melvin-bot]

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

[grgia]

@conorpendergrast could you help to handle the $1000 C+ payment to @fedirjh for #22484

[conorpendergrast]

I would happily do so

[conorpendergrast]

Contract sent via Upwork, and I'll be able to see that here

[conorpendergrast]

• External issue reporter: N/A - no one
• Contributor that fixed the issue: N/A - no one
• Contributor+ that helped on the issue and/or PR: @fedirjh for $1000. Offer link. Contract link.
• Urgency bonus: No

[fedirjh]

@conorpendergrast Thank you, accepted.

[conorpendergrast]

Paid! @grgia Is that everything? If so, close it out

[conorpendergrast]

@grgia I'm assuming that this is all done. Closing out. Please reopen if I'm wrong!