Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix opening a readonly filetype with WOPI #2073

Merged
merged 4 commits into from
Sep 27, 2021
Merged

Fix opening a readonly filetype with WOPI #2073

merged 4 commits into from
Sep 27, 2021

Conversation

wkloucek
Copy link
Contributor

@wkloucek wkloucek commented Sep 15, 2021

Eg. JPG or PNG are supported to be opened by Collabora - but in view mode only. "/app/open" request currently fails because the WOPI driver assuems that an edit vie mode always exists.

@update-docs
Copy link

update-docs bot commented Sep 15, 2021

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@wkloucek wkloucek changed the title fix opening a file with wopi driver when the office suite only supports viewing the filetype Fix opening a readonly filetype with WOPI Sep 15, 2021
@wkloucek wkloucek marked this pull request as ready for review September 15, 2021 15:33
glpatcern
glpatcern previously approved these changes Sep 16, 2021
Copy link
Member

@glpatcern glpatcern left a comment

Choose a reason for hiding this comment

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

Looks good, and also in line with the discussion in cs3org/cs3apis#145.

Just fix the reference in the changelog (2073 instead of 2051), and once the above PR is included we can also expose the capability for such cases where only a viewURL exists.

@glpatcern glpatcern self-requested a review September 23, 2021 12:47
glpatcern
glpatcern previously approved these changes Sep 23, 2021
Copy link
Member

@glpatcern glpatcern left a comment

Choose a reason for hiding this comment

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

I believe this can go ahead so we bundle it with the other ongoing modifications in the AppProvider area

pkg/app/provider/wopi/wopi.go Outdated Show resolved Hide resolved
pkg/app/provider/wopi/wopi.go Show resolved Hide resolved
@ishank011 ishank011 merged commit 83a8848 into cs3org:master Sep 27, 2021
glpatcern pushed a commit to glpatcern/reva that referenced this pull request Nov 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants