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

[stable9] Add some basic cache busting #87

Merged
merged 1 commit into from
Jun 14, 2016

Conversation

LukasReschke
Copy link
Member

At least the logo should be correct after an update… Related to #81 but only the most important items.

@blizzz @MorrisJobke @jancborchardt Please review. Would be awesome to get this into 9.

At least the logo should be correct after an update.....
@LukasReschke LukasReschke added the 3. to review Waiting for reviews label Jun 14, 2016
@LukasReschke LukasReschke added this to the Nextcloud 9 milestone Jun 14, 2016
@blizzz
Copy link
Member

blizzz commented Jun 14, 2016

a constant v=1 suffices? Or needs to be changed every release ?

@blizzz
Copy link
Member

blizzz commented Jun 14, 2016

ah, now i read it in the related issue. Ok for s one-time thing, but too error prone as a long term solution. since it's about 9, LGTM

@LukasReschke
Copy link
Member Author

a constant v=1 suffices? Or needs to be changed every release ?

All the time the images get changed … 🙈

@MorrisJobke
Copy link
Member

Makes sense and works 👍

@MorrisJobke MorrisJobke merged commit b308b95 into stable9 Jun 14, 2016
@MorrisJobke MorrisJobke deleted the add-some-basic-cache-busting branch June 14, 2016 08:57
@MorrisJobke
Copy link
Member

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants