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

[$250] [Search v2.1] [App] Create Keyword filter #46027

Closed
luacmartins opened this issue Jul 23, 2024 · 25 comments
Closed

[$250] [Search v2.1] [App] Create Keyword filter #46027

luacmartins opened this issue Jul 23, 2024 · 25 comments
Assignees
Labels
External Added to denote the issue can be worked on by a contributor NewFeature Something to build that is a new item. Reviewing Has a PR in review Weekly KSv2

Comments

@luacmartins
Copy link
Contributor

luacmartins commented Jul 23, 2024

Implement this section of the design doc for the Keyword filter only!

cc @adamgrzybowski @Kicu @WojtekBoman

Issue OwnerCurrent Issue Owner: @suneox
Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01ff6d0b9a0f6c02fb
  • Upwork Job ID: 1819041159147930925
  • Last Price Increase: 2024-08-01
  • Automatic offers:
    • ikevin127 | Contributor | 103358660
@luacmartins luacmartins added Daily KSv2 NewFeature Something to build that is a new item. labels Jul 23, 2024
@luacmartins luacmartins self-assigned this Jul 23, 2024
Copy link

melvin-bot bot commented Jul 23, 2024

Triggered auto assignment to @jliexpensify (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 23, 2024
@luacmartins luacmartins added Daily KSv2 and removed Weekly KSv2 labels Jul 23, 2024
@jliexpensify
Copy link
Contributor

Hi @luacmartins, did you want me to assign the tagged people?

@luacmartins
Copy link
Contributor Author

I think they need to comment on the issue before we can assign it to them

@melvin-bot melvin-bot bot added the Overdue label Jul 26, 2024
@jliexpensify
Copy link
Contributor

Oh yeah, of course ha!

@melvin-bot melvin-bot bot removed the Overdue label Jul 28, 2024
@luacmartins luacmartins changed the title [Search v2] [App] Create Keyword filter [Search v2.1] [App] Create Keyword filter Jul 31, 2024
@melvin-bot melvin-bot bot added the Overdue label Jul 31, 2024
@luacmartins
Copy link
Contributor Author

Trying to get this assigned

@melvin-bot melvin-bot bot removed the Overdue label Jul 31, 2024
@SzymczakJ
Copy link
Contributor

Hey! I’m Jakub Szymczak from Software Mansion, an expert agency, and I’d like to work on this issue!

@luacmartins luacmartins added the External Added to denote the issue can be worked on by a contributor label Aug 1, 2024
Copy link

melvin-bot bot commented Aug 1, 2024

Job added to Upwork: https://www.upwork.com/jobs/~01ff6d0b9a0f6c02fb

@melvin-bot melvin-bot bot changed the title [Search v2.1] [App] Create Keyword filter [$250] [Search v2.1] [App] Create Keyword filter Aug 1, 2024
@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Aug 1, 2024
Copy link

melvin-bot bot commented Aug 1, 2024

Triggered auto assignment to Contributor-plus team member for initial proposal review - @suneox (External)

@luacmartins luacmartins removed the Help Wanted Apply this label when an issue is open to proposals by contributors label Aug 1, 2024
@luacmartins luacmartins assigned ikevin127 and unassigned suneox Aug 1, 2024
Copy link

melvin-bot bot commented Aug 1, 2024

📣 @ikevin127 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app!

Offer link
Upwork job
Please accept the offer and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑‍💻
Keep in mind: Code of Conduct | Contributing 📖

@jliexpensify
Copy link
Contributor

Done, thanks @289Adam289!

@289Adam289
Copy link
Contributor

@luacmartins What should keyword filter page look like? I assumed it should be similar to merchant filter but "Has keywords" description in docs implies that page should allow for multiple keywords to be entered.

@luacmartins
Copy link
Contributor Author

It should be a text input. Users can enter any value in it

@melvin-bot melvin-bot bot added the Overdue label Aug 5, 2024
@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 and removed Overdue Daily KSv2 Weekly KSv2 labels Aug 5, 2024
@ikevin127
Copy link
Contributor

ikevin127 commented Aug 14, 2024

⚠️ Production deploy automation failed here -> this should be on [HOLD for Payment 2024-08-20] according to yesterday’s production deploy confirmed in #46799 (comment) and deploy checklist.

@jliexpensify jliexpensify changed the title [$250] [Search v2.1] [App] Create Keyword filter [HOLD for Payment 2024-08-20][$250] [Search v2.1] [App] Create Keyword filter Aug 14, 2024
@jliexpensify
Copy link
Contributor

Payment Summary

Upwork job

@ikevin127
Copy link
Contributor

@jliexpensify Thank you!
Do you mind adding the Bug label as well for auto-assigned BZ team member to handle payment once due ?

@jliexpensify
Copy link
Contributor

@ikevin127 - I am BZ!

@ikevin127
Copy link
Contributor

My bad, was looking for that Bug label without checking 🤦‍♂️

@jliexpensify
Copy link
Contributor

All good, usually that label is applied but since its a NewFeature, BZ gets assigned.

@ikevin127
Copy link
Contributor

cc @jliexpensify

@jliexpensify
Copy link
Contributor

Thanks for the bump, but It's still the 19th in the US, right? 😅 that's the timezone that we go off for payments

@jliexpensify
Copy link
Contributor

jliexpensify commented Aug 20, 2024

@ikevin127 any checklist needed here?

EDIT: paid and job closed

@jliexpensify jliexpensify changed the title [HOLD for Payment 2024-08-20][$250] [Search v2.1] [App] Create Keyword filter [CHECKLIST?][$250] [Search v2.1] [App] Create Keyword filter Aug 20, 2024
@ikevin127
Copy link
Contributor

Thanks for the payment 💯
Regarding checklist / regression tests, I was told by @luacmartins in a few of the search v2.1 issues (ref) that, quote:

No, we'll add it as part of the project wrap up

@luacmartins
Copy link
Contributor Author

Correct ^

@jliexpensify
Copy link
Contributor

Awesome, all good. Closing!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
External Added to denote the issue can be worked on by a contributor NewFeature Something to build that is a new item. Reviewing Has a PR in review Weekly KSv2
Projects
Status: Done
Development

No branches or pull requests

6 participants