? ? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
14 May 2017

Sample learn and sample test had some broken links to the JED

Pull Request for Issue #15758

avatar brianteeman brianteeman - open - 14 May 2017
avatar brianteeman brianteeman - change - 14 May 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 14 May 2017
Category SQL Installation Postgresql MS SQL Unit Tests
avatar Quy
Quy - comment - 14 May 2017

Change https://extensions.joomla.org/category/marketing-sharing/ to https://extensions.joomla.org/category/marketing/.

avatar brianteeman
brianteeman - comment - 14 May 2017

@Quy disagree - not sure where you got that link but if akeeba backup is the first item then its the wrong stuff

avatar Quy
Quy - comment - 14 May 2017

Your PR has the link https://extensions.joomla.org/category/marketing-sharing which has the Akeeba backup. The suggested URL does not and is a valid category.

avatar brianteeman
brianteeman - comment - 14 May 2017

sorry - my mistake - i misread the report

avatar brianteeman brianteeman - change - 14 May 2017
Labels Added: ? ?
avatar Quy
Quy - comment - 14 May 2017

I have tested this item successfully on fbafd3a


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

avatar Quy Quy - test_item - 14 May 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 14 May 2017

@Quy how you have test PR?

avatar Quy
Quy - comment - 14 May 2017

Code review and also manually go to each revised link in my browser.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 14 May 2017

Thanks, so there is no Way to test by Installation.

avatar brianteeman
brianteeman - comment - 14 May 2017
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 14 May 2017

I have tested this item successfully on fbafd3a

- Link "comments" > https://extensions.joomla.org/extensions/contacts-and-feedback/articles-comments

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 14 May 2017 - Tested successfully
avatar franz-wohlkoenig franz-wohlkoenig - change - 14 May 2017
Status Pending Ready to Commit
Easy No Yes
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 14 May 2017

RTC after two successful tests.

avatar elkuku
elkuku - comment - 14 May 2017

@franz-wohlkoenig note that we have added a link for such "edge cases" ?
download

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 15 May 2017

@elkuku thanks for Hint, haven't noticed before.

avatar rdeutz rdeutz - change - 22 May 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-05-22 18:24:52
Closed_By rdeutz
Labels Added: ?
avatar rdeutz rdeutz - close - 22 May 2017
avatar rdeutz rdeutz - merge - 22 May 2017
avatar brianteeman
brianteeman - comment - 22 May 2017

Thanks

Add a Comment

Login with GitHub to post a comment