-
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
[HOLD for payment 2024-09-26] [Search v2.1] Update View payment history
redirect to "Expensify, Inc" merchant filter.
#44105
Comments
Still holding on Search v2 introducing search querying for this. @luacmartins, do you have a more specific issue I can point to for this hold? Basically, we want to be able to filter the search page when they click this button to see all their billing receipts. |
This would be the issue #45026 |
Thanks! Title updated. |
View payment history
redirect to "Expensify, Inc" merchant filter. View payment history
redirect to "Expensify, Inc" merchant filter.
View payment history
redirect to "Expensify, Inc" merchant filter. View payment history
redirect to "Expensify, Inc" merchant filter.
@luacmartins seems like this issue might be able to come off hold now, what do you think? |
View payment history
redirect to "Expensify, Inc" merchant filter. View payment history
redirect to "Expensify, Inc" merchant filter.
Yes, draft PR here |
Niceeee! |
Just a heads up, the PR is still in draft because we discovered an issue with the parser grammar that was splitting |
Nice, thanks for the update. Just realised you're not assigned here either, so I'm adding yah to the issue. |
Still on hold for #47789 |
View payment history
redirect to "Expensify, Inc" merchant filter. View payment history
redirect to "Expensify, Inc" merchant filter.
@trjExpensify we're getting close to merging #47789, but I noticed that purchases don't have an associated entry in the transactions table so they won't show up in the Search page. That also seems to be the case in OldDot. Are we supposed to change this behavior? |
Hm, but when we bill someone we put the billing receipt in their Expensify account which creates an expense doesn't it? |
Hmm maybe it was the way I created the expense locally then 🤔 I just ran a script to run billing for a user, maybe there are more steps to it |
Yeah maybe, I'm not sure. |
View payment history
redirect to "Expensify, Inc" merchant filter. View payment history
redirect to "Expensify, Inc" merchant filter.
PR in staging |
View payment history
redirect to "Expensify, Inc" merchant filter. View payment history
redirect to "Expensify, Inc" merchant filter.
On prod, updated the title to reflect the payment date. |
Checklist
Regression Test Proposal
Do we agree 👍 or 👎 |
View payment history
redirect to "Expensify, Inc" merchant filter. View payment history
redirect to "Expensify, Inc" merchant filter.
Payment Summary
BugZero Checklist (@trjExpensify)
|
Offer sent! |
@trjExpensify Offer accepted, thank you! |
Done, paid. Closing! |
When we have merchant filtering, we want to update the redirect of the
View payment history >
menu item on the subscription page to navigate to Search with a filtered list of results formerchant: Expensify, Inc.
This issue is on hold until we complete that part of the Search v2 rollout, which is currently in detailed planning.
Issue Owner
Current Issue Owner: @luacmartinsThe text was updated successfully, but these errors were encountered: