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

Let SCSS cleanup only run once #17543

Merged
merged 2 commits into from
Oct 17, 2019
Merged

Let SCSS cleanup only run once #17543

merged 2 commits into from
Oct 17, 2019

Conversation

juliushaertl
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.

@juliushaertl juliushaertl added bug 3. to review Waiting for reviews labels Oct 15, 2019
@juliushaertl juliushaertl changed the title Bugfix/ Let SCSS cleanup only run once Oct 15, 2019
@juliushaertl juliushaertl added this to the Nextcloud 18 milestone Oct 15, 2019
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
@juliushaertl
Copy link
Member Author

/backport to stable17

@juliushaertl
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.

5 participants