? ? Pending

User tests: Successful: Unsuccessful:

avatar alikon
alikon
3 Jul 2019

Pull Request for Issue #25237 .

Summary of Changes

removed the individual credits from templates manifest, i.e the author tag now is:
<author>Joomla! Project</author>

like all over the core code

avatar alikon alikon - open - 3 Jul 2019
avatar alikon alikon - change - 3 Jul 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 3 Jul 2019
Category Administration Templates (admin) Front End Templates (site)
avatar brianteeman brianteeman - test_item - 3 Jul 2019 - Tested successfully
avatar brianteeman
brianteeman - comment - 3 Jul 2019

I have tested this item successfully on 59a2c7e


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

avatar Quy Quy - test_item - 3 Jul 2019 - Tested successfully
avatar Quy
Quy - comment - 3 Jul 2019

I have tested this item successfully on 59a2c7e


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

avatar Quy
Quy - comment - 3 Jul 2019

I have tested this item successfully on 59a2c7e


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

avatar Quy Quy - change - 3 Jul 2019
Status Pending Ready to Commit
avatar Quy
Quy - comment - 3 Jul 2019

RTC


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

avatar wilsonge
wilsonge - comment - 4 Jul 2019

We have done this before in Joomla https://github.com/joomla/joomla-cms/blob/staging/administrator/templates/hathor/templateDetails.xml#L6 j3 (and 2.5) and j1.5 https://github.com/gdimi/Joomla1.5.X/blob/master/administrator/templates/khepri/templateDetails.xml#L6 so for whatever reason we seem to have always treated templates different ?‍♂

avatar alikon
alikon - comment - 4 Jul 2019

so what's the point ? we should continue to do it ?...
then we should give credits in the <author> tag to every pr author if we follow this approach, why have 2nd class contributor ?

avatar brianteeman
brianteeman - comment - 4 Jul 2019

The reason it was done before was explained by @ot2sen #25237 (comment) That definitely is not a valid reason to do it now.

avatar wilsonge wilsonge - close - 4 Jul 2019
avatar wilsonge wilsonge - merge - 4 Jul 2019
avatar wilsonge wilsonge - change - 4 Jul 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-07-04 09:54:40
Closed_By wilsonge
Labels Added: ? ?
avatar brianteeman
brianteeman - comment - 4 Jul 2019

Thank you very much

Add a Comment

Login with GitHub to post a comment