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
add static:assets.prepare hook #4889
Conversation
add static:assets.prepare hook to be called before css minification so that themes/plugins can modify less vars.
@@ -85,6 +85,7 @@ function initializeNodeBB(callback) { | |||
function(next) { | |||
plugins.init(app, middleware, next); | |||
}, | |||
async.apply(plugins.fireHook, 'static:assets.prepare', []), |
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.
I don't think you need to pass anything in, do you? (Re: the []
)
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.
the theme right now does not pass anything to NodeBB so yeah it can be removed.
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.
Right. NodeBB doesn't do anything with the returned data, actually. It's just for plugins to consume. In this case, it can be removed... or rather, replaced with {}
, since core expects a parameter here.
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.
should be good to go.
👍 |
…eBB/NodeBB#4892> - allowing widgets to not show up if widget explicitly returns null <NodeBB/NodeBB@a9a522c> - add static:assets.prepare hook <NodeBB/NodeBB#4889>
add static:assets.prepare hook to be called before css minification so that themes/plugins can modify less vars.