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

Add inverse search filter to workspace planner #1005

Conversation

aliraza556
Copy link
Contributor

Describe the Changes:

  • Add inverse search functionality to workspace planner allowing users to exclude tickets containing specific text from their search results

Daily Bounty: https://community.sphinx.chat/bounty/3550

Issue ticket number and link:

Evidence:

Loom:

https://www.loom.com/share/282d294a2a0a459c918830c03a6f0a14

image

image

image

Unit Test:

image

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have tested on Chrome
  • New feature (non-breaking change which adds functionality)
  • I have provided a screenshot and demo of changes in my PR

@aliraza556
Copy link
Contributor Author

@humansinstitute Please review this PR.

@humansinstitute
Copy link
Collaborator

@aliraza556 This evidence doesn't show the exclusion filter working?

i.e. when you click to exclude we don't see the filter results update?

So

  1. include "test" and
  2. exclude "test"

Should be opposite. Can you ensure this is shown? Also for robustness in the UI this would need to refresh on click of the include / exclude tag.

@aliraza556
Copy link
Contributor Author

@aliraza556 This evidence doesn't show the exclusion filter working?

i.e. when you click to exclude we don't see the filter results update?

So

  1. include "test" and
  2. exclude "test"

Should be opposite. Can you ensure this is shown? Also for robustness in the UI this would need to refresh on click of the include / exclude tag.

@humansinstitute Because backendside changes is no merged stakwork/sphinx-tribes#2517.

@humansinstitute humansinstitute merged commit d9c3b7f into stakwork:master Feb 3, 2025
64 checks passed
@humansinstitute humansinstitute added the bug Something isn't working label Feb 5, 2025
@humansinstitute
Copy link
Collaborator

humansinstitute commented Feb 5, 2025

@aliraza556 This is all merge and still doesn't work. Can you resolve the bug.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants