App icon indicating copy to clipboard operation
App copied to clipboard

[HOLD for payment 2024-12-17] Add `Posted:` for filtering card transactions by statement dates

Open JmillsExpensify opened this issue 1 year ago • 17 comments

We missed adding this filter in the Search v2 design doc, so let's add it now. Like other date filters, it'll accept YYYY-MM-DD and will apply exclusively to the Expensify Card.

Issue OwnerCurrent Issue Owner: @JmillsExpensify

JmillsExpensify avatar Nov 12 '24 11:11 JmillsExpensify

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

melvin-bot[bot] avatar Nov 12 '24 11:11 melvin-bot[bot]

:warning: It looks like this issue is labelled as a New Feature but not tied to any GitHub Project. Keep in mind that all new features should be tied to GitHub Projects in order to properly track external CAP software time :warning:

melvin-bot[bot] avatar Nov 12 '24 11:11 melvin-bot[bot]

Triggered auto assignment to Design team member for new feature review - @dannymcclain (NewFeature)

melvin-bot[bot] avatar Nov 12 '24 11:11 melvin-bot[bot]

@JmillsExpensify will datePosted: accept both before and after values like our regular date filter?

dannymcclain avatar Nov 12 '24 14:11 dannymcclain

Hmm, good question. We should probably take that approach for consistency.

JmillsExpensify avatar Nov 18 '24 21:11 JmillsExpensify

@JmillsExpensify should this filter name also follow the other date-based filter and be just posted: instead of datePosted:? We are using submitted, approved, etc for the other ones

luacmartins avatar Nov 21 '24 00:11 luacmartins

Ah, great call. I like that, updated the OP.

JmillsExpensify avatar Nov 21 '24 10:11 JmillsExpensify

Also see this comment from a related issue. We're getting to the point that we have a ton of filters that do different things, so it's a good time to section out the options into thematic sections.

JmillsExpensify avatar Nov 22 '24 07:11 JmillsExpensify

Any updates of note here?

dannymcclain avatar Dec 02 '24 14:12 dannymcclain

Ah yes, PR is in review - https://github.com/Expensify/App/pull/52938

luacmartins avatar Dec 02 '24 17:12 luacmartins

PR in staging

luacmartins avatar Dec 10 '24 19:12 luacmartins

@luacmartins I don't see Posted in that list of filters—am I just missing something?

CleanShot 2024-12-10 at 13 46 30@2x

dannymcclain avatar Dec 10 '24 19:12 dannymcclain

@dannymcclain do you have a card added to the account? The card and posted filters only show up when you have a card associated with that account

luacmartins avatar Dec 10 '24 19:12 luacmartins

Ah gotcha—I was just going by what I was seeing in the videos in the PR. Sounds like it's taken care of though! Thanks!

dannymcclain avatar Dec 10 '24 20:12 dannymcclain

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

melvin-bot[bot] avatar Dec 10 '24 22:12 melvin-bot[bot]

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

  • https://github.com/Expensify/App/pull/52938

If no regressions arise, payment will be issued on 2024-12-17. :confetti_ball:

melvin-bot[bot] avatar Dec 10 '24 22:12 melvin-bot[bot]

BugZero Checklist: The PR adding this new feature has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

  • [ ] [@luacmartins] Please propose regression test steps to ensure the new feature will work correctly on production in further releases.
  • [ ] [@JmillsExpensify] Link the GH issue for creating/updating the regression test once above steps have been agreed upon.

melvin-bot[bot] avatar Dec 10 '24 22:12 melvin-bot[bot]

Skipping the payment summary for this issue since all the assignees are employees or vendors. If this is incorrect, please manually add the payment summary SO.

melvin-bot[bot] avatar Dec 17 '24 09:12 melvin-bot[bot]

@JmillsExpensify, @dannymcclain, @luacmartins Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

melvin-bot[bot] avatar Dec 20 '24 09:12 melvin-bot[bot]

@JmillsExpensify, @dannymcclain, @luacmartins Still overdue 6 days?! Let's take care of this!

melvin-bot[bot] avatar Dec 24 '24 09:12 melvin-bot[bot]

@JmillsExpensify, @dannymcclain, @luacmartins 8 days overdue is a lot. Should this be a Weekly issue? If so, feel free to change it!

melvin-bot[bot] avatar Dec 26 '24 09:12 melvin-bot[bot]

@JmillsExpensify, @dannymcclain, @luacmartins 12 days overdue. Walking. Toward. The. Light...

melvin-bot[bot] avatar Dec 30 '24 09:12 melvin-bot[bot]

Payment was handled via https://github.com/Expensify/App/issues/52262. Closing.

luacmartins avatar Dec 30 '24 15:12 luacmartins