search

rwth-acis / CAE-Frontend

This repository contains code for additions to the SyncMeta frontend, needed to realize the CAE.
Other
6 stars 2 forks source link

Update dependencies #60

Open pdolif opened 3 years ago

pdolif commented 3 years ago
The following dependencies should be updated to the respectively specified version: Dependency Update to this version or later CAE app or widgets?
growl (swagger) 1.10.0 widgets
ws (Yjs) 1.1.5 both
engine.io-client (Yjs) 1.6.9 both
diff (swagger) 3.5.0 widgets
lodash (swagger) 4.17.21 widgets
qs (swagger) 6.0.4 widgets
ms (Yjs) 0.7.1 both
urijs (swagger) 1.19.6 widgets
debug (Yjs) 2.6.9 app
minimist (swagger) 0.2.1 widgets
superagent (swagger) 3.7.0 widgets
trim-newlines 3.0.1 widgets
glob-parent 5.1.2 widgets
jquery 3.5.0 widgets
jsrsasign 10.2.0 app
pdolif commented 3 years ago

It seems that swagger is not used anywhere (or I did not find it). The gruntfile task for the swagger widget of the CAE only uses Yjs (see https://github.com/rwth-acis/CAE-Frontend/blob/master/widgets/gruntfile.js#L166, https://github.com/rwth-acis/CAE-Frontend/blob/master/widgets/gruntfile.js#L221). Swagger is not copied to the dist directory.

The live code editor is copying the full node_modules directory, but it has nothing to do with swagger.

Thus, we can get rid of this dependency.

pdolif commented 3 years ago

jquery got updated here: https://github.com/rwth-acis/CAE-Frontend/pull/55

pdolif commented 3 years ago

Updating grunt browserify and contrib-uglify also updated trim-newlines and glob-parent (https://github.com/rwth-acis/CAE-Frontend/commit/64552200ca60c4fc98bfea68ae12f2ea1ecf2a19).

FBasels commented 3 years ago

The openidconnect-signin is updated and does not use jsrsasign anymore, which should fix it here