Skip to content

[4.0] Reimplementing client filter in Modules Manager#23406

Merged
wilsonge merged 6 commits intojoomla:4.0-devfrom
infograf768:4.0_modules_client_filter
Jan 6, 2019
Merged

[4.0] Reimplementing client filter in Modules Manager#23406
wilsonge merged 6 commits intojoomla:4.0-devfrom
infograf768:4.0_modules_client_filter

Conversation

@infograf768
Copy link
Member

Pull Request for Issue #23400

Summary of Changes

As title says

Testing Instructions

Display Modules Manager

After patch

Client filter now displays.

screen shot 2018-12-31 at 08 14 57

screen shot 2018-12-31 at 08 14 37

@richard67
Copy link
Member

I have tested this item ✅ successfully on 373e51f


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

1 similar comment
@chmst
Copy link
Contributor

chmst commented Dec 31, 2018

I have tested this item ✅ successfully on 373e51f


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

@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/23406.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Dec 31, 2018
@infograf768 infograf768 added this to the Joomla 4.0 milestone Dec 31, 2018
@infograf768
Copy link
Member Author

@wilsonge
Please merge as I have some other PRs concerning modules . One waiting ( #23289 ) and another one I have to do for a change missing in #23418

@wilsonge
Copy link
Contributor

wilsonge commented Jan 5, 2019

@SniperSister @zero-24 RIPS please. I'm not really sure how this happened as I reran this a second time so reproducible

http://ci.joomla.org/joomla/joomla-cms/13420
http://ci.joomla.org/joomla/joomla-cms/13418

But all i did was merge in 4.0 🤔

@infograf768
Copy link
Member Author

Looks OK to merge now @wilsonge

@wilsonge wilsonge merged commit 48e4b34 into joomla:4.0-dev Jan 6, 2019
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Jan 6, 2019
@wilsonge
Copy link
Contributor

wilsonge commented Jan 6, 2019

Thanks :)

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.

5 participants