? Language Change NPM Resource Changed PR-4.3-dev Pending

User tests: Successful: Unsuccessful:

avatar sdwjoomla
sdwjoomla
2 Dec 2022
avatar sdwjoomla sdwjoomla - open - 2 Dec 2022
avatar sdwjoomla sdwjoomla - change - 2 Dec 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 2 Dec 2022
Category JavaScript Administration com_media NPM Change com_templates Language & Strings Repository Front End com_users Installation Layout Libraries Plugins
avatar joomla-cms-bot
joomla-cms-bot - comment - 2 Dec 2022

Please add more information to your issue. Without test instructions and/or any description we will close this issue within 4 weeks. Thanks.
This is an automated message from the J!Tracker Application.

avatar brianteeman
brianteeman - comment - 2 Dec 2022

changes in package.json do not look right

some packages are being removed and some others are being downgraded

avatar richard67
richard67 - comment - 3 Dec 2022

changes in package.json do not look right

some packages are being removed and some others are being downgraded

Yes, and it also seems to undo Dimitris‘ move from sass to sass-embedded.

See https://github.com/joomla/joomla-cms/pull/38663/files

avatar richard67
richard67 - comment - 3 Dec 2022

And besides that it changes package.json but not package.lock, that can’t go well.

avatar richard67
richard67 - comment - 3 Dec 2022

To properly solve this mess, it needs to study the change history of the package.json in both branches 4.2-dev and 4.3-dev and check each change individually. And after that when the package lock is right it needs to remove the lock file and run npm install, or keep the lock and run npm update for the individual packages which shall be updated.

avatar obuisard obuisard - change - 3 Dec 2022
Labels Added: ? Language Change NPM Resource Changed PR-4.3-dev
avatar richard67
richard67 - comment - 4 Dec 2022

By review the changes in package.json look ok to me. Changes from PR's #39320 and #39315 correctly merged up from 4.2-dev, and no changes for 4.3-dev got lost. The 2 mentioned PR's from 4.2-dev do not result in changes in the lock file, so it is ok that this PR here doesn't change that.

The other changes in this PR I haven't checked.

avatar sdwjoomla sdwjoomla - change - 4 Dec 2022
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-12-04 16:13:59
Closed_By sdwjoomla
avatar sdwjoomla sdwjoomla - close - 4 Dec 2022
avatar sdwjoomla sdwjoomla - merge - 4 Dec 2022

Add a Comment

Login with GitHub to post a comment