Skip to content

fix(cardDav): Only update user card on actual mapped propreties changes - #63766

Merged
hamza221 merged 1 commit into
masterfrom
fix/carddav/userUpdate
Aug 28, 2026
Merged

fix(cardDav): Only update user card on actual mapped propreties changes#63766
hamza221 merged 1 commit into
masterfrom
fix/carddav/userUpdate

Conversation

@hamza221

@hamza221 hamza221 commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Reproduction

  1. Edit a user untracked property such as quota or pronouns
  2. Look at oc_addressbookchanges

Expected

No unnecessary row is added

Actual

An unnecessary row is added

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

Signed-off-by: Hamza <hamzamahjoubi221@gmail.com>
@hamza221
hamza221 requested a review from mgallien August 28, 2026 11:18
@hamza221 hamza221 self-assigned this Aug 28, 2026
@hamza221 hamza221 added 3. to review Waiting for reviews feature: carddav Related to CardDAV internals labels Aug 28, 2026
@hamza221

Copy link
Copy Markdown
Contributor Author

/backport to stable35

@hamza221

Copy link
Copy Markdown
Contributor Author

/backport to stable34

@hamza221

Copy link
Copy Markdown
Contributor Author

/backport to stable33

@hamza221 hamza221 added the bug label Aug 28, 2026
@hamza221 hamza221 added this to the Nextcloud 36 milestone Aug 28, 2026

@ChristophWurst ChristophWurst left a comment

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.

propreties

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug feature: carddav Related to CardDAV internals

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants