Skip to content

Add some words about the Radicale integration#279

Merged
rhafer merged 4 commits intoopencloud-eu:mainfrom
rhafer:radicale
May 13, 2025
Merged

Add some words about the Radicale integration#279
rhafer merged 4 commits intoopencloud-eu:mainfrom
rhafer:radicale

Conversation

@rhafer
Copy link
Copy Markdown
Member

@rhafer rhafer commented May 8, 2025

Copilot AI review requested due to automatic review settings May 8, 2025 11:03
@rhafer rhafer requested a review from tbsbdr May 8, 2025 11:03
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds documentation for integrating Radicale for Calendar and Contacts support in OpenCloud.

  • Added a new documentation file outlining the configuration and usage of Radicale integration.
  • Included steps for updating the deployment and configuring the .env file.
Comments suppressed due to low confidence (1)

docs/admin/configuration/radicale-integration.md:48

  • Typo detected: 'an Calendar' should be corrected to 'and Calendar'.
Users should now be able to access their Addressbook an Calendar with any capable CalDAV/CardDAV software (OpenCloud does currently not provide any UI to access them).

Comment thread docs/admin/configuration/radicale-integration.md Outdated
Comment thread docs/admin/configuration/radicale-integration.md Outdated
@rhafer
Copy link
Copy Markdown
Member Author

rhafer commented May 8, 2025

@tbsbdr I'd appreciate if you could add some wording about the target audience/support status of the Radicale Integration.

Comment thread docs/admin/configuration/radicale-integration.md Outdated
Comment thread docs/admin/configuration/radicale-integration.md Outdated
pReya
pReya previously requested changes May 8, 2025
Comment thread docs/admin/configuration/radicale-integration.md Outdated
@rhafer rhafer requested review from AlexAndBear and Svanvith May 9, 2025 06:01
@rhafer rhafer dismissed pReya’s stale review May 9, 2025 07:50

Addressed

@tbsbdr
Copy link
Copy Markdown
Member

tbsbdr commented May 9, 2025

here is the statement for howm the integration is and if we support it within a business licence: opencloud-eu/opencloud#206 (comment)

Comment thread docs/admin/configuration/radicale-integration.md Outdated

As very few clients currently support authentication via OpenID Connect you can use
the "App Token" feature in the user's settings page to generate an application specific
password that can be used for authentication in most existing CalDAV/CardDAV clients.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Could you add a few more sentences on how to create the app token, or link a doc?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

What I meant to say: Yes, App tokens need documentation. But I think that shouldn't hold up this pull request.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@Svanvith now that we have an actual use-case, can you and your team take care ?

---

OpenCloud is able to act as an authenticating reverse proxy for the open-source
CalDAV and CardDAV Server [Radicale](https://radicale.org).
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Maybe add something like

With that, users get a calender- and contacts server out of the box with OpenCloud that they can use with a wide selection of desktop- and mobile clients without hassle.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

✔️

Comment thread docs/admin/configuration/radicale-integration.md Outdated
Copy link
Copy Markdown
Member

@AlexAndBear AlexAndBear left a comment

Choose a reason for hiding this comment

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

👍

@rhafer rhafer merged commit 9109d1a into opencloud-eu:main May 13, 2025
1 check passed
@openclouders openclouders mentioned this pull request May 13, 2025
1 task
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.

7 participants