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

docs(play): include billing information in play docs #4299

Merged
merged 5 commits into from
Sep 18, 2024

Conversation

huygur
Copy link
Contributor

@huygur huygur commented Sep 13, 2024

Description

closes https://github.com/camunda/play/issues/622

When should this change go live?

  • This is a bug fix, security concern, or something that needs urgent release support.
  • This is already available but undocumented and should be released within a week.
  • This on a specific schedule and the assignee will coordinate a release with the DevEx team. (apply hold label or convert to draft PR)
  • This is part of a scheduled alpha or minor. (apply alpha or minor label)
  • There is no urgency with this change and can be released at any time.

PR Checklist

  • My changes are for an already released minor and are in /versioned_docs directory.
  • My changes are for the next minor and are in /docs directory (aka /next/).

Copy link
Contributor

👋 🤖 🤔 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/.

  • docs/components/modeler/web-modeler/play-your-process.md

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.

@huygur huygur marked this pull request as ready for review September 13, 2024 10:27

- **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.
Copy link
Contributor Author

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? 🤔

Copy link
Contributor

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

Copy link
Contributor Author

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

Copy link
Contributor

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.
Copy link
Contributor

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

Copy link
Contributor Author

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.
Copy link
Contributor

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

@christinaausley christinaausley added the component:modeler Issues related with Modeler project label Sep 13, 2024
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.
Copy link
Contributor

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

Copy link
Contributor Author

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.
Copy link
Contributor

Choose a reason for hiding this comment

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

image Enterprise customers can also use Development clusters.

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.

Copy link
Contributor Author

@huygur huygur Sep 18, 2024

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.

HanselIdes
HanselIdes previously approved these changes Sep 18, 2024
Copy link
Contributor

@HanselIdes HanselIdes left a comment

Choose a reason for hiding this comment

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

LGTM

@huygur huygur requested a review from a team September 18, 2024 12:22
@conceptualshark conceptualshark requested review from conceptualshark and removed request for a team September 18, 2024 12:51
Copy link
Contributor

@conceptualshark conceptualshark left a comment

Choose a reason for hiding this comment

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

🚀

@huygur
Copy link
Contributor Author

huygur commented Sep 18, 2024

@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 😅

@conceptualshark
Copy link
Contributor

@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.

@huygur
Copy link
Contributor Author

huygur commented Sep 18, 2024

@conceptualshark ok then, thanks. I will merge this PR then to unblock Play.

@huygur huygur enabled auto-merge (squash) September 18, 2024 13:50
@huygur huygur merged commit ea9f74d into main Sep 18, 2024
7 checks passed
@huygur huygur deleted the play/add-billing-section branch September 18, 2024 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:modeler Issues related with Modeler project
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

5 participants