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-10-08][$250] [Search v2.1] Don't clear the workspace switcher when changing "default filters" #49637

Closed
JmillsExpensify opened this issue Sep 24, 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 External Added to denote the issue can be worked on by a contributor

Comments

@JmillsExpensify
Copy link

JmillsExpensify commented Sep 24, 2024

Problem

We've created a pretty confusing pattern for a fairly common flow, which goes like this:

  • You're filtered on a workspace on Inbox
  • You move to the Search page. The workspace selection is retained
  • You choose a different default filter on the Search page. The workspace selection is cleared.

While the theory of clearing the workspace makes sense on paper, it's just another filter, this experience ends up feeling confusing in practice, because it's possible to go from a completely different and not clear the filter. Yet, if you're on the same page, the filter is cleared.

Solution

Let's stop clearing the workspace switcher when you select different default filters on the Search page

cc @luacmartins

Issue OwnerCurrent Issue Owner: @
Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~021848376744190303271
  • Upwork Job ID: 1848376744190303271
  • Last Price Increase: 2024-10-21
Issue OwnerCurrent Issue Owner: @JmillsExpensify
@JmillsExpensify JmillsExpensify added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Sep 24, 2024
@JmillsExpensify JmillsExpensify self-assigned this Sep 24, 2024
Copy link

melvin-bot bot commented Sep 24, 2024

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

@JmillsExpensify
Copy link
Author

Carlos - should we make this external or internal?

@luacmartins
Copy link
Contributor

@Kicu wanna take this one since you worked on the policy filter?

@luacmartins luacmartins self-assigned this Sep 24, 2024
@Kicu
Copy link
Contributor

Kicu commented Sep 24, 2024

Yes please assign me, I might have some extra questions later to make sure I understand this correctly. Every issue that touches navigation requires some extra care 😅

@Kicu
Copy link
Contributor

Kicu commented Sep 24, 2024

@JmillsExpensify there is a small typo here (missing word) which actually makes the description confusing for me 😅

this experience ends up feeling confusing in practice, because it's possible to go from a completely different >>WHAT<< and not clear the filter. Yet, if you're on the same page, the filter is cleared.

Do you mind updating your message?

My understanding is this:

  1. You are on Search page and there is a policy chosen via Workspace Switcher
  2. You click on any button on the LHN (Expenses, Trips etc) OR you click on any of the status buttons (All, Drafts, Paid etc)
  3. (currently) new search query will be displayed AND the policy in Workspace Switcher will be reset to All.

The new expectation is that in point 3. policy should not be reset to All.
Correct?

@luacmartins
Copy link
Contributor

Your understanding is correct. (@JmillsExpensify please correct me if I'm wrong)

@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 and removed Daily KSv2 labels Sep 25, 2024
@Kicu
Copy link
Contributor

Kicu commented Sep 25, 2024

@luacmartins @JmillsExpensify PR ready here: #49694

@luacmartins
Copy link
Contributor

Nice! Thanks for the quick turnaround! I'll take a look once @rushatgabhane is done reviewing it

@luacmartins luacmartins changed the title Don't clear the workspace switcher when changing "default filters" [Search v2.1]Don't clear the workspace switcher when changing "default filters" Sep 25, 2024
@luacmartins luacmartins changed the title [Search v2.1]Don't clear the workspace switcher when changing "default filters" [Search v2.1] Don't clear the workspace switcher when changing "default filters" Sep 25, 2024
@melvin-bot melvin-bot bot added Monthly KSv2 and removed Weekly KSv2 labels Oct 18, 2024
Copy link

melvin-bot bot commented Oct 18, 2024

This issue has not been updated in over 15 days. @JmillsExpensify, @Kicu, @luacmartins, @rushatgabhane eroding to Monthly issue.

P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do!

@Kicu
Copy link
Contributor

Kicu commented Oct 21, 2024

Issue was solved.

@luacmartins luacmartins changed the title [Search v2.1] Don't clear the workspace switcher when changing "default filters" [HOLD for payment 2024-10-08][Search v2.1] Don't clear the workspace switcher when changing "default filters" Oct 21, 2024
@luacmartins luacmartins added Awaiting Payment Auto-added when associated PR is deployed to production Daily KSv2 and removed Reviewing Has a PR in review Monthly KSv2 labels Oct 21, 2024
@luacmartins luacmartins added the External Added to denote the issue can be worked on by a contributor label Oct 21, 2024
@melvin-bot melvin-bot bot changed the title [HOLD for payment 2024-10-08][Search v2.1] Don't clear the workspace switcher when changing "default filters" [$250] [HOLD for payment 2024-10-08][Search v2.1] Don't clear the workspace switcher when changing "default filters" Oct 21, 2024
Copy link

melvin-bot bot commented Oct 21, 2024

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

@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Oct 21, 2024
Copy link

melvin-bot bot commented Oct 21, 2024

Current assignee @rushatgabhane is eligible for the External assigner, not assigning anyone new.

@luacmartins luacmartins removed the Help Wanted Apply this label when an issue is open to proposals by contributors label Oct 21, 2024
@luacmartins luacmartins changed the title [$250] [HOLD for payment 2024-10-08][Search v2.1] Don't clear the workspace switcher when changing "default filters" [HOLD for payment 2024-10-08][$250] [Search v2.1] Don't clear the workspace switcher when changing "default filters" Oct 21, 2024
@melvin-bot melvin-bot bot added the Overdue label Oct 23, 2024
@rushatgabhane
Copy link
Member

@JmillsExpensify could you please attach payment summary 🙇

@JmillsExpensify
Copy link
Author

Payment summary: $250 to @rushatgabhane for PR review and testing.

@melvin-bot melvin-bot bot removed the Overdue label Oct 24, 2024
@JmillsExpensify
Copy link
Author

Payment is via NewDot, so I'm going to close this one.

Copy link

melvin-bot bot commented Oct 24, 2024

@JmillsExpensify @luacmartins Be sure to fill out the Contact List!

@garrettmknight
Copy link
Contributor

$250 approved for @rushatgabhane

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 External Added to denote the issue can be worked on by a contributor
Projects
Archived in project
Development

No branches or pull requests

5 participants