-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[4.0] Add default stages/transitions when creating a new workflow #21963
[4.0] Add default stages/transitions when creating a new workflow #21963
Conversation
I have tested this item ✅ successfully on c2ad939 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/21963. |
@@ -104,21 +105,64 @@ public function save($data) | |||
|
|||
$result = parent::save($data); | |||
|
|||
// Create a default stage | |||
// Create a default stages/transitions |
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.
very minor but ==>
Create default stages/transitions
I have tested this item ✅ successfully on 2b68b26 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/21963. |
@brianteeman can you please retest? |
Moving to RTC as the only change after Brians test was the inline comment changed. Thanks! This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/21963. |
I'm not convinced by default we should have an archived state in the default flow. but merging for now |
…omla#21963) * Make the new stage from a new workflow translateable * Create the default workflow stages/transitions when creating a new workflow * Typo fix
Pull Request for Issue #21819 .
Summary of Changes
When creating a new workflow now all default stages (publish, unpublished, trashed, archived) and transitions (unpublish, publish, trash, archive) will be created (instead of only "published").
(And yes, transitions are not translated yet, will come after #21939)
Testing Instructions
Expected result
Actual result