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

No way to accept shares if notifications app is not enabled #18255

Closed
icewind1991 opened this issue Dec 5, 2019 · 8 comments · Fixed by #18385
Closed

No way to accept shares if notifications app is not enabled #18255

icewind1991 opened this issue Dec 5, 2019 · 8 comments · Fixed by #18385
Assignees
Labels
1. to develop Accepted and waiting to be taken care of bug feature: sharing

Comments

@icewind1991
Copy link
Member

Since #16828, when a file is shared but the notifications app is not installed, there is no way to accept the share nor is any feedback given, everything looks fine for the person creating the share, but the recipient doesn't see anything.

Not having the notifications app is probably not common (outside git dev installs) but it's not a force enabled app

@nickvergessen @rullzer

@icewind1991 icewind1991 added bug 1. to develop Accepted and waiting to be taken care of feature: sharing labels Dec 5, 2019
@nickvergessen
Copy link
Member

Right, does federation still have a workaround for this, or are they also broken?

@rullzer
Copy link
Member

rullzer commented Dec 5, 2019

Probably still has a workaround I bet. I can check later.

@nickvergessen
Copy link
Member

This topic came up in the office today and @karlitschek said we dont need to workaround if people disable a default enabled shipped app.

But we should add a config to automatically accept shares in some bigger enviroments.

@nickvergessen nickvergessen self-assigned this Dec 6, 2019
@nickvergessen nickvergessen added this to the Nextcloud 18 milestone Dec 6, 2019
@karlitschek
Copy link
Member

@jospoortvliet

@jospoortvliet
Copy link
Member

jospoortvliet commented Dec 6, 2019

Imho the user sharing settings should have a

  • automatically accept incoming shares

This should be disabled by default. This makes this feature opt-in which is probably the right thing to do as it is quite a change of behavior if we make this default.

An OCC command or config.php option for this is probably also needed to change the default.

@wiswedel
Copy link
Contributor

Let's follow up on Jos' suggestion (that I fully agree on) in a separate issue: #18373

rullzer added a commit that referenced this issue Dec 12, 2019
rullzer added a commit that referenced this issue Dec 12, 2019
Fixes #18255

A new user setting allows a user to always accept (internal) shares. For
example if they don't like accepting shares manually ;)

Signed-off-by: Roeland Jago Douma <[email protected]>
@rullzer
Copy link
Member

rullzer commented Dec 12, 2019

See #18384 for the config.php setting
And #18385 for the user setting

The user setting will only be available if the config.php override is not set.

@wiswedel
Copy link
Contributor

The user setting will only be available if the config.php override is not set.

nice approach

rullzer added a commit that referenced this issue Dec 16, 2019
rullzer added a commit that referenced this issue Dec 16, 2019
rullzer added a commit that referenced this issue Dec 16, 2019
Fixes #18255

A new user setting allows a user to always accept (internal) shares. For
example if they don't like accepting shares manually ;)

Signed-off-by: Roeland Jago Douma <[email protected]>
rullzer added a commit that referenced this issue Dec 17, 2019
Fixes #18255

A new user setting allows a user to always accept (internal) shares. For
example if they don't like accepting shares manually ;)

Signed-off-by: Roeland Jago Douma <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of bug feature: sharing
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants