close
Skip to content

fix(file-provider): Existing Lock Comparison#9062

Merged
i2h3 merged 1 commit into
masterfrom
i2h3/fix/9686311-locking
Nov 14, 2025
Merged

fix(file-provider): Existing Lock Comparison#9062
i2h3 merged 1 commit into
masterfrom
i2h3/fix/9686311-locking

Conversation

@i2h3
Copy link
Copy Markdown
Collaborator

@i2h3 i2h3 commented Nov 13, 2025

Depends on #9059 being merged first.

@i2h3 i2h3 added this to the 4.0.2 milestone Nov 13, 2025
@i2h3 i2h3 self-assigned this Nov 13, 2025
@i2h3 i2h3 requested a review from Aiiaiiio as a code owner November 13, 2025 15:55
@i2h3 i2h3 added the bug label Nov 13, 2025
@i2h3 i2h3 added the feature: 📁 file provider macOS File Provider Extension, more general also known as virtual file system. label Nov 13, 2025
@github-project-automation github-project-automation Bot moved this to 🧭 Planning evaluation (don't pick) in 💻 Desktop Clients team Nov 13, 2025
@i2h3 i2h3 force-pushed the i2h3/fix/9686311-locking branch 2 times, most recently from 881e8cc to b5dbbc0 Compare November 14, 2025 11:21
@i2h3
Copy link
Copy Markdown
Collaborator Author

i2h3 commented Nov 14, 2025

/backport to stable-4.0

@i2h3 i2h3 moved this from 🧭 Planning evaluation (don't pick) to 🏗️ In progress in 💻 Desktop Clients team Nov 14, 2025
@i2h3 i2h3 enabled auto-merge November 14, 2025 11:54
@github-actions
Copy link
Copy Markdown
Contributor

Artifact containing the AppImage: nextcloud-appimage-pr-9062.zip

Digest: sha256:9ab78ac47b1486c3936ba0ee7404ebf678063664e5fcfb2c353819821a71eb7e

To test this change/fix you can download the above artifact file, unzip it, and run it.

Please make sure to quit your existing Nextcloud app and backup your data.

- Some time ago, I implemented the token based locks. In scope of that I temporarily changed the "lock" property to a complex type before reverting it. I probably forgot to turn back the changes here which compare the property like it is an optional struct even though it is a boolean.
- Improved code formatting and updated some calls to use asynchronous methods of NextcloudKit directly without continuations.
- Set the key equivalent of the close button to the escape key.

Signed-off-by: Iva Horn <iva.horn@nextcloud.com>
@i2h3 i2h3 force-pushed the i2h3/fix/9686311-locking branch from b5dbbc0 to e2b35a7 Compare November 14, 2025 14:06
@i2h3 i2h3 disabled auto-merge November 14, 2025 14:07
@i2h3 i2h3 merged commit ace2585 into master Nov 14, 2025
14 of 16 checks passed
@i2h3 i2h3 deleted the i2h3/fix/9686311-locking branch November 14, 2025 14:07
@github-project-automation github-project-automation Bot moved this from 🏗️ In progress to ☑️ Done in 💻 Desktop Clients team Nov 14, 2025
@sonarqubecloud
Copy link
Copy Markdown

@mgallien mgallien modified the milestones: 4.0.2, 4.1.0 Nov 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug feature: 📁 file provider macOS File Provider Extension, more general also known as virtual file system.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants