Skip to content

Strip of users home path from share api message #19528

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.

Sign up for GitHub

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

Jump to bottom
Merged
rullzer merged 1 commit into from
Feb 22, 2020
Merged

Strip of users home path from share api message #19528

rullzer merged 1 commit into from
Feb 22, 2020

Conversation

@nickvergessen
Copy link
Member

@nickvergessen nickvergessen commented Feb 18, 2020 •
edited
Loading

Before

You are not allowed to share /445d1288-a75a-44df-87fd-bb4c72bdc488/files/Resharing disabled/1P8A0501.jpg

After

You are not allowed to share /Resharing disabled/1P8A0501.jpg

@nickvergessen nickvergessen added this to the Nextcloud 19 milestone Feb 18, 2020
@nickvergessen nickvergessen mentioned this pull request Feb 18, 2020
Merged
@nickvergessen
Copy link
Member Author

nickvergessen commented Feb 18, 2020

/backport to stable18

@nickvergessen
Copy link
Member Author

nickvergessen commented Feb 18, 2020

/backport to stable17

@nickvergessen
Copy link
Member Author

nickvergessen commented Feb 18, 2020

/backport to stable16

@blizzz

This comment has been minimized.

Sign in to view
icewind1991
icewind1991 reviewed Feb 19, 2020
View reviewed changes
@nickvergessen
Strip of users home path from share api message
e97d643 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen force-pushed the bugfix/noid/remove-user-root-from-error-message branch from 3182748 to e97d643 Compare February 20, 2020 13:57
@nickvergessen
Copy link
Member Author

nickvergessen commented Feb 20, 2020

Fixed tests

@rullzer rullzer merged commit ab7f6dd into master Feb 22, 2020
@rullzer rullzer deleted the bugfix/noid/remove-user-root-from-error-message branch February 22, 2020 20:25
@backportbot-nextcloud backportbot-nextcloud bot mentioned this pull request Feb 22, 2020
Merged
@backportbot-nextcloud
Copy link

backportbot-nextcloud bot commented Feb 22, 2020

backport to stable18 in #19591

@backportbot-nextcloud backportbot-nextcloud bot mentioned this pull request Feb 22, 2020
Merged
@backportbot-nextcloud
Copy link

backportbot-nextcloud bot commented Feb 22, 2020

backport to stable17 in #19592

@backportbot-nextcloud backportbot-nextcloud bot mentioned this pull request Feb 22, 2020
Merged
@backportbot-nextcloud
Copy link

backportbot-nextcloud bot commented Feb 22, 2020

backport to stable16 in #19594

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

Reviewers

@icewind1991 icewind1991 icewind1991 left review comments

@ChristophWurst ChristophWurst ChristophWurst approved these changes

@blizzz blizzz blizzz approved these changes

@georgehrke georgehrke Awaiting requested review from georgehrke

Assignees

No one assigned

Labels

3. to review Waiting for reviews bug feature: sharing

Projects

None yet

Milestone

Nextcloud 19

Development

Successfully merging this pull request may close these issues.

6 participants

@nickvergessen @blizzz @icewind1991 @ChristophWurst @rullzer