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.

Sign up for GitHub

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

Jump to bottom

[Task]: Do not start the search if no search term is provided. #4721

Closed
nadment opened this issue Dec 17, 2024 · 0 comments · Fixed by #4729
Closed

[Task]: Do not start the search if no search term is provided. #4721

nadment opened this issue Dec 17, 2024 · 0 comments · Fixed by #4729
Assignees
@nadment
Labels
Hop Gui P3 Nice to have task
Milestone
2.12

Comments

@nadment
Copy link
Contributor

nadment commented Dec 17, 2024

What needs to happen?

In the search perspective, do not start the search if no search term is provided.
This can be very time-consuming in a large project.

Issue Priority

Priority: 3

Issue Component

Component: Hop Gui
@github-actions github-actions bot added P3 Nice to have Hop Gui labels Dec 17, 2024
@nadment nadment self-assigned this Dec 17, 2024
nadment added a commit to nadment/hop that referenced this issue Dec 18, 2024
@nadment
Do not start the search if no search term is provided. apache#4721
1080a92
@nadment nadment linked a pull request Dec 18, 2024 that will close this issue
Do not start the search if no search term is provided. #4721 #4729
Merged
hansva added a commit that referenced this issue Dec 24, 2024
@hansva
Merge pull request #4729 from nadment/4721
d11e081 
Do not start the search if no search term is provided. #4721
@hansva hansva added this to the 2.12 milestone Dec 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nadment nadment

Labels
Hop Gui P3 Nice to have task
Projects
None yet
Milestone
2.12
Development

Successfully merging a pull request may close this issue.

Do not start the search if no search term is provided. #4721 nadment/hop
2 participants
@hansva @nadment