-
Notifications
You must be signed in to change notification settings - Fork 95
extended task priorty to cover enterprise and runbooks/deployments #2675
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
base: main
Are you sure you want to change the base?
Conversation
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 👍
@@ -62,6 +62,9 @@ Any future environments you define will also be deployed to as part of this phas | |||
|
|||
When a phase is defined as a priority, deployments to the phase will be created as a priority task unless otherwise specified. When creating a deployment via UI, the priority checkbox will be selected by default. | |||
|
|||
:::div{.info} | |||
From <insert version number here>, **Priority Lifecycle Phase** and **Deployment with Priority** will require an Enterprise license. |
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 know the version number?
This is an MDX file, so this will fail the build as it looks like an opening tag with no closing tag.
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 was a place-holder for the real version number
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.
The version number has been updated
## Priority Deployments and Runbooks | ||
|
||
:::div{.info} | ||
Priority runbooks requires version <insert version 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.
Just a reminder that the version number is needed here, too.
If we don't know it yet, this PR should be a draft.
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.
The version number has been updated
[sc-107381]
This is incomplete. It is waiting on a version number where the the feature toggle has been removed and enterprise license is the only way to get this feature.