Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
OOTB PDLC Happy Path documentation #4144
OOTB PDLC Happy Path documentation #4144
Changes from 16 commits
aa06746
8fdf4b4
c6541a3
331d0a2
5c30023
b6a92a0
623b0e6
193eb54
6c738d3
ab9ca04
e7c8018
9a1029f
2b740e7
cf8a30d
0d000a5
a89b2c7
caeec16
aed6c3a
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
A minor comment: this feels verbose and hyper-specific. I don't have a concrete requirement or request for change so feel free to ignore it for now
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.
Heads-up: In 8.6, I will start pushing users to use Play for this purpose rather than Run
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.
Should I remove the "Run" section in this page then? If so, do we need to replace it with a "Play" section, or just point people towards the Play feature docs I would guess?
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.
You don't need to make changes for now, but we'll need to revisit this for the 8.6.0 release
Between now and 8.6, it makes sense to deploy your process app since Play doesn't connect to a cluster
Starting with 8.6, we should tell people to use Play to run their process in a development cluster and only use the Deploy/Run modals for interacting with other stages
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.
Thanks for the heads up: I've created #4182 for this work so it doesn't get missed.
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.
Although technically the functionality is a bulk milestone, it functions as a version. Versions make more sense in the context. We'll improve this in Q4
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.
Thanks - I have to document it as is, so technically have to say bulk milestone and so on, but as soon as we can, we can start using 'version' 👍