-
Notifications
You must be signed in to change notification settings - Fork 611
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
module search gives help when it shouldn't #45
Labels
enhancement
New feature or request
Comments
I believe I have it set up where any time the user doesn't get search results, it reminds the user how the search interface works just in case they forgot to use regex. |
Fair enough if it is deliberate, I just didn't expect help info when it
looked like I'd used the function correctly.
…On Wed, 6 Nov 2019 at 05:22, lanmaster53 ***@***.***> wrote:
I believe I have it set up where any time the user doesn't get search
results, it reminds the user how the search interface works just in case
they forgot to use regex.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#45?email_source=notifications&email_token=AAA4SWKHLX3PPCGISWKSOPTQSJIAZA5CNFSM4JJBZDS2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDFJ2EQ#issuecomment-550149394>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAA4SWNAWZFE27FEYQQLVQLQSJIAZANCNFSM4JJBZDSQ>
.
|
You're right though. I plan to look at the logic and figure out of there is a less jarring way to achieve the same goal. Thanks for the feedback. |
Want this closing or left open as a reminder? |
Leave it open for now. Thanks. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When the search is used correctly, it gives the reply to say nothing is found, but then still dumps out the usage information:
The text was updated successfully, but these errors were encountered: