Skip to content

Conversation

@juliusknorr
Copy link
Member

Make sure we redirect to the files app and show a proper error message instead of a technical page caused by the NotFoundException.

Signed-off-by: Julius Härtl <jus@bitgrid.net>
Copy link
Member

@skjnldsv skjnldsv left a comment

Choose a reason for hiding this comment

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

make sense!

Copy link
Member

@ChristophWurst ChristophWurst left a comment

Choose a reason for hiding this comment

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

👍

@skjnldsv skjnldsv added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jun 18, 2019
@skjnldsv skjnldsv merged commit 3a0e4a1 into master Jun 19, 2019
@skjnldsv skjnldsv deleted the bugfix/noid/file-not-found branch June 19, 2019 08:35
@kesselb
Copy link
Collaborator

kesselb commented Aug 27, 2019

Could you consider a backport?

@juliusknorr
Copy link
Member Author

/backport to stable16

@juliusknorr
Copy link
Member Author

/backport to stable15

@backportbot-nextcloud
Copy link

The backport to stable15 failed. Please do this backport manually.

@backportbot-nextcloud
Copy link

backport to stable16 in #16891

@juliusknorr
Copy link
Member Author

No backport for stable15 needed, it has been broken with 16 only.

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

Labels

4. to release Ready to be released and/or waiting for tests to finish bug feature: files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants