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
Added function support on plugins registration #94
Conversation
…ed package.json to version 4.1.1
This is for issue #89 |
server.start(() => { | ||
|
||
console.log('hapi days!'); | ||
plugin = require('./awesome-plugin.js'); |
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 coding style does not conform to https://github.com/hapijs/contrib/blob/master/Style.md
@@ -1,7 +1,7 @@ | |||
{ | |||
"name": "glue", | |||
"description": "Server composer for hapi.js", | |||
"version": "4.1.0", | |||
"version": "4.1.1", |
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.
don't change the version
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.
@csrl I will revert does changes and make a new pull request. I will let you know about this tonight.
I would love to see this change merged? Is it still in the works? |
It was implemented in #95 and released in v4.2 |
This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions. |
Hi I add function support on registrations: [{plugin: require('./my-awesome-plugin.js')}] because some times to use things like server rendering is needed that webpack or something like that parse imports/require it self.
Also let people do it with his own require/import logic.