Skip to content

feat: add default exception handler for litestar integration - #308

Merged
cofin merged 6 commits into
mainfrom
exc-handler-litestar
Dec 5, 2024
Merged

cofin merged 6 commits into
mainfrom
exc-handler-litestar

Conversation

@cofin

@cofin cofin commented Dec 4, 2024

Copy link
Copy Markdown
Member

Description

This adds a configuration option to automatically enable an excpetion handler for Repository errors.

This will update the exception handler if you do not have one already configured for the RepositoryException class

Closes

@cofin

cofin commented Dec 5, 2024

Copy link
Copy Markdown
Member Author

This change should replace #294

Comment thread advanced_alchemy/extensions/litestar/plugins/init/plugin.py
Comment thread advanced_alchemy/extensions/litestar/plugins/init/plugin.py
Comment thread advanced_alchemy/extensions/litestar/plugins/init/plugin.py
Comment thread advanced_alchemy/extensions/litestar/exception_handler.py Outdated
Comment thread advanced_alchemy/extensions/litestar/exception_handler.py Outdated
cofin and others added 5 commits December 5, 2024 11:21
Co-authored-by: Alc-Alc <45509143+Alc-Alc@users.noreply.github.com>
Signed-off-by: Cody Fincher <204685+cofin@users.noreply.github.com>
…nit/plugin.py"

This reverts commit 1aa2e60.

I did not not realize the loop was used for other stuff
@cofin
cofin force-pushed the exc-handler-litestar branch from 5adcf20 to a17b8ef Compare December 5, 2024 17:22
@cofin
cofin merged commit 2644e1e into main Dec 5, 2024
@cofin
cofin deleted the exc-handler-litestar branch December 5, 2024 17:33
@sonarqubecloud

sonarqubecloud Bot commented Dec 5, 2024

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@github-actions

github-actions Bot commented Dec 5, 2024

Copy link
Copy Markdown

Documentation preview will be available shortly at https://litestar-org.github.io/advanced-alchemy-docs-preview/308

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enhancement: Add exception handling for NotFoundException

2 participants