Skip to content

Commit

Permalink
Remove placeholder notifications endpoint in capabilities
Browse files Browse the repository at this point in the history
  • Loading branch information
pascalwengerter committed Sep 16, 2021
1 parent cb3a5b8 commit f5df717
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 3 deletions.
7 changes: 7 additions & 0 deletions changelog/unreleased/remove-notifications-placeholder.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
Bugfix: Remove notifications placeholder

Since Reva was communicating its notification capabilities incorrectly, oCIS relied on a hardcoding string to overwrite them.
This has been fixed in [reva#1819](https://github.com/cs3org/reva/pull/1819) so we now removed the hardcoded string
and don't modify Reva's notification capabilities anymore in order to fix clients having to poll a (non-existent) notifications endpoint.

https://github.com/owncloud/ocis/pull/2514
3 changes: 0 additions & 3 deletions storage/pkg/command/frontend.go
Original file line number Diff line number Diff line change
Expand Up @@ -250,9 +250,6 @@ func frontendConfigFromStruct(c *cli.Context, cfg *config.Config, filesCfg map[s
"incoming": true,
},
},
"notifications": map[string]interface{}{
"endpoints": []string{"disable"},
},
},
"version": map[string]interface{}{
"edition": "reva",
Expand Down

0 comments on commit f5df717

Please sign in to comment.