Skip to content

fix(StorageService): Use OCP\..\FileAccess interface - #142

Merged
marcelklehr merged 12 commits into
mainfrom
fix/use-filaccess
Jul 17, 2025
Merged

fix(StorageService): Use OCP\..\FileAccess interface#142
marcelklehr merged 12 commits into
mainfrom
fix/use-filaccess

Conversation

@marcelklehr

@marcelklehrmarcelklehr commented Jul 14, 2025

Copy link
Copy Markdown
Member

for enumerating mounts and nodes

see nextcloud/server#51810

for enumerating mounts and nodes
Signed-off-by: Marcel Klehr <mklehr@gmx.net>

@kyteinskykyteinsky left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this PR is just for testing or?

Comment thread.github/workflows/integration-test.yml Outdated
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
@marcelklehr

Copy link
Copy Markdown
MemberAuthor

this PR is just for testing or?

this PR is not just for testing, but we should revert the CI branch change before merging

instead of strings
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
Comment threadlib/BackgroundJobs/StorageCrawlJob.php Outdated
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
not available on stable30
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
Signed-off-by: Marcel Klehr <mklehr@gmx.net>

@kyteinskykyteinsky left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🚀

@marcelklehr
marcelklehr merged commit 20e6956 into mainJul 17, 2025
26 checks passed
@marcelklehr
marcelklehr deleted the fix/use-filaccess branch July 17, 2025 07:20
@kyteinskykyteinsky mentioned this pull request Jul 21, 2025
Merged
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@marcelklehr@kyteinsky