-
Notifications
You must be signed in to change notification settings - Fork 331
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
the theme does not provide placement for the ethical ads of readthedoc #345
Comments
A lot of sites using this theme self-host, so i guess this hasn't come up. However, we do host this repo's example site there, so it would be reasonable to give something back. The quickest win would be to add an example of the markup needed in a custom template... this would likely have legs beyond hard-coding something for that particular host and that particular ad provider. Would you happen to know what we need to do? |
no but I will dig ;-) |
I think that we need to add this snippet of HTML somewhere on the page: https://github.com/pradyunsg/furo/blob/main/src/furo/theme/furo/sidebar/ethical-ads.html |
Ah yes! We added it in #355 🎉 |
I know we all hate ads, but at the same time we all love the possibility to use the readTheDoc server free of charge to serve our documentations.
When I run my documentation in readthedoc I don't see the ethical ads (usually digital ocean for me).
The js console confirms :
in ethicalads.min.js:201
Will you consider adding it ?
The text was updated successfully, but these errors were encountered: