NPM Resource Changed ? ? Pending

User tests: Successful: Unsuccessful:

avatar SharkyKZ
SharkyKZ
17 Dec 2019

Pull Request for Issue #27287.

Summary of Changes

Updates Bootstrap from 4.3.1 to 4.4.1 and Popper.js from 1.15.0 to 1.16.0.

Testing Instructions

Run npm install.
Browse around frontend and backend.
Check that styling still OK.
Check that Bootstrap JS components (modal, dropdowns, popovers) still work.

Expected result

Works.

Documentation Changes Required

No.

avatar SharkyKZ SharkyKZ - open - 17 Dec 2019
avatar SharkyKZ SharkyKZ - change - 17 Dec 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 17 Dec 2019
Category NPM Change
avatar SharkyKZ SharkyKZ - change - 17 Dec 2019
The description was changed
avatar SharkyKZ SharkyKZ - edited - 17 Dec 2019
avatar brianteeman
brianteeman - comment - 17 Dec 2019

There are quite a few other npm resources that are "hard coded" to a specific release that are now outdated. Is there a policy on how we handle this. Should all npm resources be set to update automatically

avatar SharkyKZ SharkyKZ - change - 19 Dec 2019
Labels Added: NPM Resource Changed ?
avatar SharkyKZ
SharkyKZ - comment - 19 Dec 2019

Depends on the package. Some are locked to the last release of a major version (e.g. Punycode). And some may introduce B/C breaks even in minor releases. I think that's why chosen-js is locked to 1.6.2?

avatar brianteeman
brianteeman - comment - 19 Dec 2019

If they dont follow semantic versioning eg chosen then that make sense

But then we have others which do follow semantic versioning such as tinymce which are "locked" at a patch release.

The entire point of using npm is that we shouldnt need to be manually checking all external scripts for updates

avatar jwaisner jwaisner - test_item - 8 Jan 2020 - Tested successfully
avatar jwaisner
jwaisner - comment - 8 Jan 2020

I have tested this item successfully on 027bc9e


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

avatar Quy Quy - test_item - 15 Jan 2020 - Tested successfully
avatar Quy
Quy - comment - 15 Jan 2020

I have tested this item successfully on a1c1831


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

avatar Quy Quy - change - 15 Jan 2020
Status Pending Ready to Commit
avatar Quy
Quy - comment - 15 Jan 2020

RTC


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

avatar wilsonge wilsonge - change - 16 Jan 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-01-16 12:03:41
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 16 Jan 2020
avatar wilsonge wilsonge - merge - 16 Jan 2020
avatar wilsonge
wilsonge - comment - 16 Jan 2020

Thanks!

I think it largely depends on the product. As we know ourselves - some people claim to do semver but not :) I think we're good in most places to allow patch and minors in our package.json as the general rule tho

Add a Comment

Login with GitHub to post a comment