Skip to content

clear filecache cache manually#13

Closed
butonic wants to merge 1 commit into
masterfrom
clear-cache-manually
Closed

clear filecache cache manually#13
butonic wants to merge 1 commit into
masterfrom
clear-cache-manually

Conversation

@butonic

@butonic butonic commented Sep 6, 2017

Copy link
Copy Markdown
Contributor

needed for and requires owncloud/core@3783534 because the migration manipulates the db filecache directly. bypassing the core implementation, which would take care of cleaning the cache

@butonic butonic added this to the development milestone Sep 6, 2017
@butonic butonic self-assigned this Sep 6, 2017
@butonic butonic requested a review from tomneedham September 6, 2017 07:54
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@PVince81

PVince81 commented Sep 6, 2017

Copy link
Copy Markdown
Contributor

why not put this small commit directly into owncloud/core#28166 ?

@butonic

butonic commented Sep 6, 2017

Copy link
Copy Markdown
Contributor Author

@PVince81 because it is a different app?

@PVince81

PVince81 commented Sep 6, 2017

Copy link
Copy Markdown
Contributor

🤦‍♂️ you're right... I forgot that encryption app is separate on master

@felixboehm

Copy link
Copy Markdown

@tomneedham please review. blocking QA / release

@tomneedham

Copy link
Copy Markdown
Contributor

This PR is only required if the core PR: owncloud/core#28166 is merged in - else it can be ignored

@tomneedham tomneedham modified the milestones: development, planned Oct 17, 2017
@butonic

butonic commented Feb 18, 2019

Copy link
Copy Markdown
Contributor Author

closing in favor of OCIS

@butonic butonic closed this Feb 18, 2019
@butonic butonic deleted the clear-cache-manually branch February 18, 2019 10:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants