Skip to content
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

[PAID][Search v2.1][Web-E] Deprecate old Search params #45029

Closed
1 task done
luacmartins opened this issue Jul 8, 2024 · 19 comments
Closed
1 task done

[PAID][Search v2.1][Web-E] Deprecate old Search params #45029

luacmartins opened this issue Jul 8, 2024 · 19 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Daily KSv2 Internal Requires API changes or must be handled by Expensify staff NewFeature Something to build that is a new item.

Comments

@luacmartins
Copy link
Contributor

luacmartins commented Jul 8, 2024

Implement this section of the design doc.

Holding on:

Issue OwnerCurrent Issue Owner: @strepanier03
@luacmartins luacmartins added Daily KSv2 NewFeature Something to build that is a new item. labels Jul 8, 2024
@luacmartins luacmartins self-assigned this Jul 8, 2024
Copy link

melvin-bot bot commented Jul 8, 2024

Triggered auto assignment to @slafortune (NewFeature), see https://stackoverflowteams.com/c/expensify/questions/14418#:~:text=BugZero%20process%20steps%20for%20feature%20requests for more details. Please add this Feature request to a GH project, as outlined in the SO.

@melvin-bot melvin-bot bot added Weekly KSv2 and removed Daily KSv2 labels Jul 8, 2024
@luacmartins luacmartins added Daily KSv2 Internal Requires API changes or must be handled by Expensify staff and removed Weekly KSv2 labels Jul 8, 2024
@melvin-bot melvin-bot bot added the Overdue label Jul 11, 2024
@luacmartins luacmartins added Weekly KSv2 and removed Daily KSv2 labels Jul 11, 2024
@melvin-bot melvin-bot bot removed the Overdue label Jul 11, 2024
@melvin-bot melvin-bot bot added the Overdue label Jul 22, 2024
@luacmartins
Copy link
Contributor Author

Still on hold

@melvin-bot melvin-bot bot removed the Overdue label Jul 23, 2024
@JmillsExpensify
Copy link

We're good to review the hold, yeah?

@luacmartins luacmartins changed the title [HOLD 45027][Search v2][Web-E] Deprecate old Search params [HOLD 46481][Search v2][Web-E] Deprecate old Search params Jul 30, 2024
@luacmartins
Copy link
Contributor Author

Not yet, the holding PR is still in draft

@luacmartins luacmartins changed the title [HOLD 46481][Search v2][Web-E] Deprecate old Search params [HOLD 46481][Search v2.1][Web-E] Deprecate old Search params Jul 31, 2024
@melvin-bot melvin-bot bot added the Overdue label Aug 8, 2024
@slafortune
Copy link
Contributor

Just in case - adding another BZ - I'll be out until 8/21 and will check in on this then.

@melvin-bot melvin-bot bot removed the Overdue label Aug 13, 2024
@slafortune slafortune removed their assignment Aug 13, 2024
@slafortune slafortune added NewFeature Something to build that is a new item. and removed NewFeature Something to build that is a new item. labels Aug 13, 2024
Copy link

melvin-bot bot commented Aug 13, 2024

Triggered auto assignment to @Christinadobrzyn (NewFeature), see https://stackoverflowteams.com/c/expensify/questions/14418#:~:text=BugZero%20process%20steps%20for%20feature%20requests for more details. Please add this Feature request to a GH project, as outlined in the SO.

@slafortune slafortune self-assigned this Aug 13, 2024
@luacmartins
Copy link
Contributor Author

That PR is not related to this issue. I still haven't worked on this one.

@luacmartins
Copy link
Contributor Author

Since this is all internal, I'll unassign you from this issue.

@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 and removed Daily KSv2 labels Aug 20, 2024
@brunovjk
Copy link
Contributor

brunovjk commented Sep 3, 2024

@luacmartins I reviewed the PR, which has now been in production for a week. However, I wasn't assigned to the issue or involved in the proposal review. Should I be assigned to this issue for compensation, or given the simple nature of the PR, can we close the issue and move on? Thank you.

@luacmartins luacmartins added NewFeature Something to build that is a new item. Awaiting Payment Auto-added when associated PR is deployed to production and removed NewFeature Something to build that is a new item. Reviewing Has a PR in review labels Sep 3, 2024
Copy link

melvin-bot bot commented Sep 3, 2024

Triggered auto assignment to @strepanier03 (NewFeature), see https://stackoverflowteams.com/c/expensify/questions/14418#:~:text=BugZero%20process%20steps%20for%20feature%20requests for more details. Please add this Feature request to a GH project, as outlined in the SO.

@luacmartins luacmartins changed the title [Search v2.1][Web-E] Deprecate old Search params [HOLD for payment 2024-09-04][Search v2.1][Web-E] Deprecate old Search params Sep 3, 2024
@luacmartins luacmartins added Daily KSv2 and removed Weekly KSv2 labels Sep 3, 2024
@luacmartins
Copy link
Contributor Author

@brunovjk I updated the issue to make sure we process payment for the PR review. Thanks for raising this!

@brunovjk
Copy link
Contributor

brunovjk commented Sep 3, 2024

Thank you 🙏

@strepanier03
Copy link
Contributor

Sent an offer to @brunovjk - I'll check tomorrow to issue payment.

@strepanier03
Copy link
Contributor

strepanier03 commented Sep 3, 2024

Payment summary

@brunovjk
Copy link
Contributor

brunovjk commented Sep 4, 2024

Regression Test Proposal

  • Open the search page.
  • Select "Transactions."
  • Open your browser console.
  • Click on "Download."
  • Verify that the API request to ExportSearchToCSV includes jsonQuery in the payload.

Do we agree? 👍 or 👎

@luacmartins
Copy link
Contributor Author

No need to add the steps above, we're covering them as part of the project wrap up.

@strepanier03
Copy link
Contributor

Thanks @luacmartins, was just going to close the GH and saw you took care of that too, appreciate it!

@strepanier03
Copy link
Contributor

All right, I've paid and closed the contract and this is ready to be closed as well. Thanks everyone!

@strepanier03 strepanier03 changed the title [HOLD for payment 2024-09-04][Search v2.1][Web-E] Deprecate old Search params [PAID][Search v2.1][Web-E] Deprecate old Search params Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Daily KSv2 Internal Requires API changes or must be handled by Expensify staff NewFeature Something to build that is a new item.
Projects
Status: Done
Development

No branches or pull requests

6 participants