Skip to content

feat: enable EnableInsertRemoteFile WOPI flag for Collabora#2555

Open
pedropintosilva wants to merge 1 commit intoopencloud-eu:mainfrom
pedropintosilva:feat/enable-insert-remote-file
Open

feat: enable EnableInsertRemoteFile WOPI flag for Collabora#2555
pedropintosilva wants to merge 1 commit intoopencloud-eu:mainfrom
pedropintosilva:feat/enable-insert-remote-file

Conversation

@pedropintosilva
Copy link
Copy Markdown

Set EnableInsertRemoteFile: true in CheckFileInfo response so Collabora
shows "Insert Multimedia" and "Compare Document" menu entries. This
triggers UI_InsertFile postMessages to the WOPI host, following the same
pattern as the existing EnableInsertRemoteImage flag.

Signed-off-by: Pedro Pinto Silva pedro.silva@collabora.com

Screenshots (if appropriate):

opencloud-compare-documents

Set EnableInsertRemoteFile: true in CheckFileInfo response so Collabora
shows "Insert Multimedia" and "Compare Document" menu entries. This
triggers UI_InsertFile postMessages to the WOPI host, following the same
pattern as the existing EnableInsertRemoteImage flag.

Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
C Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@pedropintosilva
Copy link
Copy Markdown
Author

client-side: opencloud-eu/web#2270

@pedropintosilva
Copy link
Copy Markdown
Author

I cannot see what are the specific complains from SonarCloud , I'm not part of the org so, I get some error when attempting to open it in my SonarCloud account.

Copy link
Copy Markdown
Member

@JammingBen JammingBen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM and works fine from my point of view.

IMO the SonarCloud error can be neglected in this case. @rhafer how do you handle these kind of things at the moment?

@pedropintosilva
Copy link
Copy Markdown
Author

pedropintosilva commented Apr 2, 2026

Thanks @JammingBen! It seems the workflows will not run until some sort of approval:

2 workflows awaiting approval

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants