-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
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
Fix the problem of not using error template in theme #3166
Fix the problem of not using error template in theme #3166
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ruibaby The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/ping @halo-dev/sig-halo |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
What type of PR is this?
/kind bug
/area core
What this PR does / why we need it:
Currently, if there is no
error.html
error template in theme, but there is a404.html
error template, this will not work correctly.We always get rendering result from global error template
error.html
.This PR mainly provides a
ThemeTemplateAvailabilityProvider
to check if the template is available in theme instead of in globally predefined templates.Which issue(s) this PR fixes:
Fixes #3062
Special notes for your reviewer:
templates/error
templates/error/error.html
template filetemplates/error/404.html
Does this PR introduce a user-facing change?