Skip to content

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented May 29, 2025

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.

@backportbot backportbot bot requested review from artonge, skjnldsv and susnux May 29, 2025 00:06
@backportbot backportbot bot added this to the Nextcloud 31.0.6 milestone May 29, 2025
@susnux susnux force-pushed the backport/53171/stable31 branch from 86dd159 to 24bcb54 Compare May 31, 2025 17:07
@susnux susnux marked this pull request as ready for review May 31, 2025 17:07
@AndyScherzinger AndyScherzinger force-pushed the backport/53171/stable31 branch from 24bcb54 to 89ac098 Compare June 1, 2025 11:45
@AndyScherzinger
Copy link
Member

/compile /

@AndyScherzinger AndyScherzinger enabled auto-merge June 1, 2025 11:46
susnux and 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 AndyScherzinger force-pushed the backport/53171/stable31 branch from e5c7517 to 1b6b357 Compare June 1, 2025 13:42
@AndyScherzinger AndyScherzinger merged commit 3c6dfa9 into stable31 Jun 1, 2025
128 of 130 checks passed
@AndyScherzinger AndyScherzinger deleted the backport/53171/stable31 branch June 1, 2025 16:09
@nextcloud-bot nextcloud-bot mentioned this pull request Jun 4, 2025
10 tasks
Sign up for free to 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