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

Optionally split space from endpoint #79

Merged
merged 3 commits into from
Jul 14, 2022

Conversation

butonic
Copy link
Contributor

@butonic butonic commented Jul 14, 2022

We now optionally split the space id from the endpoint. This keeps the wopiserver compatible with reva master and reva edge, where we add the space id with: cs3org/reva#3074

@butonic butonic requested a review from glpatcern as a code owner July 14, 2022 10:07
@lgtm-com
Copy link
Contributor

lgtm-com bot commented Jul 14, 2022

This pull request introduces 1 alert when merging b280e69 into 2bf00ed - view on LGTM.com

new alerts:

  • 1 for Unused import

Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>

drop unused import

Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
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, especially as we keep the implementation agnostic to the two Reva flavours.

Thanks also for updating the docs!

@glpatcern glpatcern merged commit 4643de3 into cs3org:master Jul 14, 2022
@glpatcern
Copy link
Member

This pull request introduces 1 alert when merging b280e69 into 2bf00ed - view on LGTM.com

new alerts:

  • 1 for Unused import

Not sure why this fires here, will review at a later stage

@butonic
Copy link
Contributor Author

butonic commented Jul 14, 2022

@glpatcern I cleaned that up and force pushed. code is no longer importing string

@butonic butonic deleted the optionally-split-space-from-endpoint branch July 14, 2022 11:51
@micbar micbar mentioned this pull request Jul 14, 2022
36 tasks
wkloucek added a commit to owncloud-devops/ocis-team-ocis that referenced this pull request Jul 20, 2022
wkloucek added a commit to owncloud-devops/ocis-team-ocis that referenced this pull request Jul 20, 2022
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