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

Pricing: hide "Basic" plan and show "Enterprise" #316

Merged
merged 1 commit into from
Jul 30, 2024

Conversation

humitos
Copy link
Member

@humitos humitos commented Jul 24, 2024

I copied the text and features from http://localhost:8080/pricing/enterprise/

Screenshot_2024-07-24_16-21-26

Closes #289


📚 Documentation preview 📚: https://readthedocs-about--316.org.readthedocs.build/

I copied the text and features from http://localhost:8080/pricing/enterprise/

Closes #289
@humitos humitos requested a review from a team as a code owner July 24, 2024 14:22
@humitos humitos requested a review from ericholscher July 24, 2024 14:22
Copy link
Member

@ericholscher ericholscher left a comment

Choose a reason for hiding this comment

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

This looks like a great start. I think we should still probably mention the Basic plans price somewhere on the page, since this is a pricing page after all. Not sure the best UX for it, but I think its at least worth having as a small callout, perhaps at the bottom? If it's a pain, I'm also fine shipping this without changes.

@humitos
Copy link
Member Author

humitos commented Jul 24, 2024

Yeah. I thought the same but I didn't find an obvious easy to mention it.

I kept the comparison table that mention Basic, tho, so the features are differentiated at least.

Do you have an idea for this?

Since we don't want to promote it, I think that not having a big card is fine. People really wanting to pay 50$ a month will figure it out from the subscription page anyways 🤷‍♂️

@ericholscher
Copy link
Member

Perhaps just put the pricing in the comparison table? 🤔

@humitos
Copy link
Member Author

humitos commented Jul 25, 2024

If it's a pain, I'm also fine shipping this without changes.

I will probably take this sentence 😉 and move forward with this PR. We can open an new issue to deal with Basic price in the next immediate iteration (next Sprint?)

@humitos humitos merged commit 030c950 into main Jul 30, 2024
5 checks passed
@humitos humitos deleted the humitos/enterprise-pricing branch July 30, 2024 11:11
@ericholscher
Copy link
Member

Follow up here: #321

humitos added a commit that referenced this pull request Sep 12, 2024
Since we removed the Basic plan in #316, the phrase
"Includes all Basic plan features" doesn't make sense anymore.

I removed that phrase and write down the features included instead.
humitos added a commit that referenced this pull request Sep 12, 2024
Since we removed the Basic plan in #316, the phrase
"Includes all Basic plan features" doesn't make sense anymore.

I removed that phrase and write down the features included instead.
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.

Pricing: remove "Basic plan" and add "Enterprise plan"
2 participants