? ? ? Pending

User tests: Successful: Unsuccessful:

avatar mbabker
mbabker
28 Jul 2018

Pull Request for Issue #21278

Summary of Changes

Run a check during the application bootup to make sure that composer install and npm install are run (well, relatively sane checks at least).

This uses the same approach as the incompatible PHP version page as it relates to language strings, page design, etc. If you're reviewing and don't like the JavaScript, you don't like your own script because it's a copy of the PHP page logic ?

Testing Instructions

Try to run Joomla without the Composer autoloader (file libraries/vendor/autoload.php generated by composer install) or the media/vendor directory (created by npm install). You'll get a friendly error page pointing out you have some extra work to do and a link to the docs page for the extra info.

TODO

  • Try to DRY up the code for these error pages

Screenshot

screen shot 2018-07-28 at 3 41 25 pm

avatar mbabker mbabker - open - 28 Jul 2018
avatar mbabker mbabker - change - 28 Jul 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 28 Jul 2018
Category Administration JavaScript Repository CLI Installation Language & Strings Front End Templates (site)
avatar ggppdk ggppdk - test_item - 28 Jul 2018 - Tested successfully
avatar ggppdk
ggppdk - comment - 28 Jul 2018

I have tested this item successfully on 9e8aadb

Works as described

Just i can not do much of code review on this PR, as i do not have enough knowledge for this


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

avatar Fedik Fedik - test_item - 29 Jul 2018 - Tested successfully
avatar Fedik
Fedik - comment - 29 Jul 2018

I have tested this item successfully on 9e8aadb

very cool!


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

avatar Quy Quy - change - 29 Jul 2018
Status Pending Ready to Commit
avatar Quy
Quy - comment - 29 Jul 2018

RTC


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

avatar brianteeman brianteeman - test_item - 29 Jul 2018 - Tested successfully
avatar brianteeman
brianteeman - comment - 29 Jul 2018

I have tested this item successfully on 9e8aadb


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

avatar wilsonge
wilsonge - comment - 29 Jul 2018

Conflicts please

avatar mbabker mbabker - change - 29 Jul 2018
Labels Added: ?
avatar mbabker
mbabker - comment - 29 Jul 2018

Rebased.

avatar laoneo laoneo - close - 30 Jul 2018
avatar laoneo laoneo - merge - 30 Jul 2018
avatar laoneo laoneo - change - 30 Jul 2018
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-07-30 05:39:31
Closed_By laoneo
Labels Added: ? ?
avatar laoneo
laoneo - comment - 30 Jul 2018

Thanks

Add a Comment

Login with GitHub to post a comment