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

Doc search should not show deprecated results #536

Open
ammkrn opened this issue Jul 16, 2022 · 3 comments
Open

Doc search should not show deprecated results #536

ammkrn opened this issue Jul 16, 2022 · 3 comments

Comments

@ammkrn
Copy link
Contributor

ammkrn commented Jul 16, 2022

Describe the bug

The search feature in the book/docs returns misleading results about deprecated versions. If I search 'startup', the first result is Startup commands (jonathandturner), which is very outdated. The URL does indicate that this is old, but that's not part of the search result, and you have to scroll to the top of the page to see the version number (and you'd already have to be a user to know that this is not the newest version).

Thank you.

How to reproduce

Navigate to https://www.nushell.sh/ and search for startup

Expected behavior

I guess the expectation is that the search function would only draw from docs that are current relative to the latest version.

Screenshots

No response

Configuration

No response

Additional context

No response

@fdncred fdncred transferred this issue from nushell/nushell Jul 16, 2022
@fdncred
Copy link
Contributor

fdncred commented Jul 16, 2022

Transferred to docs repo, as it seems a better fit.

Excluding the deprecated commands shouldn't be that tricky as we just need to stop generating the md files for them. However, filtering out blogs may not be the best thing. I agree that they can contain outdated material but they also can contain good information. So, that's a tougher thing to deal with.

@ammkrn
Copy link
Contributor Author

ammkrn commented Jul 17, 2022

@fdncred

Thanks for taking a look. It might be more intuitive if the search bar returned results from the book when the "book" tab is active, and results from the blog when that tab is active. Currently, the search prompt just says "search docs" regardless of what tab is active and returns results from any of them, which I think is unexpected for people used to cargo docs/mdbook stuff in the rust ecosystem.

@fdncred
Copy link
Contributor

fdncred commented Jul 17, 2022

That sounds interesting. We'd probably accept a PR for that functionality.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants