Skip to content

Fix share dialog sidebar in share views - #10700

Merged
MorrisJobke merged 3 commits into
masterfrom
fix/10516/sharepermissions
Aug 20, 2018
Merged

Fix share dialog sidebar in share views#10700
MorrisJobke merged 3 commits into
masterfrom
fix/10516/sharepermissions

Conversation

@blizzz

Copy link
Copy Markdown
Member

This was due to a change where permission are taken over from the DAV endpoint.

Also removes dead code which i reasonable to backport (first commit).

OCA.Sharing.SharedFileInfo was never defined and that stopped execution.
Interestingly, FF never showed me an error.
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
@blizzz
blizzzforce-pushed the fix/10516/sharepermissions branch from 402da0d to e89567eCompareAugust 15, 2018 13:27
@rullzer

Copy link
Copy Markdown
Member

unit tests are not happy https://drone.nextcloud.com/nextcloud/server/9669/259

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
@blizzz

Copy link
Copy Markdown
MemberAuthor

unit tests are not happy https://drone.nextcloud.com/nextcloud/server/9669/259

fixed

@MorrisJobkeMorrisJobke left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and works 👍

@MorrisJobke
MorrisJobke merged commit 6d749bf into masterAug 20, 2018
@MorrisJobke
MorrisJobke deleted the fix/10516/sharepermissions branch August 20, 2018 15:14
@rullzerrullzer mentioned this pull request Aug 22, 2018
3 tasks
@MorrisJobke

Copy link
Copy Markdown
Member

@blizzz What about the backport?

@MorrisJobkeMorrisJobke mentioned this pull request Aug 22, 2018
8 tasks
@blizzz

Copy link
Copy Markdown
MemberAuthor

Thanks for the reminder. Will do, on my list.

@MorrisJobke

Copy link
Copy Markdown
Member

Thanks for the reminder. Will do, on my list.

cough tomorrow is RC day cough

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@blizzz@rullzer@MorrisJobke@skjnldsv