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

Gold makes projects ad-free again #6073

Merged
merged 6 commits into from Aug 20, 2019

Conversation

@davidfischer
Copy link
Contributor

@davidfischer davidfischer commented Aug 14, 2019

  • Be clear about this in the docs and gold app
  • Re-enable the gold member screen to designate gold projects for all members
  • $5/mo for personal or small biz and $50 for corporate

This mostly reverts #5628. The areas where I would most like feedback are in subscription_detail.html, projects.html, and subscription_form.html.

Concerns

  • There is absolutely nothing stopping a corporate OSS project from becoming a $5/mo member and removing ads. Only the text says they are supposed to pay more.
  • While it isn't a big concern, we weren't quite as clear about gold removing all ads before although the screen totally did that in the past. It is possible that projects where we generate more than $5/mo in advertising suddenly go ad-free resulting in less revenue.

Screenshots

Becoming a gold member

Screen Shot 2019-08-13 at 10 06 22 PM

Designating gold projects

Screen Shot 2019-08-13 at 9 47 40 PM

- Be clear about this in the docs and gold app
- $5/mo for personal or small biz and $50 for corporate
@davidfischer davidfischer requested review from and ericholscher Aug 14, 2019
Copy link
Member

@ericholscher ericholscher left a comment

Looks great, we should clean up the adoption language, but otherwise I'm 👍

readthedocs/gold/templates/gold/projects.html Outdated Show resolved Hide resolved
readthedocs/gold/templates/gold/projects.html Outdated Show resolved Hide resolved
readthedocs/gold/templates/gold/projects.html Outdated Show resolved Hide resolved
readthedocs/gold/templates/gold/subscription_form.html Outdated Show resolved Hide resolved
-- you will still see :ref:`community ads <advertising/ethical-advertising:Community Ads>` --
or you can go ad-free by `becoming a Gold Member`_ or a `Supporter`_ of Read the Docs.
although you will still see :ref:`community ads <advertising/ethical-advertising:Community Ads>`.
You can go ad-free by `becoming a Gold Member`_ or a `Supporter`_ of Read the Docs.
Copy link
Member

@ericholscher ericholscher Aug 20, 2019

I think this PR should also make a publicly accessible landing page for Gold. We really need a way to link to it for people that are logged out, and it seems like the most obvious missing piece currently to making it more "understandable".

Copy link
Contributor Author

@davidfischer davidfischer Aug 20, 2019

My plan was to repurpose https://readthedocs.org/sustainability/ and go into Gold in more details there.

davidfischer and others added 5 commits Aug 20, 2019
Co-Authored-By: Eric Holscher <25510+ericholscher@users.noreply.github.com>
Co-Authored-By: Eric Holscher <25510+ericholscher@users.noreply.github.com>
Co-Authored-By: Eric Holscher <25510+ericholscher@users.noreply.github.com>
Co-Authored-By: Eric Holscher <25510+ericholscher@users.noreply.github.com>
- Remove "gold subscription"
- Membership is not capitalized unless in a title
- Don't use "gold" without member or membership
@davidfischer
Copy link
Contributor Author

@davidfischer davidfischer commented Aug 20, 2019

Ok, I updated the verbiage:

  • Always use "Gold member" or "Gold membership"
  • Remove "gold subscription" and replace with membership
  • Membership is not capitalized unless in a title
  • Don't use "Gold" without member or membership

Copy link
Member

@ericholscher ericholscher left a comment

This is great. 👍

@ericholscher ericholscher merged commit aaa3ade into master Aug 20, 2019
3 checks passed
@ericholscher ericholscher deleted the davidfischer/gold-makes-ad-free-again branch Aug 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants