mrmap-community / Mapbender2.8

Other
3 stars 1 forks source link

jQuery update #95

Open SvenTUM opened 2 years ago

SvenTUM commented 2 years ago

The extensions used in Mapbender are using jQuery on multiple different versions. These need to be updated / replaced.

GUI Elements in need for an adjustment should be identified as well.

armin11 commented 2 years ago

hi sven - much of the work has been done one and a half ago and lives in a local environment - but there are many things that should be checked and further adoptions that have to be done ;-) - first i updated jquery to the last 1.X version - the following updates may be also a little bit tricky :-(

SvenTUM commented 2 years ago

Hi @armin11 ! Yeah I figured it'd be tricky with all the stuff hidden in the GUI settings.

How should we tackle this issue?

Iterative w/ communication

  1. Introduce newer JS files (in the repository)
  2. Use newer versions in the GUIs (Testing -> Production)
  3. Remove old JS files (in the repository)

This requires good communication as the old files should not be deleted before the GUIs in all instances are up-to-date.

DB Update Script

  1. Replace the JS files (in the repository)
  2. Create an SQL Update script replacing the JS files in the GUI Element DB entries

This seems to be the cleaner update method, yet I think it's risky to start introducing it now.

More suggestions...

karlbrink commented 6 months ago