Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[admin menus] Correcting filters and sorting #14786

Merged
merged 1 commit into from
Mar 20, 2017

Conversation

infograf768
Copy link
Member

@infograf768 infograf768 commented Mar 19, 2017

Pull Request for Issue #14655

Summary of Changes

Adding a different filter file when client is administrator.
This file does not contain the Access and Language Filters.
The sorting does not contain: Home, Language, Access, Association

Testing Instructions

Patch and display administrator menu items page.

Expected result

screen shot 2017-03-19 at 18 45 08

@brianteeman @izharaazmi @yasirunilan

@AlexRed
Copy link
Contributor

AlexRed commented Mar 20, 2017

I have tested this item ✅ successfully on 602f259

Patch ok for me


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

@ghost
Copy link

ghost commented Mar 20, 2017

I have tested this item ✅ successfully on 602f259


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

@infograf768
Copy link
Member Author

RTC. Thanks for testing!


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Mar 20, 2017
@wilsonge wilsonge merged commit 5b44471 into joomla:staging Mar 20, 2017
@wilsonge wilsonge added this to the Joomla 3.7.0 milestone Mar 20, 2017
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Mar 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants