Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bust cached javascript ressources with each new release #486

Closed
Archaeopteryx opened this issue Jun 17, 2015 · 3 comments
Closed

Bust cached javascript ressources with each new release #486

Archaeopteryx opened this issue Jun 17, 2015 · 3 comments
Assignees

Comments

@Archaeopteryx
Copy link

@Archaeopteryx Archaeopteryx commented Jun 17, 2015

Enabling the checkbox to set the value of 'Search In' as default doesn't get used as default value the next time transvision gets loaded. Test with Firefox 38.0.5

@flodolo
Copy link
Contributor

@flodolo flodolo commented Jun 17, 2015

It's a cache issue (goes away with a forced reload). Not sure if we want to be less aggressive with cache
https://github.com/mozfr/transvision/blob/master/web/.htaccess#L15-L22

@flodolo flodolo closed this Jun 17, 2015
@pascalchevrel
Copy link
Member

@pascalchevrel pascalchevrel commented Jun 18, 2015

We can add a query parameter with the version number, this way the http cache will be busted with each release. Looking at the code, that's what we are already doing with css, reopening and assigning to myself.

@pascalchevrel pascalchevrel reopened this Jun 18, 2015
@pascalchevrel pascalchevrel self-assigned this Jun 18, 2015
@pascalchevrel pascalchevrel changed the title 'Search In' default not remembered (Strings & Entities) Bust cached javascript ressources with each new release Jun 18, 2015
pascalchevrel added a commit to pascalchevrel/transvision that referenced this issue Jun 18, 2015
pascalchevrel added a commit to pascalchevrel/transvision that referenced this issue Jun 18, 2015
pascalchevrel added a commit that referenced this issue Jun 18, 2015
Issue #486: bust js and css caches with new release
@pascalchevrel
Copy link
Member

@pascalchevrel pascalchevrel commented Jun 18, 2015

fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.