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

[HOLD for Payment 2024-09-19][Search v2.1] Set currently applied filters on Advanced Filters Page upon clicking Filters Button #48156

Closed
289Adam289 opened this issue Aug 28, 2024 · 17 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2

Comments

@289Adam289
Copy link
Contributor

289Adam289 commented Aug 28, 2024

Coming from here

Problem

  • When clicking on a Filter Button currently applied filters are not maintained
  • Type and Status filters are not accessible in Advanced filters Page

Solution

  • Store filters present in url upon clicking Filters Button

cc @luacmartins @Kicu

Issue OwnerCurrent Issue Owner: @RachCHopkins
@luacmartins luacmartins added the Bug Something is broken. Auto assigns a BugZero manager. label Aug 28, 2024
Copy link

melvin-bot bot commented Aug 28, 2024

Triggered auto assignment to @RachCHopkins (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 melvin-bot bot added the Daily KSv2 label Aug 28, 2024
@melvin-bot melvin-bot bot added the Overdue label Aug 30, 2024
@rayane-djouah
Copy link
Contributor

Not overdue, PR in progress

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

PR merged

@rayane-djouah
Copy link
Contributor

Note

The production deploy automation failed: This should be on [HOLD for Payment 2024-09-19] according to #49025 production deploy checklist, confirmed in #48312 (comment)

@luacmartins luacmartins changed the title [Search v2.1] Set currently applied filters on Advanced Filters Page upon clicking Filters Button [HOLD for Payment 2024-09-19][Search v2.1] Set currently applied filters on Advanced Filters Page upon clicking Filters Button Sep 17, 2024
@luacmartins luacmartins added Awaiting Payment Auto-added when associated PR is deployed to production and removed Reviewing Has a PR in review labels Sep 17, 2024
@luacmartins luacmartins moved this from Polish to Done in [#whatsnext] #expense Sep 17, 2024
@luacmartins luacmartins added Daily KSv2 and removed Weekly KSv2 labels Sep 17, 2024
@melvin-bot melvin-bot bot removed the Overdue label Sep 17, 2024
@RachCHopkins
Copy link
Contributor

RachCHopkins commented Sep 18, 2024

@rayane-djouah can I just confirm this is $250 each for you and @289Adam289, and there are no regression tests to add?

If so, I'll whip up Offers in Upwork tomorrow.

Edit: Looks like @289Adam289 is SWM, so just an offer for @rayane-djouah

@luacmartins
Copy link
Contributor

@289Adam289 is from SWM and will get paid separately. No regressions tests to add this time, we'll handle it as part of the project wrap up.

@RachCHopkins
Copy link
Contributor

Does that mean it's $250 for @rayane-djouah then @luacmartins ?

@luacmartins
Copy link
Contributor

Yes

@RachCHopkins
Copy link
Contributor

@rayane-djouah I've just sent you the offer!

@melvin-bot melvin-bot bot added the Overdue label Sep 23, 2024
Copy link

melvin-bot bot commented Sep 23, 2024

@luacmartins, @RachCHopkins, @rayane-djouah, @289Adam289 Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

@rayane-djouah

This comment has been minimized.

@RachCHopkins
Copy link
Contributor

@luacmartins can you please just give that a 👍🏼 before I pay it? i.e. that the offer/payment total should be $500?

@melvin-bot melvin-bot bot removed the Overdue label Sep 24, 2024
@luacmartins
Copy link
Contributor

@rayane-djouah #47918 is linked to #47497 and payment for that issue was already processed. Did you mean to link a different PR above?

@rayane-djouah
Copy link
Contributor

rayane-djouah commented Sep 24, 2024

Oh my bad 🤦‍♂️ I apologize for the oversight. I mistakenly thought that #47918 was related to this issue based on the issue references above.

Screenshot Screenshot 2024-09-24 at 3 16 21 PM

I should have double-checked. Please ignore my previous comment; the payment should be a standard $250.

@luacmartins Thank you for correcting me!

@luacmartins
Copy link
Contributor

Np! Just wanted to check in case you had an incorrect link. Thanks for confirming. @RachCHopkins seems like $250 it is.

@melvin-bot melvin-bot bot added Daily KSv2 and removed Daily KSv2 labels Sep 24, 2024
@RachCHopkins
Copy link
Contributor

Thanks team! Will handle that now.

Payment Summary:

Contributor: @289Adam289 - No external payment
Contributor+: @rayane-djouah to be paid $250 via Upwork
Upwork job here

@RachCHopkins
Copy link
Contributor

Contributor has been paid, the contract has been completed, and the Upwork post has been closed.

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 Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2
Projects
Status: Done
Development

No branches or pull requests

4 participants