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

Enable ads on the readthedocs mkdocs theme #3922

Merged
merged 1 commit into from May 16, 2018

Conversation

Projects
None yet
3 participants
@davidfischer
Contributor

davidfischer commented Apr 7, 2018

This PR enables ads on the readthedocs theme for projects using Markdown.

Before merging, we should probably notify users of the mkdocs theme.

NOTE: #3920 should probably be merged first as some themes are incorrectly identified as the readthedocs theme. It isn't strictly critical as it is unlikely that the ad injection selectors will match on a custom theme and so it will likely just result in API calls that don't result in an ad being displayed.

SCREENSHOTS
screen shot 2018-04-06 at 5 12 34 pm
screen shot 2018-04-06 at 5 13 32 pm
screen shot 2018-04-06 at 5 14 53 pm

@davidfischer davidfischer requested a review from ericholscher Apr 7, 2018

@agjohnson agjohnson removed the Mkdocs label Apr 10, 2018

@agjohnson agjohnson modified the milestones: Mkdocs, 2.4 Apr 10, 2018

@ericholscher

Looks good.

@davidfischer

This comment has been minimized.

Contributor

davidfischer commented Apr 27, 2018

I emailed all users who are owners of an mkdocs project.

@davidfischer davidfischer merged commit b05c82a into rtfd:master May 16, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment