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 project sponsorship changes #5628

Merged
merged 2 commits into from
May 10, 2019

Conversation

davidfischer
Copy link
Contributor

  • new gold users can no longer sponsor projects
  • applies to gold users created after 2019-05-01

- new gold users can no longer sponsor projects
- applies to gold users created after 2019-05-01
@davidfischer davidfischer requested a review from a team April 24, 2019 21:55
@stsewd
Copy link
Member

stsewd commented Apr 24, 2019

Shouldn't we use a time based feature flag for this?

@davidfischer
Copy link
Contributor Author

Shouldn't we use a time based feature flag for this?

Feature flags apply to projects, correct? This applies to the user subscription.

Copy link
Member

@humitos humitos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

I left a comment about the message. Although, it's not a blocker on the PR. We can make the change later if we think it's better.

<p class="empty">
{% blocktrans trimmed %}
<strong>Note:</strong> this is a legacy feature.
New gold members cannot sponsor projects to be ad-free.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It doesn't feel good to me to expose this to new gold users.

I'm not sure what's best here, but maybe removing this page completely for new gold users is better, instead of showing them that they could have sponsored their projects.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm on the fence about whether to include it, but I do think we should probably prompt them to email us here about removing ads if they want as well. Or perhaps some other messaging?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It doesn't feel good to me to expose this to new gold users.

The page is removed for new gold users. New users will be redirected. This message is shown only to older gold users to let them know that we no longer do this.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent! 💯

@humitos
Copy link
Member

humitos commented May 1, 2019

New condition makes sense to me.

@davidfischer davidfischer merged commit 87079ef into master May 10, 2019
@delete-merged-branch delete-merged-branch bot deleted the davidfischer/gold-projects-change branch May 10, 2019 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants