hapijs / glue

Server composer for hapi.js
Other
245 stars 62 forks source link

Glue Documentation Additions #23

Closed zoe-1 closed 9 years ago

zoe-1 commented 9 years ago

Explained how to navigate plugin dependency loading issues when using Glue. Explained strengths and weaknesses of setting an Array or JSON object at the manifest.plugins key. Plus, gave an example of how to properly declare dependencies with server.dependency(dependencies, [after]).

zoe-1 commented 9 years ago

@csrl If you want more help with Glue documentation just let me know. Cheers!

csrl commented 9 years ago

I appreciate the documentation contributions. Please continue to do so.

For this particular update, it seems to concern itself with explaining Hapi's plugin management. I do not think Glue is the proper place to dig into the details of that. Updating Hapi's documentation to be more clear on the matter, and pointing to it from Glue would be preferred. This subject would also make for good tutorial content at http://hapijs.com/tutorials

In any case plugins should be written to properly deal with their own dependencies. Glue should not have to care about that at all. However, I merged the patch to support ordered plugin loading because the reality is, one does not always have full control over the possibly broken plugins being used, so providing ordered plugin loading was pragmatic.

zoe-1 commented 9 years ago

@csrl I do plan to follow your recommendations made above. I began adding information to the tutorials. However, work has been busy so have not completed this yet. As soon as things ease up at work, I will finish this up.

lock[bot] commented 4 years ago

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.