Skip to content

Conversation

@juliusknorr
Copy link
Member

Possible fix for #14615

I could not reliably reproduce the bug bug I encountered that there might be cases where the first request encounters that the scss variables have changed, clears the cache and during the same time another request does the same. This PR makes sure that the cache cleanup is only running once.

Signed-off-by: Julius Härtl <jus@bitgrid.net>
@juliusknorr juliusknorr added bug 3. to review Waiting for reviews labels Oct 15, 2019
@juliusknorr juliusknorr changed the title Bugfix/ Let SCSS cleanup only run once Oct 15, 2019
@juliusknorr juliusknorr added this to the Nextcloud 18 milestone Oct 15, 2019
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Copy link
Member

@rullzer rullzer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sure

@ChristophWurst ChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Oct 17, 2019
@skjnldsv skjnldsv merged commit bd5189f into master Oct 17, 2019
@skjnldsv skjnldsv deleted the bugfix/14615 branch October 17, 2019 08:03
@juliusknorr
Copy link
Member Author

/backport to stable17

@juliusknorr
Copy link
Member Author

/backport to stable16

@backportbot-nextcloud
Copy link

backport to stable17 in #17579

@backportbot-nextcloud
Copy link

backport to stable16 in #17580

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants