- 🇺🇸United States karlshea Minneapolis 🇺🇸
This would be very helpful, I needed to add a custom transition to our commerce orders so shipping emails didn't get sent and it would be very helpful to have some description text to remind the users which transition sends the emails and which doesn't.
- last update
10 months ago 20 pass, 1 fail - last update
10 months ago 27 pass - last update
10 months ago 27 pass - last update
10 months ago 27 pass - Status changed to Needs review
10 months ago 7:44pm 12 January 2024 - 🇺🇸United States karlshea Minneapolis 🇺🇸
Two issues I can see:
- Constructor signature changes for
WorkflowState
andWorkflowTransition
. - I'm not sure whether/how to surface the state description anywhere,
OrderListBuilder
doesn't really seem to be used since that's normally a view so there would have to be something done to add a title attribute to that cell.
- Constructor signature changes for
- First commit to issue fork.
- last update
7 months ago 27 pass - Open on Drupal.org →Core: 9.5.x + Environment: PHP 7.3 & MySQL 5.7last update
7 months ago Not currently mergeable. - Open on Drupal.org →Core: 9.5.x + Environment: PHP 7.3 & MySQL 5.7last update
7 months ago Not currently mergeable. - Status changed to RTBC
7 months ago 11:45am 1 May 2024 - 🇩🇪Germany gogowitsch
I have tested the feature: very useful for my situation.
I have read the diff (code changes): looks good to me.
- last update
7 months ago 27 pass - 🇮🇱Israel jsacksick
Not against the idea, but the problem is the signature change especially for the WorkflowTransition... The description should be added last, and should default to an empty string so we don't break any code instantiating workflow transition objects.
- 🇨🇦Canada redsky
Thought I'd mention issue #3382018 ✨ Add a field to the schema for transitions to support #button_type Active . It's a different need but it might be good to know there's another idea related to adding an optional field.
Also I wanted to add my support for the great idea being discussed here!
- 🇩🇪Germany gogowitsch
I improved the code based on @jsacksick's feedback in Comment #7 ✨ Add a 'description' field to the schema for states and transitions RTBC .