Skip to content

Point the version switcher to a name listed in switcher.json #23175

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

Merged
merged 1 commit into from
Jun 2, 2022

Conversation

timhoffm
Copy link
Member

@timhoffm timhoffm commented Jun 1, 2022

The switcher default value must match a version given in switcher.json. We either want to go to stable or devdocs.

Extracted from #23142, because I don't want to mix this fix with position and styling discussions.

Additional question to @QuLogic: Local and CI versions will still not work after the fix, because they cannot access switcher.json:

Access to XMLHttpRequest at 'https://matplotlib.org/devdocs/_static/switcher.json' from origin 'https://output.circle-artifacts.com' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.

Is it possible to set Access-Control-Allow-Origin on the website (https://developer.mozilla.org/en-US/docs/Web/HTTP/CORS/Errors/CORSMissingAllowOrigin)?

@timhoffm timhoffm added the Documentation: website layout/behavior/styling changes label Jun 1, 2022
@timhoffm timhoffm added this to the v3.6.0 milestone Jun 1, 2022
@jklymak jklymak merged commit e60b757 into matplotlib:main Jun 2, 2022
@timhoffm timhoffm deleted the version-switcher-default branch June 2, 2022 18:14
@QuLogic
Copy link
Member

QuLogic commented Jun 15, 2022

Is it possible to set Access-Control-Allow-Origin on the website (https://developer.mozilla.org/en-US/docs/Web/HTTP/CORS/Errors/CORSMissingAllowOrigin)?

I put in a temporary exception for this file, and it appears to be working for the CI docs here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation: website layout/behavior/styling changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants