? NPM Resource Changed ? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
26 Feb 2020

Removes the remaining parts of the helpsite code that was removed in 3.9. due to file renaming these were missed.

code review

discovered by finally having the lint running

avatar brianteeman brianteeman - open - 26 Feb 2020
avatar brianteeman brianteeman - change - 26 Feb 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 26 Feb 2020
Category Administration com_users Language & Strings Templates (admin) JavaScript Repository NPM Change Libraries
avatar Quy
Quy - comment - 26 Feb 2020

Delete \administrator\help\helpsites.xml?

avatar brianteeman
brianteeman - comment - 26 Feb 2020

No that is used by the help button

avatar Quy
Quy - comment - 26 Feb 2020

Deleted the file without affecting the Help button. The value is obtained from the configuration file.

public $helpurl = 'https://help.joomla.org/proxy?keyref=Help{major}{minor}:{keyref}&lang={langcode}';

avatar brianteeman brianteeman - change - 26 Feb 2020
Labels Added: ? NPM Resource Changed ?
avatar Quy
Quy - comment - 26 Feb 2020

I have tested this item successfully on a0f64ec


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

avatar Quy Quy - test_item - 26 Feb 2020 - Tested successfully
avatar astridx
astridx - comment - 29 Feb 2020

If I am right, first PR was this: #24927

I applied this patch and run composer i and npm ci.

I opened user manager and clicked Help (Aide). Help Site https://help.joomla.org/proxy?keyref=Help40:Users_User_Manager&lang=fr was opened.

I opened some more Help Sites and had no problems.

avatar astridx
astridx - comment - 29 Feb 2020

I have tested this item successfully on a0f64ec


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

avatar astridx astridx - test_item - 29 Feb 2020 - Tested successfully
avatar Quy Quy - change - 29 Feb 2020
Status Pending Ready to Commit
avatar Quy
Quy - comment - 29 Feb 2020

RTC


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

avatar wilsonge wilsonge - change - 2 Mar 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-03-02 13:30:40
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 2 Mar 2020
avatar wilsonge wilsonge - merge - 2 Mar 2020
avatar wilsonge
wilsonge - comment - 2 Mar 2020

Thanks!

avatar brianteeman
brianteeman - comment - 2 Mar 2020

thanks

Add a Comment

Login with GitHub to post a comment