Skip to content

fix(notifications): Use the target id in guest conversion notifications - #1669

Merged
CarlSchwan merged 1 commit into
nextcloud:mainfrom
solracsf:fix/transfer-notification-target-id
Aug 21, 2026
Merged

fix(notifications): Use the target id in guest conversion notifications#1669
CarlSchwan merged 1 commit into
nextcloud:mainfrom
solracsf:fix/transfer-notification-target-id

Conversation

@solracsf

Copy link
Copy Markdown
Member

The rich object describing the converted-to account fell back to the guest id when the target account could not be resolved, so a failed conversion reported the guest twice instead of naming the account it was supposed to become.

@solracsf solracsf added bug Something isn't working 3. to review Waiting for reviews labels Aug 16, 2026
The rich object describing the converted-to account fell back to the
guest id when the target account could not be resolved, so a failed
conversion reported the guest twice instead of naming the account it was
supposed to become.

Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
@CarlSchwan
CarlSchwan force-pushed the fix/transfer-notification-target-id branch from a5ea525 to 00b52d8 Compare August 21, 2026 08:34
@CarlSchwan
CarlSchwan enabled auto-merge August 21, 2026 08:34
@CarlSchwan
CarlSchwan merged commit b671349 into nextcloud:main Aug 21, 2026
55 checks passed
@solracsf
solracsf deleted the fix/transfer-notification-target-id branch August 21, 2026 09:13
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 Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants