Skip to content
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.

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

[AdminBundle] fixed base route name for exception admin lists #3414

Merged
merged 1 commit into from
Apr 28, 2024

Conversation

virtualize
Copy link
Contributor

@virtualize virtualize commented Apr 22, 2024

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Fixed tickets

I don't know if this the admin list for exceptions is still used. Or if exceptions are still logged in 7.1. Maybe it's deprecated and there's a config flag to disable it. Haven't found anything on the docs.

Anyway, in our admin UI there's a menu entry for it that throws route-not-found exceptions. So here is a fix for it.

@acrobat acrobat added this to the 6.4.3 milestone Apr 28, 2024
@acrobat acrobat changed the base branch from 7.1 to 6.4 April 28, 2024 19:27
@acrobat acrobat force-pushed the fix-admin-exception-route-71 branch from 4530db8 to c05533e Compare April 28, 2024 19:27
@acrobat acrobat merged commit 9c6560b into Kunstmaan:6.4 Apr 28, 2024
9 checks passed
@acrobat
Copy link
Member

acrobat commented Apr 28, 2024

Thanks @virtualize. You can disable the cms error logging with the kunstmaan_admin.exception_logging.enabled: false if you don't need it in your application because you use sentry for example.

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.

None yet

2 participants