-
-
Notifications
You must be signed in to change notification settings - Fork 10.1k
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
Es6 migration for web/middleware #9737
Conversation
Thanks :) Will review on Monday! |
@@ -107,7 +106,7 @@ _private.ThemeErrorRenderer = function ThemeErrorRenderer(err, req, res, next) { | |||
// It can be that something went wrong with the theme or otherwise loading handlebars | |||
// This ensures that no matter what res.render will work here | |||
// @TODO: split the error handler for assets, admin & theme to refactor this away | |||
if (_.isEmpty(req.app.engines)) { | |||
if (req.app.engines.length === 0) { |
This comment was marked as abuse.
This comment was marked as abuse.
Sorry, something went wrong.
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.
See comment :) Thanks for the PR
@@ -107,7 +106,7 @@ _private.ThemeErrorRenderer = function ThemeErrorRenderer(err, req, res, next) { | |||
// It can be that something went wrong with the theme or otherwise loading handlebars | |||
// This ensures that no matter what res.render will work here | |||
// @TODO: split the error handler for assets, admin & theme to refactor this away | |||
if (_.isEmpty(req.app.engines)) { | |||
if (Object.keys(req.app.engines).length === 0) { |
This comment was marked as abuse.
This comment was marked as abuse.
Sorry, something went wrong.
This comment was marked as abuse.
This comment was marked as abuse.
Sorry, something went wrong.
581e273
to
9f8284b
Compare
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 👍
refs #9589
Changed the files
in the web/middleware folder and the web/admin, web/api files.