search

Expensify / App

Welcome to New Expensify: a complete re-imagination of financial collaboration, centered around chat. Help us build the next generation of Expensify by sharing feedback and contributing to the code.
https://new.expensify.com
MIT License
3.32k stars 2.76k forks source link

[Search v2.1][Auth] Deprecate old syntax code #45030

Closed luacmartins closed 2 weeks ago

luacmartins commented 2 months ago

Implement this section of the design doc.

Holding on:

melvin-bot[bot] commented 2 months ago

Triggered auto assignment to @zanyrenney (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

melvin-bot[bot] commented 2 months ago

Triggered auto assignment to @JmillsExpensify (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 commented 1 month ago

Still on hold

JmillsExpensify commented 1 month ago

Can we remove the hold now?

luacmartins commented 1 month ago

Not yet, the holding PR is still in draft. Also, this is just clean up work, so I think it's lower priority than the remaining filters.

zanyrenney commented 1 month ago

Are we able to remove the hold yet? I can't see the linked PR.

zanyrenney commented 1 month ago

cc @luacmartins

luacmartins commented 1 month ago

Not yet

zanyrenney commented 3 weeks ago

Looks like the PR is merged, good to remove the hold @luacmartins ?

luacmartins commented 3 weeks ago

Yea, hold removed. Thanks!

luacmartins commented 2 weeks ago

@zanyrenney this is all internal work, so I'll unassign you.

luacmartins commented 2 weeks ago

This is done!