? Pending

User tests: Successful: Unsuccessful:

avatar dgt41
dgt41
29 Nov 2017

Pull Request for Issue # .

Summary of Changes

For the Joomla update process there are 2 external libraries that are required, one for json parsing and another for encryption (AES).

For Json parsing we are switching from json2 to json3 (It is a drop-in replacement for JSON 2: https://bestiejs.github.io/json3/) because Douglas Crockford turned down our request for publishing json2 to NPM (douglascrockford/JSON-js#92)!

For AES we just bumped the version to the latest available.

And that is the part that takes care of the maintenance of the external scripts, but there is an ongoing work to drop jQuery from all the scripts. The later will be finished by the javascript team, but feel free to help with a PR.

Testing Instructions

Not ready for testing yet!

Expected result

Actual result

Documentation Changes Required

Nope. Housekeeping

Att @Fedik @C-Lodder @dneukirchen @yvesh

c0ac215 29 Nov 2017 avatar dgrammatiko CS
ed879f7 29 Nov 2017 avatar dgrammatiko sync
avatar dgt41 dgt41 - open - 29 Nov 2017
avatar dgt41 dgt41 - change - 29 Nov 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 29 Nov 2017
Category JavaScript Repository Administration com_joomlaupdate
avatar dgt41 dgt41 - change - 29 Nov 2017
The description was changed
avatar dgt41 dgt41 - edited - 29 Nov 2017
9364018 29 Nov 2017 avatar dgrammatiko typo
avatar dgt41 dgt41 - change - 29 Nov 2017
Labels Added: ?
avatar dgrammatiko
dgrammatiko - comment - 21 May 2018

Joomla cannot use npm for the aes script. Another solution needed here

avatar dgrammatiko dgrammatiko - change - 21 May 2018
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2018-05-21 20:03:28
Closed_By dgrammatiko
avatar dgrammatiko dgrammatiko - close - 21 May 2018

Add a Comment

Login with GitHub to post a comment