Skip to content

Conversation

@drmenzelit
Copy link
Contributor

Pull Request replacing #8580

Summary of Changes

That is a clean pull request replacing #8580 since I accidentally removed the installation folder on my last commit. Sorry for the inconvenience!
Removed the image column parameters as asked by @wilsonge

Testing Instructions

Same as #8580 but without parameters.

@ghost
Copy link

ghost commented Apr 14, 2016

I have tested this item ✅ successfully on 9947a91

Test on Protostar.


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

@brianteeman
Copy link
Contributor

I think this will need a change in the beez3 template as that template has an override for this view


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

@ghost
Copy link

ghost commented Apr 14, 2016

I have tested this item 🔴 unsuccessfully on 9947a91

Works on Protostar, not on Beez3, so i have to change the Test-Result.


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

@joomla-cms-bot
Copy link

This PR has received new commits.

CC: @franz-wohlkoenig


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

@wojsmol
Copy link
Contributor

wojsmol commented Apr 14, 2016

@drmenzelit See drmenzelit#2

@joomla-cms-bot
Copy link

This PR has received new commits.

CC: @franz-wohlkoenig


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

@rdeutz rdeutz added this to the Joomla! 3.6.0 milestone Apr 15, 2016
@iljabos
Copy link

iljabos commented Apr 15, 2016

I have tested this item 🔴 unsuccessfully on bfa98d8

I have tested this but there was no image in the contactlist.
Contact webmaster does have an image.
I tried it on Beez3 and Protostar.


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

@drmenzelit
Copy link
Contributor Author

@iljabos Can you please explain what you did?

@mikeveeckmans
Copy link

I have tested this item ✅ successfully on bfa98d8

Works fine here


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

@iljabos
Copy link

iljabos commented Apr 15, 2016

I have tested this item ✅ successfully on bfa98d8

Second time I tested succesfull for a list of contacts in a single category. The first time I used featured contacts.


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

@brianteeman
Copy link
Contributor

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Apr 15, 2016
@MDXBilal12
Copy link

I have tested this item ✅ successfully on bfa98d8


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

@rdeutz rdeutz added the Feature label May 2, 2016
@rdeutz rdeutz merged commit 9b75278 into joomla:staging May 2, 2016
@brianteeman brianteeman removed the RTC This Pull Request is Ready To Commit label May 11, 2016
@drmenzelit drmenzelit deleted the image-contact-category-list-new branch December 9, 2020 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature Language Change This is for Translators

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants