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

Fail initialization if app is unsupported #2034

Merged
merged 2 commits into from
Sep 7, 2021

Conversation

glpatcern
Copy link
Member

A mini PR to improve error handling when an app is not WOPI-compliant nor it is supported by the WOPI bridge extensions

@update-docs
Copy link

update-docs bot commented Sep 3, 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.

@glpatcern glpatcern marked this pull request as draft September 3, 2021 08:28
@glpatcern glpatcern force-pushed the appprovider-patch branch 2 times, most recently from 1ea182f to 3b78a63 Compare September 6, 2021 13:10
As the frontend uses a separate tab, the folderurl argument
becomes less relevant and WOPI will hide its breadcrumb when missing
@labkode
Copy link
Member

labkode commented Sep 7, 2021

@glpatcern if this is ready just mark the PR as "Ready for Review" so it moves out of draft state.

@glpatcern glpatcern marked this pull request as ready for review September 7, 2021 08:17
@glpatcern
Copy link
Member Author

Indeed, it can be merged right away - then we still have your patches Hugo ;-)

@labkode labkode merged commit f004c26 into cs3org:master Sep 7, 2021
refs added a commit to refs/reva that referenced this pull request Sep 8, 2021
spaces: refactor decomposedfs spaces related code into spaces.go

spaces: move list spaces to the new spaces.go file

spaces: move private create space to the new spaces.go file

space: spaces can have duplicated names

space: fix linter

space: uint conversions

linter: file header

linter: change receiver name

linter: space out comments

spaces: refactor and cleanup a bit around uuids

xattrs: add space name extended attribute, as opposed to node name

lint: redundancy ...

lint: ...

xattrs: comment on space.name

add changelog

spaces: a space creator becomes an editor within the new resource

escape ldap filters (cs3org#2042)

fix the storageid of shares (cs3org#2033)

[Build-deps]: Bump github.com/go-chi/chi/v5 from 5.0.3 to 5.0.4 (cs3org#2038)

Bumps [github.com/go-chi/chi/v5](https://github.com/go-chi/chi) from 5.0.3 to 5.0.4.
- [Release notes](https://github.com/go-chi/chi/releases)
- [Changelog](https://github.com/go-chi/chi/blob/master/CHANGELOG.md)
- [Commits](go-chi/chi@v5.0.3...v5.0.4)

---
updated-dependencies:
- dependency-name: github.com/go-chi/chi/v5
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

[Build-deps]: Bump github.com/aws/aws-sdk-go from 1.40.17 to 1.40.37 (cs3org#2040)

Bumps [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) from 1.40.17 to 1.40.37.
- [Release notes](https://github.com/aws/aws-sdk-go/releases)
- [Changelog](https://github.com/aws/aws-sdk-go/blob/main/CHANGELOG.md)
- [Commits](aws/aws-sdk-go@v1.40.17...v1.40.37)

---
updated-dependencies:
- dependency-name: github.com/aws/aws-sdk-go
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

[Build-deps]: Bump go.opentelemetry.io/otel/sdk (cs3org#2039)

Bumps [go.opentelemetry.io/otel/sdk](https://github.com/open-telemetry/opentelemetry-go) from 1.0.0-RC2 to 1.0.0-RC3.
- [Release notes](https://github.com/open-telemetry/opentelemetry-go/releases)
- [Changelog](https://github.com/open-telemetry/opentelemetry-go/blob/main/CHANGELOG.md)
- [Commits](open-telemetry/opentelemetry-go@v1.0.0-RC2...v1.0.0-RC3)

---
updated-dependencies:
- dependency-name: go.opentelemetry.io/otel/sdk
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

runtime: do not truncate logs on reload (cs3org#2047)

Bump core commit id (cs3org#2048)

update cs3apis, add utility methods for share filters (cs3org#2044)

Fail initialization if app is unsupported (cs3org#2034)

Nextcloud storage integration tests & add Nextcloud drivers for auth and user (cs3org#2043)
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.

2 participants