Shallow scanning of specific paths

[steffenclemens]

Adding feature requested in #1168.
Close #1168

[solracsf]

Thanks. Can you fix DCO?

[zak39]

Hi @steffenclemens 🙂

To fix the DCO, you can follow these steps 👉 https://github.com/nextcloud/groupfolders/pull/2232/checks?check_run_id=10651495499

[steffenclemens]

Hi @steffenclemens 🙂

To fix the DCO, you can follow these steps 👉 https://github.com/nextcloud/groupfolders/pull/2232/checks?check_run_id=10651495499

Hi @solracsf and @zak39, thanks for the hint.

[steffenclemens]

Hi @solracsf, @zak39. Is anything left for me to do here?

[solracsf]

No, wait for reviews and merge. ⏳

[steffenclemens]

No, wait for reviews and merge. ⏳

Alright, thanks! 🙂

[icewind1991]

looks good otherwise

[Ornanovitch]

Sorry to bump this but is there any hope to see this implemented soon? This would be a really great improvement..

[steffenclemens]

Sorry to bump this but is there any hope to see this implemented soon? This would be a really great improvement..

Thanks for the reminder! I added the required commit.

[Ornanovitch]

Thanks :)

Will it work with scanning a specific file, like occ files:scan --path?

[steffenclemens]

Thanks :)

Will it work with scanning a specific file, like occ files:scan --path?

You can scan files of a specific directory without stepping down in the tree using: occ groupfolders:scan <folderid> -p /path/to/your/directory --shallow.
So this will only scan files that are directly in /path/to/your/directory.
Use occ groupfolders:list to find the folder id.

[steffenclemens]

@Ornanovitch occ groupfolders:scan -h shows you all options with description.

[Ornanovitch]

Note that it works with a single file 🥳

[Ornanovitch]

@icewind1991 any chance to merge this awesome feature?

Addressed

[skjnldsv]

@Ornanovitch & @steffenclemens : I rebased and pinged more devs for this :)
cc @icewind1991, please address your review again 🙇

[provokateurin]

Sorry, we don't backport features.

[Ornanovitch]

I understand but does it mean this feature won't be able before NC 31? That would be unexpected, because it was ready before the milestone even exist.. :/

[provokateurin]

It will be available for NC 31, but not 30 or any other existing version.

[provokateurin]

It was just merged too later after the branch off.