? ? Pending

User tests: Successful: Unsuccessful:

avatar chmst
chmst
21 Sep 2018

Summary of Changes

Improve the transitons view by adding the icon for the respective condition to every stage.

After Patch
transactions_after

Testing Instructions

Compare the view transition before and after patch.

Actual result

transactions_before

Documentation Changes Required

yes, layout has changed

avatar chmst chmst - open - 21 Sep 2018
avatar chmst chmst - change - 21 Sep 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 21 Sep 2018
Category Administration com_workflow Language & Strings
avatar chmst chmst - change - 21 Sep 2018
Title
Improve layout of transitions overview by adding the condition icons
[4.0] Improve layout of transitions overview
avatar chmst chmst - edited - 21 Sep 2018
avatar chmst chmst - change - 21 Sep 2018
The description was changed
avatar chmst chmst - edited - 21 Sep 2018
avatar brianteeman
brianteeman - comment - 21 Sep 2018

I know it is shorter but it doesn't make sense to me and would be better as

The condition of items at this stage is published

So you can perhaps reuse existing string

COM_WORKFLOW_CONDITION . JPUBLISHED

avatar chmst chmst - change - 21 Sep 2018
Labels Added: ? ?
avatar joomla-cms-bot joomla-cms-bot - change - 21 Sep 2018
Category Administration com_workflow Language & Strings Administration com_workflow
avatar chmst
chmst - comment - 21 Sep 2018

Thank you @brianteeman , good idea.

avatar brianteeman
brianteeman - comment - 21 Sep 2018

I have tested this item successfully on 3de041e


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/22315.

avatar brianteeman brianteeman - test_item - 21 Sep 2018 - Tested successfully
avatar bembelimen
bembelimen - comment - 22 Sep 2018

I have tested this item successfully on 3de041e


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/22315.

avatar bembelimen bembelimen - test_item - 22 Sep 2018 - Tested successfully
avatar franz-wohlkoenig franz-wohlkoenig - change - 22 Sep 2018
Status Pending Ready to Commit
avatar joomla-cms-bot joomla-cms-bot - edited - 22 Sep 2018
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 22 Sep 2018

Ready to Commit after two successful tests.

avatar wilsonge
wilsonge - comment - 23 Sep 2018

Can you fix conflicts and address @Quy 's question please :)

avatar chmst chmst - change - 23 Sep 2018
Labels Added: ?
Removed: ?
avatar joomla-cms-bot joomla-cms-bot - change - 23 Sep 2018
Category Administration com_workflow Administration com_associations com_banners com_categories com_contact com_content com_contenthistory com_finder com_languages com_menus com_newsfeeds com_users
avatar wojsmol
wojsmol - comment - 23 Sep 2018

@chmst Please revert 1016abd and only fix merge conflicts in administrator/components/com_workflow/tmpl/transitions/default.php file. In 1016abd you merged outdated 4.0-dev state into this branch.

avatar wojsmol
wojsmol - comment - 23 Sep 2018

@franz-wohlkoenig Please remove RTC label temporally - see my previous comment.

avatar chmst
chmst - comment - 23 Sep 2018

sorry, I cannot revert the commot need help someone for help. I do it asap

avatar chmst
chmst - comment - 23 Sep 2018

@wojsmol I have pulled the latest v4.0 dev branch of joomla. Is this corret now?

avatar wojsmol
wojsmol - comment - 23 Sep 2018

No, you still have 50 files changed. This should fix the issue:

git checkout -b workflow-transitions-layout-backup #create local backup branch and switch to it.
git checkout workflow-transitions-layout # switch back to This PR branch.
git reset --hard 3de041e27864e96edeea9d90e724c1c6777a91d6 # reset PR to good state.
git push <your-remote-name> -f # push changes to github - replace <your-remote-name> with your joomla fork remote name. This can be checked with git remote -vv 

Then we are at Fix quotes, Use existing language strings commit and then only conflicts neds to to be fixed.

avatar chmst
chmst - comment - 23 Sep 2018

Could I delete this branch? It is simpler to make a new one - it is not much code.

avatar wojsmol
wojsmol - comment - 23 Sep 2018

You can - this will close this PR, If you create new branch with the same name then you should have reopen option on bottom of comment form on github..

avatar franz-wohlkoenig franz-wohlkoenig - change - 23 Sep 2018
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2018-09-23 18:53:12
Closed_By franz-wohlkoenig
avatar joomla-cms-bot joomla-cms-bot - close - 23 Sep 2018
avatar joomla-cms-bot
joomla-cms-bot - comment - 23 Sep 2018
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 23 Sep 2018

Closed as stated in #22343 by @chmst.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/22315.

Add a Comment

Login with GitHub to post a comment