Skip to content

fix(API): Use a distinct exception so apps can react to it and custom… #41779

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
blizzz merged 1 commit into from
Nov 28, 2023
Merged

fix(API): Use a distinct exception so apps can react to it and custom… #41779

blizzz merged 1 commit into from
Nov 28, 2023

Conversation

@nickvergessen
Copy link
Member

@nickvergessen nickvergessen commented Nov 27, 2023 •
edited
Loading

…ize the return

Checklist

@nickvergessen nickvergessen added this to the Nextcloud 29 milestone Nov 27, 2023
@nickvergessen nickvergessen self-assigned this Nov 27, 2023
@nickvergessen nickvergessen added the pending documentation This pull request needs an associated documentation update label Nov 27, 2023
@nickvergessen nickvergessen mentioned this pull request Nov 28, 2023
Merged
3 tasks
@nickvergessen
fix(API): Use a distinct exception so apps can react to it and custom…
f6b6776 
…ize the return

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen force-pushed the bugfix/noid/use-destinct-exception-so-apps-dont-parse-messages branch from b676496 to f6b6776 Compare November 28, 2023 05:12
@blizzz blizzz merged commit 53f3149 into master Nov 28, 2023
@blizzz blizzz deleted the bugfix/noid/use-destinct-exception-so-apps-dont-parse-messages branch November 28, 2023 09:25
@blizzz blizzz mentioned this pull request Mar 5, 2024
Merged
@nickvergessen nickvergessen removed the pending documentation This pull request needs an associated documentation update label Jul 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AndyScherzinger AndyScherzinger AndyScherzinger approved these changes

@blizzz blizzz blizzz approved these changes

@st3iny st3iny Awaiting requested review from st3iny

Assignees

@nickvergessen nickvergessen

Labels

3. to review Waiting for reviews bug technical debt

Projects

None yet

Milestone

Nextcloud 29

Development

Successfully merging this pull request may close these issues.

OutOfRangeException on integration tests

4 participants

@nickvergessen @AndyScherzinger @blizzz