Skip to content

Recursive bucket sync and delete actions#231

Merged
TimCsaky merged 2 commits intomasterfrom
sync-on-connect
Apr 2, 2025
Merged

Recursive bucket sync and delete actions#231
TimCsaky merged 2 commits intomasterfrom
sync-on-connect

Conversation

@TimCsaky
Copy link
Contributor

@TimCsaky TimCsaky commented Mar 27, 2025

Description

  • Trigger recursive sync when connecting a new storage location

    • Minimal implementation of the COMS recursive sync feature. When user connects a new storage location, trigger a sync. More advanced progress indicator and 'auto-syncing' to follow in a future commit.
  • Do a recursive delete when deleting a folder (bucket)
    this will require COMS pr 297 to be merged first)

Types of changes

New feature (non-breaking change which adds functionality)

Checklist

  • I have read the CONTRIBUTING doc
  • I have checked that unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

Further comments

@github-actions
Copy link

Coverage Report (Application)

Totals Coverage
Statements: 70.67% ( 53 / 75 )
Methods: 62.5% ( 5 / 8 )
Lines: 81.63% ( 40 / 49 )
Branches: 44.44% ( 8 / 18 )

@github-actions
Copy link

github-actions bot commented Mar 27, 2025

Coverage Report (Frontend)

Totals Coverage
Statements: 16.86% ( 785 / 4657 )
Methods: 17.26% ( 151 / 875 )
Lines: 20.68% ( 544 / 2630 )
Branches: 7.81% ( 90 / 1152 )

@TimCsaky TimCsaky requested review from norrisng-bc and removed request for norrisng-bc March 27, 2025 23:52
@TimCsaky TimCsaky changed the title Trigger recursive sync when connecting a new storage location Recursive bucket sync and delete actions Apr 2, 2025
@TimCsaky TimCsaky merged commit 7986aae into master Apr 2, 2025
16 of 17 checks passed
Sign up for free to 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