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

Feedback and version chooser missing #536

Closed
amotl opened this issue Oct 8, 2024 · 6 comments
Closed

Feedback and version chooser missing #536

amotl opened this issue Oct 8, 2024 · 6 comments
Assignees
Labels
bug Something isn't working

Comments

@amotl
Copy link
Member

amotl commented Oct 8, 2024

Problem

@matriv reported:

Hi! For the CrateDB ref docs, I don't see the drop down to select versions: https://cratedb.com/docs/crate/reference/en/latest/
(The arrow is there, but there are 0 options).

@msbt reported:

Guides and reference are the ones that aren't working properly.

@seut reported:

Also https://crate-jdbc.readthedocs.io/en/latest/ lost it’s version chooser. Old existing builds still has them. As old builds (they were build quite some time ago) work and only newer once which probably include a new version of the crate-docs-theme, this highly indicates that a recent crate-theme-version broke the version chooser.

Possible root cause

@msbt found this:

Related blogpost (which pretty much correlates with the day the problems began): https://about.readthedocs.com/blog/2024/07/addons-by-default/.

We are announcing the introduction of Read the Docs Addons to all the projects by default starting on October 7, 2024. Read the Docs Addons is the new modular implementation of all the features Read the Docs adds on top of your documentation: analytics, flyout menu, notifications and search, among others.

So, rolling back any release, as suggested, will probably not help, because the problem has been caused by an update at RTD.

Outlook

[...] try installing sphinx-build-compatibility extension in your project to mitigate them.

@amotl
Copy link
Member Author

amotl commented Oct 8, 2024

Root Cause

Indeed, it is about https://about.readthedocs.com/blog/2024/07/addons-by-default/.

Solution

That patch fixes most of the issues already.

Thanks for your excellent support, @msbt!

@amotl
Copy link
Member Author

amotl commented Oct 8, 2024

Backlog

404 on readthedocs-addons.js

@msbt said:

Maybe something changed on RTD, apparently it tries to load this file https://cratedb.com/_/static/javascript/readthedocs-addons.js which obviously is 404, actual file is localted here: https://cratedb.com/docs/crate/reference/_/static/javascript/readthedocs-addons.js

Resources from other extensions will get rendered like this in HTML:

_static/doctools.js
_static/copybutton.js
_static/design-elements.bbdccc18c4abea9397628f9fea3d48c2.js

Unfortunately, RTD injects an URL like this into the page:

/_/static/javascript/readthedocs-addons.js

This does not harmonize well with our custom domain setup.

Problem with item contents in version chooser

@amotl
Copy link
Member Author

amotl commented Oct 8, 2024

Works

Both feedback tools and version chooser are present.

Does not work

Feedback tools are present, but version chooser is missing.

@amotl
Copy link
Member Author

amotl commented Oct 11, 2024

We guess this has converged well, so let's close the ticket. Cheers, and thanks again @msbt.

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

No branches or pull requests

2 participants