? NPM Resource Changed ? Pending

User tests: Successful: Unsuccessful:

avatar dgrammatiko
dgrammatiko
12 Mar 2022

Pull Request for Issue # .

Kinda replaces #36906 (no FA upgrade to v6)

Summary of Changes

  • patch the font awesome files
  • local fixes

Testing Instructions

  • Apply this Pr's branch
  • run npm ci
  • Check that nothing is broken visually and there are no console deprecation when running any node (css related) cli command

Actual result BEFORE applying this Pull Request

Expected result AFTER applying this Pull Request

Documentation Changes Required

e37f93f 10 Mar 2022 avatar dgrammatiko CS
5362f4a 10 Mar 2022 avatar dgrammatiko meh
49d2787 10 Mar 2022 avatar dgrammatiko grrr
3c8a789 10 Mar 2022 avatar dgrammatiko order
bec8d8f 10 Mar 2022 avatar dgrammatiko fo
avatar dgrammatiko dgrammatiko - open - 12 Mar 2022
avatar dgrammatiko dgrammatiko - change - 12 Mar 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 12 Mar 2022
Category Administration com_media NPM Change JavaScript Repository
52fd472 12 Mar 2022 avatar dgrammatiko nope
avatar dgrammatiko dgrammatiko - change - 12 Mar 2022
Labels Added: NPM Resource Changed ?
avatar brianteeman
brianteeman - comment - 12 Mar 2022

I can confirm that the deprecation notices have now gone

Is it intentional that the joomla.asset.json files no longer have a version hash?

Commented on the code where there is an error in your math

avatar brianteeman
brianteeman - comment - 12 Mar 2022

I have tested this item successfully on 5f01580

Thank you


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

avatar brianteeman brianteeman - test_item - 12 Mar 2022 - Tested successfully
avatar dgrammatiko
dgrammatiko - comment - 12 Mar 2022

Is it intentional that the joomla.asset.json files no longer have a version hash?

The hash is not generated automatically per npm ci, you need to run specifically npm run versioning. (the reason is because for development having to invalidate this hash every time you apply some changes in any of the css/js would be an extra step which will be a way slower workflow)

avatar richard67
richard67 - comment - 14 Mar 2022

I have tested this item successfully on 5f01580


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

avatar richard67 richard67 - test_item - 14 Mar 2022 - Tested successfully
avatar richard67 richard67 - change - 14 Mar 2022
Status Pending Ready to Commit
avatar richard67 richard67 - edited - 14 Mar 2022
avatar richard67
richard67 - comment - 14 Mar 2022

RTC


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

avatar bembelimen bembelimen - change - 17 Mar 2022
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-03-17 19:55:20
Closed_By bembelimen
Labels Added: ?
avatar bembelimen bembelimen - close - 17 Mar 2022
avatar bembelimen bembelimen - merge - 17 Mar 2022
avatar bembelimen
bembelimen - comment - 17 Mar 2022

Thx

Add a Comment

Login with GitHub to post a comment