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

Enhancement: improve the wopi app driver #3291

Merged
merged 1 commit into from
Oct 4, 2022

Conversation

micbar
Copy link
Member

@micbar micbar commented Sep 29, 2022

Description

The wopi app driver supports more options

We now generate a folderurl that is used in the wopi protocol to provide an endpoint to go back from the app to the containing folder in the file list. In addition to that we now inlclude the UI_LLCC parameter in the app open URL.

https://learn.microsoft.com/en-us/microsoft-365/cloud-storage-partner-program/online/discovery#ui_llcc

@update-docs
Copy link

update-docs bot commented Sep 29, 2022

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.

@micbar micbar force-pushed the wopi-driver-improvements branch from e17c730 to fb28bb6 Compare September 29, 2022 15:07
@micbar micbar requested a review from a team as a code owner September 29, 2022 15:07
@micbar micbar force-pushed the wopi-driver-improvements branch 2 times, most recently from 0c0441f to 8424a05 Compare September 29, 2022 15:10
@micbar micbar marked this pull request as draft September 29, 2022 21:03
@glpatcern
Copy link
Member

This is also a welcome improvement in master. How are you going to generate the PrivateLinkBaseURL?

@micbar micbar force-pushed the wopi-driver-improvements branch 2 times, most recently from be4cc9e to ad80d34 Compare September 30, 2022 21:02
@micbar micbar marked this pull request as ready for review September 30, 2022 21:06
@micbar micbar force-pushed the wopi-driver-improvements branch from ad80d34 to d1ec2bf Compare October 1, 2022 19:11
@micbar
Copy link
Member Author

micbar commented Oct 1, 2022

How are you going to generate the PrivateLinkBaseURL?

I plan to use the private link which not be user facing in the future in ocis but ownCloud Web will redirect.

@micbar micbar force-pushed the wopi-driver-improvements branch 2 times, most recently from 0a5e2a7 to 5d9efbb Compare October 4, 2022 09:10
@micbar micbar requested a review from kobergj October 4, 2022 09:10
@micbar micbar force-pushed the wopi-driver-improvements branch from 5d9efbb to 9415263 Compare October 4, 2022 09:11
@micbar micbar requested a review from kobergj October 4, 2022 11:15
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