User tests: Successful: Unsuccessful:
A sample workflow is added to sample data.
You need a fresh installation with a clean database.
Tip: you can copy the empty database (phpmyadmin) in case you need again an empty database.
Apply the patch, go to the main dashboard and install blog samole data.
Refresh the Screen (because the workflow must be loaded into the menu). Then you should have a new workflow in your content area:
Status | New | ⇒ | Pending |
Category | ⇒ | Administration Language & Strings Front End Plugins |
Title |
|
Labels |
Added:
?
?
|
Thank you @richard67
@chmst I've just applied the patch of this PR and then made a blog sample data installation. The good thing: There are no PHP errors or warnings or notices, and no SQL errors. And in general it looks fine.
Only one thing confuses me. Can it be that the names and target states of the 2 transitions "Trash" and "Set on Hold" are mixed up?
I would expect transition "Trash" end with the "Trashed" state, and "Set on Hold" end with the "On Hold" state.
Maybe just the language strings for the titles of those 2 (transitions 7 and 8) have been mixed up.
Thanks, the titles were swapped, as you said.
I have added another transition to make an article from "on hold" to "idea" again.
I have added another transition to make an article from "on hold" to "idea" again.
Yes, that would have been my next question, if there is a way away from "On Hold", but you already solved it
@chmst I've just tested what happens if I have a multilingual site and install the Blog Sample Data two times, one time with backend language = English (GB) and one time with German (DE).
For the articles, the language code is appended in brackets at the end of the titles on a multilingual site, so I have e.g. "Your Template (de-DE)" and "Your Template (en-GB)". Similar with categories.
The sample workflow is also inserted 2 times, one time for English (GB) and one time with German (DE), but the titles of workflow, stages and transitions are all the same.
I am not sure now what would be better, insert one time for each language but the with different titles? In this case @infograf768 could advise you what to do in the sample dara code.
Or should the sample workflow be inserted only one time?
Meanwhile I think it should be inserted only one time because workflows and transitions are not language-specific (they have no language column in database, as far as I remember), and their titles and descriptions are translated anyway.
Each time the sample data are running, a new workflow has to be created, because:
I see.
Drone is not happy and it concerns token.php ??
@infograf768 no clue ?? I did not touch tokens
No clue at all.
PR I merged yesterday with some bad code style. I already fixed it and retriggered drone. should have a new result in about 10-15 minutes
I have tested this item
Works great
Largely seems good but I wonder after expert revision whether it should be able to go straight to published? I don't really understand the intent of the "Content Revision" step.
Also I wonder if this should be applied to the blog category as part part of the sample data. Before we setup the blog articles and then run the blog articles through the workflow stages (so editing existing sample data blog articles gives users the same experience)
Well, I had in mind that somone must check the html and accessibility (alt-texts, captions) before setting it online. Maybe a description would help?
There are much more or other tranitions and roles possible, for example translators - but then the example is to big.
I also did not know if I can add this sample workflow to the blog category, because the workflow is disabled by default.
Well, I had in mind that somone must check the html and accessibility (alt-texts, captions) before setting it online. Maybe a description would help?
There also much more or other tranitions and roles possible, for example translators - but then the example is to big.
I also did not know if I can add this sample workflow to the blog category, because the workflow is disabled by default.
@chmst, as discussed in BFH channel, I gave this a test yesterday, and the PR passed in what it's designed to do, but I think there's some changes that might fix some of the things I encountered.
@wilsonge wrote:
Also I wonder if this should be applied to the blog category as part part of the sample data. Before we setup the blog articles and then run the blog articles through the workflow stages (so editing existing sample data blog articles gives users the same experience)
I agree, as there's a step that needed to be done to enable the workflow that would not be intuitive to those just clicking the "Install Blog Sample Data".
@chmst wrote in Glip:
There is one thing where I also did not realise - changing a category does not change the workflow where the article is stored. It is not a bug - because: the article was in a workflow maybe in stage "confirmed". Then you change the category and the new category has only the default stages and transitions. So to which stage the article should be mapped?
The change below outlines perhaps the overview of how to implement the sample data so that the workflow applies on installation.
Before the proposed changes
Proposed changes
There may be more discovered by those testing scenarios.
After the proposed changes
There may be a need to further develop a 3rd sample data option: Blog Sample Data with Workflow
Not sure if this is already able to be done?
I'm keen to also learn more on creating sample data sets that can be installed via the extension manager. That would enable workflow scenarios to be developed as samples that can be downloaded to install via documentation, or developed by third party extension developers to allow them to package workflows or workflow examples for their extensions if they're looking at implementing Workflow in their components.
Labels |
Added:
?
|
Labels |
Added:
?
Removed: ? |
@particthistle Thank you very much for testing and feedback. I have implemented your proposals. Could you re-test?
@infograf768: Hopefully I did not break any mulitlingual concerns
Labels |
Removed:
?
|
Thank you @brianteeman - seems I have closed some proposals too quickly. But will revise ;) the language files
I have tested this item
Tested updates successfully. Specifically:
I have tested this item
Testing again following language change.
Last update was only a clean update to latest changes in the base branch, so the previous test result is still valid. I've restored it in the issue tracker.
@richard67 thank you for testing, I am on it. The resume should be possible from any stage.
@richard67 thank you for testing, I am on it. The resume should be possible from any stage.
@chmst Sure you mean "resume"? Or shouldn't it be "set on hold"?
@particthistle Could you test again, this time with checking if the possible transitions depending on the current stage are as shown as in the diagram in the testing instructions? Thanks in advance. I will try to find time, too, but it may take a while.
@chmst we're getting very close - it's probably a pass when you watch the GIF below, but, the transitions available when you "Publish and Feature" might need an addition... though they're also not shown in the workflow flow chart either.
Currently the only transition that is available when the article reaches "Published and Featured" is "Set on Hold" when the following options may be better:
Resetting the article to On Hold where it then goes to an Archived status then resets the workflow back to Idea and starts the flow again. Just trying to get the example to be as comprehensive as it can be to assist with selling the workflow feature.
@particthistle thank you very much for your patience in testing! I'll check again.
Labels |
Added:
?
|
Labels |
Added:
?
Removed: ? |
@particthistle Following your suggestion I have added a stage unpublished. The article now can be unpublished and published again.
I have tested this item
It has Patricks additions and works well. Good job
I have tested this item
Excellent work @chmst!
All the scenarios outlined in the flowchart work.
Status | Pending | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2020-10-17 11:11:16 |
Closed_By | ⇒ | wilsonge | |
Labels |
Added:
?
Removed: ? |
Thank you @brianteeman