Update Gold Member marketing #4063
Conversation
|
I don't think this lint error is related to these changes... |
| {% endblocktrans %} | ||
| </p> | ||
|
|
||
| {% if request.user.gold.count or request.user.goldonce.count %} |
There was a problem hiding this comment.
I think you can use .exists here instead of .count
|
Updated based on feedback. |
docs/ethical-advertising.rst
Outdated
| @@ -135,6 +135,9 @@ Opting Out | |||
|
|
|||
| We have added multiple ways to opt out of the advertising on Read the Docs. | |||
|
|
|||
| Users can go ad-free for as long as they are logged-in | |||
| by becoming a `Gold Member to Read the Docs <https://readthedocs.org/accounts/gold/>`_. | |||
There was a problem hiding this comment.
For some reason the to feels weird here. I guess it implies a subscription? That makes sense, just reads weird to me and I'm not 100% sure why.
There was a problem hiding this comment.
I think of is better than to.
docs/support.rst
Outdated
| nobody is paid to handle readthedocs.org support. | ||
| We are hoping to bring in enough money with our `Gold`_ program to change that, | ||
| so please sign up if you are able. | ||
| Read the Docs is a community- and ad-supported site. |
There was a problem hiding this comment.
community- is a bit odd here. I think maybe reword it:
Read the Docs is supported by community contributions and advertising.
| @@ -59,6 +59,9 @@ <h2>Read the Docs Gold</h2> | |||
| we suggest giving at least $20/month to help cover our support and operations costs. | |||
There was a problem hiding this comment.
I feel like these other parts of the page are just blabbing in an awkward way also. I'm almost tempted to just remove them?
There was a problem hiding this comment.
I'll give it a read and see if there's anything that should be removed. I tried to remove anything inaccurate already.
There was a problem hiding this comment.
I do think everything around domains can be deleted once we get Lets Encrypt support. I don't quite think I'll delete that yet.
There was a problem hiding this comment.
Why would LE support change our infra cost for domains? If anything, it would increase the overhead we have for each domain we serve, managing the certs, etc.
readthedocs/profiles/views.py
Outdated
| @@ -279,3 +280,34 @@ def profile_detail( | |||
|
|
|||
| context.update({'profile': profile_obj}) | |||
| return render(request, template_name, context=context) | |||
|
|
|||
|
|
|||
| def account_advertising(request): | |||
There was a problem hiding this comment.
Does this need a login_requires decorator? Guessing it will blow up with a non-logged in user otherwise?
It would be fully automated at that point. There would be no marginal cost to additional domains once the work on LE is done. Currently there's a manual step there. |
|
What manual step? We don't support SSL at all for them currently. The idea was for people using CNAME's to pay more, because they were getting more value and we were getting less brand awareness. |
|
Ahhh. I thought we did support it if they gave us the cert. I guess not. |
Changes
In general, the goal of this PR is to update some of our marketing and documentation to better reflect the reality at Read the Docs.
Here's an overview of the changes:
settings.USE_PROMOS = False, remove the links to manage advertising.One change I did not do but am considering is to allow staying logged-in longer than 2 weeks. This becomes somewhat important when users are becoming a Gold Member to skip ads. Having people log in every two weeks seems suboptimal.