? ? Pending

User tests: Successful: Unsuccessful:

avatar alikon
alikon
15 Aug 2018

Pull Request for Issue #21593 .

Summary of Changes

removed workflow menuitem from user menu
and fields menu item pointing to content

avatar alikon alikon - open - 15 Aug 2018
avatar alikon alikon - change - 15 Aug 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 15 Aug 2018
Category Administration com_menus
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 15 Aug 2018 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 15 Aug 2018

I have tested this item successfully on e05e4f1

"Workflow" is removed from User-Menue.


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

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 15 Aug 2018

@alikon do you plan to remove "content fields" in another PR?

avatar alikon
alikon - comment - 15 Aug 2018

no no ....just not looking at this with the needed attention..... let me check better let me recheck ?

avatar alikon alikon - change - 15 Aug 2018
Labels Added: ?
avatar infograf768
infograf768 - comment - 15 Aug 2018

I think that Workflows is voluntarily added there.
The issue imho is that we have 2 Fields menu item, one for users (normal), the other for content which should not be there imho..

avatar alikon
alikon - comment - 15 Aug 2018

removed the content field menu item on user menu now

avatar alikon
alikon - comment - 15 Aug 2018

think that Workflows is voluntarily added there

@infograf768 why ?

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 15 Aug 2018

I have tested this item successfully on 3e9e528


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

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 15 Aug 2018 - Tested successfully
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 15 Aug 2018 - Not tested
avatar infograf768
infograf768 - comment - 15 Aug 2018

Why?

It is a good question indeed.
I can't believe it's a mistake as It is the ONLY direct menu item to Workflows.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 15 Aug 2018

@alikon can you please change Title to show that "workflow" and double "Fields" are removed?

avatar infograf768
infograf768 - comment - 15 Aug 2018

Before deciding to take off the Workflows menu item, this should be seen by maintainers an Workflow people.

avatar alikon
alikon - comment - 15 Aug 2018

uhmm ... there is a Workflow item in the CONTENT menu
screenshot from 2018-08-15 11-21-09

avatar infograf768
infograf768 - comment - 15 Aug 2018

there is a Workflow item in the CONTENT menu

I stand corrected.

avatar alikon alikon - change - 15 Aug 2018
The description was changed
avatar alikon alikon - edited - 15 Aug 2018
avatar alikon
alikon - comment - 15 Aug 2018

@infograf768 on workflow things we are moving on quicksand ?

avatar infograf768
infograf768 - comment - 15 Aug 2018

on workflow things we are moving on quicksand

Just trying to get at least one leg out of it... ?

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 15 Aug 2018 - Tested successfully
avatar franz-wohlkoenig franz-wohlkoenig - alter_testresult - 15 Aug 2018 - infograf768: Tested successfully
avatar franz-wohlkoenig franz-wohlkoenig - change - 15 Aug 2018
Status Pending Ready to Commit
avatar franz-wohlkoenig franz-wohlkoenig - alter_testresult - 15 Aug 2018 - infograf768: Tested unsuccessfully
avatar franz-wohlkoenig franz-wohlkoenig - alter_testresult - 15 Aug 2018 - infograf768: Not tested
avatar franz-wohlkoenig franz-wohlkoenig - change - 15 Aug 2018
Status Ready to Commit Pending
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 15 Aug 2018 - Not tested
avatar wilsonge wilsonge - change - 15 Aug 2018
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-08-15 10:50:04
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 15 Aug 2018
avatar wilsonge wilsonge - merge - 15 Aug 2018
avatar wilsonge
wilsonge - comment - 15 Aug 2018

Thanks!

Add a Comment

Login with GitHub to post a comment