Pending

User tests: Successful: Unsuccessful:

avatar chmst
chmst
13 Oct 2018

Summary of Changes

Workflows and stages do not have checkout fields, therefore the checkin buttons are superfluous.

Testing Instructions

See workflows and stages view before and after patch,
checkout

Documentation Changes Required

yes

avatar chmst chmst - open - 13 Oct 2018
avatar chmst chmst - change - 13 Oct 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 13 Oct 2018
Category Administration com_workflow
avatar chmst chmst - change - 13 Oct 2018
The description was changed
avatar chmst chmst - edited - 13 Oct 2018
avatar brianteeman
brianteeman - comment - 13 Oct 2018

Wondering if instead of removing the button we should instead be adding the functionality - as far as i am aware everything else has a checked out field

avatar chmst
chmst - comment - 13 Oct 2018

This is a bigger change. So we should have more opinions there? @wilsonge, @bembelimen ?

avatar wilsonge
wilsonge - comment - 13 Oct 2018

It's not actually too bad. We just need to add the fields into the database table and everything else should be picked up automatically. My gut is we should probably have checked out support (the entire purpose of this is to avoid simultaneous edits and I don't see why this shouldn't apply to workflows) any other thoughts

avatar chmst
chmst - comment - 13 Oct 2018

All right, thank you for your opinion.
The changes are in the database (and sample data) and the tables of the views. I will do it in the next days.
So this PR can be closed.

avatar Quy Quy - change - 13 Oct 2018
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2018-10-13 23:30:06
Closed_By Quy
avatar joomla-cms-bot
joomla-cms-bot - comment - 13 Oct 2018

Set to "closed" on behalf of @Quy by The JTracker Application at issues.joomla.org/joomla-cms/22596

avatar joomla-cms-bot joomla-cms-bot - change - 13 Oct 2018
Closed_Date 2018-10-13 23:30:06 2018-10-13 23:30:08
Closed_By Quy joomla-cms-bot
avatar joomla-cms-bot joomla-cms-bot - close - 13 Oct 2018

Add a Comment

Login with GitHub to post a comment