-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[Due for payment 2025-02-26] Bring the "done" report status filter to New Expensify #56488
Comments
Triggered auto assignment to @dannymcclain ( |
I think what we need to decide first is what icon we want to use for closed @dannymcclain ![]() |
This comment has been minimized.
This comment has been minimized.
One thing I think is worth mentioning is that being closed is a positive thing in a sense, and maybe that might influence the icon we choose. In other words, when a report is closed, it has been submitted, which means it has progressed down the track toward either payment or export as a next step (depending on whether there are reimbursable expenses on the report or not). Those examples feel a little negative to me for want of a better observation Also, I think it makes sense to position Closed between Outstanding and Approved when you think about the flow: Approvals disabled: Approvals enabled: |
Oh hmm I think that does change things 🤔 |
Will we only show this filter to people that have a workspace with submit&close, is that right? |
Correct but someone could be a member of both types of workspaces so they could see both Closed and Approved |
Yeah, in theory, though I'd still vote to put |
Coming from this thread we landed on doing this: https://expensify.slack.com/archives/C03TME82F/p1738968491101599?thread_ts=1738884571.891239&cid=C03TME82F
|
@dubielzyk-expensify when you have time, will you clean up that icon and properly add it to the library? 🙏 |
Will do 👍 |
Thanks @dubielzyk-expensify , once we've got the new property added I can label this "new feature" and keep the ball rolling! |
I'll work on this one |
@MitchExpensify can you confirm the name we'll use for the filter? The OP says |
Got a couple of PRs up, just waiting on the confirmation above |
It's "done" @luacmartins, @dubielzyk-expensify, can you please share the icon we landed on, please? (The one in the OP) Thank you! |
I see that the OP icon for |
@luacmartins I hadn't realized that, but I do kinda think it makes sense to change it? That's the icon we use in the |
Cool, so which icons should I use for this one? Can you share the assets here please? I think we already have the checkmark one in app, but I couldn't find the checkbox one |
I think @dubielzyk-expensify is sick right now, but he's on it. ☑ |
Thanks! PRs in review |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.1.0-2 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue: If no regressions arise, payment will be issued on 2025-02-26. 🎊 For reference, here are some details about the assignees on this issue:
|
Background:
In #wave-collect: Search v2 we introduced a
done
badge, which shows up within thepaid
status. We decided against adone
orfinished
default state because the final state for IOUs ispaid
, and IOUs are not a feature that can be turned off. In a similar fashion for workspaces, the majority of our customers track or make payments (even if they don’t pay in product).Problem:
The
paid
report status is confusing forclosed
reports in New Expensify in two scenarios;paid
upon submission.paid
upon submission before any payment has been actioned.Solution:
Stop including
closed
expense reports that have no reimbursable expenses or reimbursement has not been actioned yet in thepaid
status filter, and house them in their owndone
status filter instead. Like Classic,done
reports that have reimbursement action on them will move to thepaid
status filter.Notes for completeness:
Tests:
Preconditions: Use a workspace with approvals disabled
The text was updated successfully, but these errors were encountered: