Skip to content

Add a wrapper to fall back to the share owner on public shares - #20475

Closed
rullzer wants to merge 1 commit into
masterfrom
fix/public_share_owner_activity_mess
Closed

Add a wrapper to fall back to the share owner on public shares#20475
rullzer wants to merge 1 commit into
masterfrom
fix/public_share_owner_activity_mess

Conversation

@rullzer

Copy link
Copy Markdown
Member

Else it might happen that we setup a storage that has no real owner and
activities do boom etc (looking at you groupfolders and external
storages!).

Steps

  1. Create a new external stroage
  2. Share the external storage by link and allow editing
  3. Open it in p0rn mode
  4. Upload a file

before: 💥
now: 🎸

Else it might happen that we setup a storage that has no real owner and
activities do boom etc (looking at you groupfolders and external
storages!).
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
@rullzer
rullzerforce-pushed the fix/public_share_owner_activity_mess branch from 79cec4d to 909d79aCompareApril 14, 2020 12:49
@rullzerrullzer closed this Apr 14, 2020
@rullzer
rullzer deleted the fix/public_share_owner_activity_mess branch April 14, 2020 18:37
@MorrisJobke

Copy link
Copy Markdown
Member

Why was this closed?

@rullzer

Copy link
Copy Markdown
MemberAuthor

Because a different PR from joas fixed it :)

@nickvergessen

Copy link
Copy Markdown
Member

#19936

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@rullzer@MorrisJobke@nickvergessen