NPM Resource Changed ? ? Pending

User tests: Successful: Unsuccessful:

avatar C-Lodder
C-Lodder
1 Nov 2020

Summary of Changes

Libsass and node-sass are deprecated, so this PR moves J4 over to Dart SASS.

This will also fix not being able to run CSS related build scripts for Joomla 4 on NodeJS 15+

Testing Instructions

Run npm run build:css from your terminal

avatar C-Lodder C-Lodder - open - 1 Nov 2020
avatar C-Lodder C-Lodder - change - 1 Nov 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 1 Nov 2020
Category Administration Templates (admin) NPM Change JavaScript Repository Front End Templates (site)
avatar HLeithner
HLeithner - comment - 1 Nov 2020

please fix cs @C-Lodder thx

avatar brianteeman
brianteeman - comment - 1 Nov 2020

Is it correct that there is a new file in this PR build/media_source/system/scss/_jalert.scss

avatar SharkyKZ
SharkyKZ - comment - 1 Nov 2020

@brianteeman No, it should be removed.

avatar C-Lodder C-Lodder - change - 2 Nov 2020
Labels Added: NPM Resource Changed ?
avatar C-Lodder
C-Lodder - comment - 2 Nov 2020

done

avatar ghazal ghazal - test_item - 2 Nov 2020 - Tested successfully
avatar ghazal
ghazal - comment - 2 Nov 2020

I have tested this item successfully on 1620dc7

Tested OK providing that Composer dependency phpcodesniffer-composer-installer is updated as I use composer 2.
I don't PR this update as I can't test on a lesser version of Composer (i.e Composer 1.x). But it is ready if needed.

composer 2.0.4
npm 6.14.8
node v15.0.1

PHP 7.4.9
macOS Catalina


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

avatar jwaisner
jwaisner - comment - 2 Nov 2020

@C-Lodder can you please correct the conflict in this PR?

avatar C-Lodder C-Lodder - change - 3 Nov 2020
Labels Added: Conflicting Files
avatar C-Lodder C-Lodder - change - 3 Nov 2020
The description was changed
avatar C-Lodder C-Lodder - edited - 3 Nov 2020
avatar jwaisner jwaisner - test_item - 3 Nov 2020 - Tested successfully
avatar jwaisner
jwaisner - comment - 3 Nov 2020

I have tested this item successfully on f565079

No issues on NPM build. Checked a new install and everything functions as expected.


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

avatar SharkyKZ SharkyKZ - test_item - 4 Nov 2020 - Tested successfully
avatar SharkyKZ
SharkyKZ - comment - 4 Nov 2020

I have tested this item successfully on f565079


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

avatar SharkyKZ SharkyKZ - change - 4 Nov 2020
Status Pending Ready to Commit
avatar SharkyKZ
SharkyKZ - comment - 4 Nov 2020

RTC.


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

avatar wilsonge wilsonge - close - 5 Nov 2020
avatar wilsonge wilsonge - merge - 5 Nov 2020
avatar wilsonge wilsonge - change - 5 Nov 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-11-05 09:57:49
Closed_By wilsonge
Labels Added: ?
Removed: Conflicting Files
avatar wilsonge
wilsonge - comment - 5 Nov 2020

Thanks!

Add a Comment

Login with GitHub to post a comment