Skip to content

[stable28] fix(files): Properly handle denied ownership transfers - #45905

Merged
susnux merged 1 commit into
stable28from
backport/45884/stable28
Jun 25, 2024
Merged

[stable28] fix(files): Properly handle denied ownership transfers#45905
susnux merged 1 commit into
stable28from
backport/45884/stable28

Conversation

@backportbot

Copy link
Copy Markdown

Backport of PR #45884

@susnuxsusnux added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jun 17, 2024
@blizzzblizzz mentioned this pull request Jun 17, 2024
9 tasks
@blizzzblizzz mentioned this pull request Jun 25, 2024
When the receiver denies the transfer the notification handler was missing,
so no notification was created for the transfer owner.
But also the internal notification was created two times:
1. When rejecting the transfer
2. By the reject function when dismissing the notification
This is fixed by only relying on the dismiss function.
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux
susnuxforce-pushed the backport/45884/stable28 branch from 90b85f3 to 320283bCompareJune 25, 2024 16:01
@susnux
susnux merged commit d126e17 into stable28Jun 25, 2024
@susnux
susnux deleted the backport/45884/stable28 branch June 25, 2024 19:27
@AltahrimAltahrim mentioned this pull request Jul 10, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finishbugfeature: files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@SystemKeeper@susnux@blizzz