[pve-devel] applied: [PATCH manager] ui: add version to extjs script urls

Thomas Lamprecht t.lamprecht at proxmox.com
Mon Sep 6 13:00:15 CEST 2021


On 06.09.21 12:19, Dominik Csapak wrote:
> so that we circumvent browsers caching 6.0 extjs js/css
> 
> also depend on extjs 7.0.0 in debian control
> 
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
> this should (at least for new users upgrading) fix the browser caching
> issue for extjs (we had some now in the forums)
> 
> i did it this way since we do not often change version of the extjs
> package (since its a big task everytime anyway), but if wanted i can
> do it differently. e.g.:
>     * hardcode it in the perl code
>     * generate it during the build (also for the control file)
> 
> but imho this is fine for extjs
> 

I put above info with only slight adaptions in the commit message, IMO such
reasoning is very valuable information that should be archived in source control.

>  debian/control     |  2 +-
>  www/index.html.tpl | 14 +++++++-------
>  2 files changed, 8 insertions(+), 8 deletions(-)
> 
>

applied, thanks!





More information about the pve-devel mailing list