feat(fileprovider): client integration with server features#9458
Closed
feat(fileprovider): client integration with server features#9458
Conversation
- declarativeui: add endpoints model. - capabilities: fetch endpoints in capabilities. - display endpoints in an application window. - list file actions in context menu. - remove EndpointModel from DeclarativeUi class. - list declarative ui and file actions in the tray activitiy. - add icon and filter to endpoint model and UI. Signed-off-by: Camila Ayres <hello@camilasan.com>
- display response from request from file actions. Signed-off-by: Camila Ayres <hello@camilasan.com>
Signed-off-by: Camila Ayres <hello@camilasan.com>
- add helper function to match string to SimpleApiJob::Verb. - filter and display file actions based on the file mimetype. - adjust logic to get context menu by mimeType. - create function to set file id and mime type. - rename Endpoint to FileActions. Signed-off-by: Camila Ayres <hello@camilasan.com>
- improve error handling. - display a different icon for each mimetype. Signed-off-by: Camila Ayres <hello@camilasan.com>
…n't have one. - improve file actions window UI. Signed-off-by: Camila Ayres <hello@camilasan.com>
…ements like url. - add opacity mask and more spacing. - implement hover for the file action buttons. - fix logic to display error/success messages. - improve text for error/success messages. Signed-off-by: Camila Ayres <hello@camilasan.com>
The declarative ui in fact will be returned in the response from file actions. Signed-off-by: Camila Ayres <hello@camilasan.com>
Signed-off-by: Camila Ayres <hello@camilasan.com>
- use reference type in for loop. - fix spaces and sizes in the file actions window. - remove missing file from resources. Signed-off-by: Camila Ayres <hello@camilasan.com>
…etype check. Signed-off-by: Camila Ayres <hello@camilasan.com>
Signed-off-by: Camila Ayres <hello@camilasan.com>
…lable to all mimetypes. Signed-off-by: Camila Ayres <hello@camilasan.com>
Signed-off-by: Camila Ayres <hello@camilasan.com>
- remove DeclarativeUi files. - add return default value in switch. - add const. Signed-off-by: Camila Ayres <hello@camilasan.com>
…on map. Signed-off-by: Camila Ayres <hello@camilasan.com>
…port it. Signed-off-by: Camila Ayres <hello@camilasan.com>
Signed-off-by: Camila Ayres <hello@camilasan.com>
Signed-off-by: Camila Ayres <hello@camilasan.com>
Signed-off-by: Camila Ayres <hello@camilasan.com>
…ize. Signed-off-by: Camila Ayres <hello@camilasan.com>
Signed-off-by: Camila Ayres <hello@camilasan.com>
Signed-off-by: Camila Ayres <hello@camilasan.com>
- Done by codex. - Added a new File Provider context-menu action (FileActionsAction) in the File Provider UI extension plist, gated by userInfo.displayFileActions, so Finder can show a File actions menu entry for provider-backed items. - Wired DocumentActionViewController to handle that new action: it resolves the selected item to a user-visible local path and calls into the extension service to request opening file actions in the main app. - Extended the File Provider extension service protocol/source with a new openFileActions(forLocalPath:) method, and forwarded it to the app-facing XPC protocol (AppProtocol). - Implemented the new app-side XPC method in fileproviderservice.mm to spawn the desktop client with a dedicated CLI option and pass the target local path to open the existing Qt file-actions dialog path flow. - Added --open-file-actions <path> handling in Application, persisted the path in a new member, and trigger-opened the existing slotShowFileActionsDialog both during initial startup and for single-instance message parsing paths. Signed-off-by: Camila Ayres <hello@camilasan.com>
|
Artifact containing the AppImage: nextcloud-appimage-pr-9458.zip Digest: 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. |
|
Collaborator
|
Just looking at the files touched I think Codex did a lot of nonsense. This should be much simpler. As written in direct chat:
|
Collaborator
|
Closed in favor of #9469. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Alternative solution (done by codex 🤖) to display integration in file provider using xpc:
See socket solution option at #9461.
Related to #9403.