Skip to content

fixes handling multiple arguments in Storage.discovery()#34

Open
mab122 wants to merge 1 commit into
39aldo39:masterfrom
mab122:fix/storage_discovery_arguments_mismatch
Open

fixes handling multiple arguments in Storage.discovery()#34
mab122 wants to merge 1 commit into
39aldo39:masterfrom
mab122:fix/storage_discovery_arguments_mismatch

Conversation

@mab122

@mab122 mab122 commented Oct 25, 2024

Copy link
Copy Markdown

Seems like radicale added support for groups and is now passing one more argument in radicale/storage/multifilesystem/discover.py. I've added missing argument to decsync.storage but nothing else so this does not implement GROUPS/ support.

@pudup

pudup commented Nov 23, 2024

Copy link
Copy Markdown

Unusable on Fedora 41 without this PR. Have to add it manually until this is merged

@Nino-Bock

Copy link
Copy Markdown

Awesome, thank you so much for this fix.
Please merge the PR. I tested it on Arch, via a venv install, not AUR and it works wonderfully.

@Alerymin

Copy link
Copy Markdown

I need this PR too on Fedora 41

gotlougit added a commit to gotlougit/nix-config that referenced this pull request Jun 22, 2025
decsync allows syncing calendar data over Syncthing,
radicale is a CalDAV server running locally

ref: 39aldo39/Radicale-DecSync#34
Thanks @mab122
@waltmck

waltmck commented Aug 25, 2025

Copy link
Copy Markdown

I have tested this patch and confirmed that it fixes the issue.

@AminBlg

AminBlg commented Sep 8, 2025

Copy link
Copy Markdown

fixes, please merge

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.

6 participants