Skip to content

chore(core): update VCard filename fix - #3080

Merged
FabianDevel merged 1 commit into
mainfrom
fabiandevel-fix-vcard-filename
Sep 22, 2026
Merged

FabianDevel merged 1 commit into
mainfrom
fabiandevel-fix-vcard-filename

Conversation

@FabianDevel

Copy link
Copy Markdown
Contributor

Description

  • Update the Core submodule to include the ContactCard VCard filename fix.
  • Remove the unrelated attachments_cache prefix from exported VCard filenames.

Depends on Infomaniak/android-core#862.

Validation

  • ./gradlew :app:compileStandardDebugKotlin --stacktrace

Copilot AI balanced review requested due to automatic review settings September 21, 2026 14:47
@github-actions

Copy link
Copy Markdown

This PR/issue depends on:

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The focused fix is tested upstream and preserves the existing FileProvider-compatible cache location.

Review effort: Balanced
Findings: None

What changed in this PR

Updates the Core submodule to deliver corrected exported VCard filenames.

Changes:

  • Removes the unintended attachments_cache filename prefix.
  • Includes upstream filename tests and intervening Core updates.
File Description
Core Advances the submodule to upstream commit f8056d2.

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

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@FabianDevel
FabianDevel force-pushed the fabiandevel-fix-vcard-filename branch from d342662 to caf29d2 Compare September 22, 2026 08:37
@sonarqubecloud

Copy link
Copy Markdown

@FabianDevel
FabianDevel merged commit a33d70e into main Sep 22, 2026
10 checks passed
@FabianDevel
FabianDevel deleted the fabiandevel-fix-vcard-filename branch September 22, 2026 08:43
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.

3 participants