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

logo link no longer points to www.pymc.io #6054

Closed
OriolAbril opened this issue Aug 17, 2022 · 7 comments
Closed

logo link no longer points to www.pymc.io #6054

OriolAbril opened this issue Aug 17, 2022 · 7 comments

Comments

@OriolAbril
Copy link
Member

we use logo_link to make clicking on the logo go to www.pymc.io instead of the build based homepage (learn for versioned docs and gallery for pymc-examples). This no longer works with the new version of the theme, it should now use a different way to set this as explained in https://pydata-sphinx-theme.readthedocs.io/en/latest/user_guide/branding.html#customize-logo-link. The same thing probably happens over at pymc-examples but I haven't checked nor opened an issue (at least so far). Ideally fixing this would mean checking pymc-examples and making 1-2 prs on both repos, otherwise, please open an issue on pymc-examples to track this

@rowan-schaefer
Copy link
Contributor

I can have a go at this!

@OriolAbril
Copy link
Member Author

Thanks!

@hdnl
Copy link
Contributor

hdnl commented Oct 4, 2022

I changed

"logo_link": "https://www.pymc.io",

(line 259 from https://github.com/pymc-devs/pymc/blob/f3ac08b96641a418569f8423a68c44e22cfc483e/docs/source/conf.py)
to

    "logo": {
        "link": "https://pymc.io",
        },

which didn't seem to work.

The logo from my generated HTML point to http://[::]:8000/https%3A//www.pymc.io.html after this code fix. I attempted variants of https://pymc.io for the purposes of testing, but all of them produced a similar, relative URL-link (as opposed to an absolute-link). I'm unsure why this is the case. Could this be from Sphinx's end?

@OriolAbril
Copy link
Member Author

There is a bug in the latest release of the sphinx theme we use (0.10.1) which has already been fixed and release 0.11.0 will probably be released in the coming days (multiple release candidates have been released already). I would recommend already opening a draft PR and triggering a rebuild once the next release for the theme is out. The fix should happen in both pymc and pymc-examples repo.

Note: https://pymc.io is not a valid link, not sure why but the www part is necessary.

@OriolAbril
Copy link
Member Author

I have fixed the issue for pymc-examples in #6054. Same thing still needs to happen in pymc repo though. @hdnl or @rowangayleschaefer, will any of you be able to do that?

@hdnl
Copy link
Contributor

hdnl commented Oct 10, 2022

I can -- give me about ~24 hours.

@michaelosthege
Copy link
Member

Closed by #6209

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

No branches or pull requests

4 participants