Skip to content

feat: Caldav library configuration - #129

Open
benjaminVadon wants to merge 3 commits into
mainfrom
caldav-library-configuration
Open

feat: Caldav library configuration#129
benjaminVadon wants to merge 3 commits into
mainfrom
caldav-library-configuration

Conversation

@benjaminVadon

Copy link
Copy Markdown
Contributor

Add support of caldav library configuration in debug build

Copilot AI 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.

Pull request overview

Adds configurable CalDAV client behavior, including Android debug proxy interception and connection reuse.

Changes:

  • Adds Kotlin and UniFFI configuration APIs.
  • Caches clients per account.
  • Enables debug-only interception and upgrades fast-dav-rs.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
CaldavClientConfig.kt Exposes Kotlin configuration types.
lib.rs Registers the configuration module.
config.rs Implements configuration, caching, and interception.
client.rs Uses cached clients.
Cargo.toml Adds feature/profile and updates dependency.
build.gradle.kts Enables interception for Android debug builds.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread CalendarKmpDav/rust/caldav_bridge/src/config.rs
Comment thread CalendarKmpDav/rust/caldav_bridge/src/config.rs Outdated
@benjaminVadon
benjaminVadon force-pushed the caldav-library-configuration branch 9 times, most recently from 271b7c6 to b1f5286 Compare August 24, 2026 11:10
@benjaminVadon
benjaminVadon force-pushed the caldav-library-configuration branch from b1f5286 to cc633ab Compare August 24, 2026 13:03
@benjaminVadon
benjaminVadon force-pushed the caldav-library-configuration branch from cc633ab to 06d30b0 Compare August 24, 2026 13:11
@sonarqubecloud

Copy link
Copy Markdown

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