Skip to content

[stable31] fix(files): do nothing if view local dialog was just closed - #53177

Merged
AndyScherzinger merged 2 commits into
stable31from
backport/53171/stable31
Jun 1, 2025
Merged

[stable31] fix(files): do nothing if view local dialog was just closed#53177
AndyScherzinger merged 2 commits into
stable31from
backport/53171/stable31

Conversation

@backportbot

@backportbotbackportbotBot commented May 29, 2025

Copy link
Copy Markdown

Backport of #53171

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

Todo

  • Review and resolve any conflicts
  • Remove all the empty commits

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

@backportbotbackportbotBot added this to the Nextcloud 31.0.6 milestone May 29, 2025
@susnux
susnuxforce-pushed the backport/53171/stable31 branch from 86dd159 to 24bcb54CompareMay 31, 2025 17:07
@susnux
susnux marked this pull request as ready for review May 31, 2025 17:07
@AndyScherzinger
AndyScherzingerforce-pushed the backport/53171/stable31 branch from 24bcb54 to 89ac098CompareJune 1, 2025 11:45
@AndyScherzinger

Copy link
Copy Markdown
Member

/compile /

susnuxand others added 2 commits June 1, 2025 15:42
We try to open a file in the Nextcloud client.
If this fails a dialog is shown with 3 options:
1. Retry: If it fails no further dialog is shown.
2. Open online: The viewer is used to open the file.
3. Close the dialog and nothing happens (abort).
This correctly implements 3 and also adds some comments + order file in
reading order (using `function` instead of arrow functions allows this
easily).
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@AndyScherzinger
AndyScherzingerforce-pushed the backport/53171/stable31 branch from e5c7517 to 1b6b357CompareJune 1, 2025 13:42
@AndyScherzinger
AndyScherzinger merged commit 3c6dfa9 into stable31Jun 1, 2025
@AndyScherzinger
AndyScherzinger deleted the backport/53171/stable31 branch June 1, 2025 16:09
@nextcloud-botnextcloud-bot mentioned this pull request Jun 4, 2025
10 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.

3 participants

@AndyScherzinger@susnux@nextcloud-command