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

Normal search stops working for old versions when add-ons are enabled #219

Closed
astrojuanlu opened this issue Dec 14, 2023 · 6 comments
Closed
Assignees

Comments

@astrojuanlu
Copy link

See for example: https://docs.kedro.org/en/0.17.1/search.html?q=partitioned&check_keywords=yes&area=default

image

The search from the flyout displays the search-as-you-type modal and everything works. But the Sphinx search box (which doesn't trigger the relevant event, because it hasn't been modified) takes the user to an empty page.

cc @inigohidalgo @rashidakanchwala

@humitos
Copy link
Member

humitos commented Dec 14, 2023

Hi @astrojuanlu! Can you try rebuilding the old version please and letting me know if it solves the issue?

(I'm from the phone 📱)

@astrojuanlu
Copy link
Author

I tried rebuilding but it failed https://beta.readthedocs.org/projects/kedro/builds/22854520/ because of https://blog.readthedocs.com/use-build-os-config/

We don't follow the "versions as branches" approach so I don't think we'll rebuild very old versions of the docs. But if it helps troubleshooting, I can try to do an experiment with some other project (I'd need some time to think which).

@humitos
Copy link
Member

humitos commented Dec 14, 2023

Hrm... This is an interesting scenario 🤔 . The fix deployed in #213 works only when rebuilding the documentation, so it won't have any effect on those old versions that can't be re-built 😢 . At least, it stop the issue to keep propagating on new projects, tho.

I will need to think more about this case because I don't have a clear solution in mind about how we can fix this in older versions that are static and don't allow rebuilding. However, I'm not sure if there is a generic solution that works for all the cases... 😞 I need to think more 🧠

@humitos humitos closed this as completed Dec 17, 2023
@humitos humitos reopened this Dec 17, 2023
@humitos
Copy link
Member

humitos commented Dec 17, 2023

I found a way to fix this and I opened a PR internally. Hopefully it can be reviewed in the following days and deployed. I will let you know when it's deployed so you can confirm the issue is solved 👍🏼

@humitos
Copy link
Member

humitos commented Dec 18, 2023

I deployed a fix for this. @astrojuanlu can you please confirm that's working as you expected?

Note that this starts working on version 0.18.4, because it seems the older versions were built with older Sphinx versions where the search was broken on their side it seems --I didn't find anything related to Read the Docs at least 1:

On 0.18.4 works fine for me: https://docs.kedro.org/en/0.18.4/search.html?q=partitioned&check_keywords=yes&area=default

Footnotes

  1. if you notice an issue in our side I'm happy to trace it down a little more and try to find a solution.

@humitos humitos self-assigned this Dec 18, 2023
@github-project-automation github-project-automation bot moved this to Planned in 📍Roadmap Dec 18, 2023
@humitos humitos moved this from Planned to Needs review in 📍Roadmap Dec 18, 2023
@humitos humitos closed this as completed Dec 18, 2023
@github-project-automation github-project-automation bot moved this from Needs review to Done in 📍Roadmap Dec 18, 2023
@astrojuanlu
Copy link
Author

This is great, thanks @humitos! Yeah nothing can be done with old Sphinx versions 😅 but at least we're in a better place now

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

No branches or pull requests

2 participants