Skip to content

[stable30] fix(files_sharing): Add proper user facing messages on success - #48730

Merged
susnux merged 2 commits into
stable30from
backport/48696/stable30
Oct 21, 2024
Merged

[stable30] fix(files_sharing): Add proper user facing messages on success#48730
susnux merged 2 commits into
stable30from
backport/48696/stable30

Conversation

@backportbot

@backportbotbackportbotBot commented Oct 16, 2024

Copy link
Copy Markdown

Backport of #48696

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

1. Use proper translated strings
2. Fix invalid use of translation message on API result
3. Log all errors
Co-authored-by: Ferdinand Thiessen <opensource@fthiessen.de>
Co-authored-by: Pytal <24800714+Pytal@users.noreply.github.com>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux
susnuxforce-pushed the backport/48696/stable30 branch from 5c69635 to ef8547bCompareOctober 17, 2024 16:10
@susnux
susnux marked this pull request as ready for review October 17, 2024 16:10
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@susnux
susnux requested a review from nfebeOctober 19, 2024 13:44
@susnux
susnux enabled auto-merge October 21, 2024 10:20
@susnux
susnux merged commit be931b7 into stable30Oct 21, 2024
@susnux
susnux deleted the backport/48696/stable30 branch October 21, 2024 10:25
@AltahrimAltahrim mentioned this pull request Oct 30, 2024
2 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@susnux@nfebe@blizzz@nextcloud-command