-
Notifications
You must be signed in to change notification settings - Fork 182
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
docs(play): include billing information in play docs #4299
Conversation
👋 🤖 🤔 Hello! Did you make your changes in all the right places? These files were changed only in docs/. You might want to duplicate these changes in versioned_docs/version-8.5/.
You may have done this intentionally, but we wanted to point it out in case you didn't. You can read more about the versioning within our docs in our documentation guidelines. |
|
||
- **Trial/Internal Plans:** You can use any cluster. | ||
- **Starter/Professional Plans:** Use a [Development cluster](http://localhost:3000/docs/next/components/console/manage-clusters/create-cluster/#development-clusters). For Professional Plans, you may need to purchase a Development cluster. | ||
- **Enterprise Plans:** Your organization must be designated as a development organization. Contact your CSM for further assistance. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a link we can use here to contact CSM? 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i don't think this is documented anywhere in the outside world
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do you mean we should leave this without a link, or change the wording here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i think leaving it like this is fine - if "CSM" is a term known to customers
|
||
The use of Play may result in additional charges depending on your organization's plan and the type of cluster you are using. To avoid incurring extra costs, please follow these guidelines based on your plan: | ||
|
||
- **Trial/Internal Plans:** You can use any cluster. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
our users do not need to know that there is something like an internal plan.
also not sure if trial plan is actually what should be taken? isn't it rather called "Camunda SaaS Trial" by now? probably best for @felix-mueller to comment on
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed the 'Internal Plan', waiting for @felix-mueller suggestion for the Trial 👀
|
||
- **Trial/Internal Plans:** You can use any cluster. | ||
- **Starter/Professional Plans:** Use a [Development cluster](http://localhost:3000/docs/next/components/console/manage-clusters/create-cluster/#development-clusters). For Professional Plans, you may need to purchase a Development cluster. | ||
- **Enterprise Plans:** Your organization must be designated as a development organization. Contact your CSM for further assistance. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i don't think this is documented anywhere in the outside world
The use of Play may result in additional charges depending on your organization's plan and the type of cluster you are using. To avoid incurring extra costs, please follow these guidelines based on your plan: | ||
|
||
- **Trial Plans:** You can use any cluster. | ||
- **Starter/Professional Plans:** Use a [Development cluster](http://localhost:3000/docs/next/components/console/manage-clusters/create-cluster/#development-clusters). For Professional Plans, you may need to purchase a Development cluster. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This links to localhost- probably not where we want to send customers
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😮 fixed it
|
||
- **Trial Plans:** You can use any cluster. | ||
- **Starter/Professional Plans:** Use a [Development cluster](http://localhost:3000/docs/next/components/console/manage-clusters/create-cluster/#development-clusters). For Professional Plans, you may need to purchase a Development cluster. | ||
- **Enterprise Plans:** Your organization must be designated as a development organization. Contact your CSM for further assistance. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be the first option.
I am trying to consolidate multiple instances of Web Modeler wherever possible. This is critical for upcoming changes for tracking the development lifecycle across multiple stages and consolidating development and optimization efforts.
In 8.7, when deployment permissions are pushed to the clusters rather than Console, we can start making this shift.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I included this part and added a link to where this is mentioned. please check the plan descriptions again.
I kept the part where enterprise organizations can be marked as 'dev organization' to avoid costs as @ultraschuppi mentioned this was also a way to avoid the costs.
faea078
to
2aa501d
Compare
2aa501d
to
a620ad1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
a620ad1
to
04cea48
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
@conceptualshark the only open question that is blocking this PR to be merge is this : #4299 (comment) , which was awaiting input from @felix-mueller , but maybe you or @HanselIdes also have information regarding this? another alternative would be to keep it as it is and release a patch later if @felix-mueller would like it to be changed, as this PR is currently blocking Play to move forward with another feature that requires links to this doc 😅 |
@huygur Unfortunately I don't have any more information on trial plan naming - we use Trial/Free Trial throughout the docs, so if there is a decision to start specifying SaaS (or using another name), we can always make a ticket for that and update it across the docs. |
@conceptualshark ok then, thanks. I will merge this PR then to unblock Play. |
04cea48
to
56bfd56
Compare
Description
closes https://github.com/camunda/play/issues/622
When should this change go live?
hold
label or convert to draft PR)PR Checklist
/versioned_docs
directory./docs
directory (aka/next/
).