Skip to content
This repository has been archived by the owner on Jan 3, 2025. It is now read-only.

Add json response exceptions_app #675

Merged
merged 2 commits into from
Sep 9, 2024
Merged

Add json response exceptions_app #675

merged 2 commits into from
Sep 9, 2024

Conversation

FinnIckler
Copy link
Member

So we can display a general error, currently we were returning HTML which crashed the error handling code

@dunkOnIT
Copy link
Collaborator

dunkOnIT commented Sep 9, 2024

@thewca-bot deploy staging

1 similar comment
@dunkOnIT
Copy link
Collaborator

dunkOnIT commented Sep 9, 2024

@thewca-bot deploy staging

Copy link
Collaborator

@dunkOnIT dunkOnIT left a comment

Choose a reason for hiding this comment

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

Tested in staging, works as expected

@dunkOnIT dunkOnIT merged commit 957dea5 into main Sep 9, 2024
1 check passed
@dunkOnIT dunkOnIT deleted the default-error branch September 9, 2024 14:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants