You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
🐞 Bug fix (a non-breaking change that fixes an issue)
👌 Enhancement (improving an existing functionality like performance)
✨ New feature (a non-breaking change that adds functionality)
🧹 Chore (updates to the build process or auxiliary tools and libraries)
⚠️ Breaking change (fix or feature that would cause existing functionality to change)
📚 Description
Attempt to more explicitly outline the available hooks and their usage for Nitro server. This could help support the addition of more hooks in the linked issue.
Introduce new guide section for hooks, including examples with hook callback argument types.
Move hooks examples from Guides/Plugins into Guides/Hooks.
Add Hook documentation for new on/before/after request hooks.
- Introduce new guide section for hooks, including examples with hook callback
argument types.
- Move hooks examples from Guides/Plugins into Guides/Hooks.
- Add Hook documentation for new on/before/after request hooks.
Merging #1521 (dcac214) into main (451d314) will increase coverage by 0.03%.
Report is 12 commits behind head on main.
The diff coverage is n/a.
❗ Current head dcac214 differs from pull request most recent head 20d8aae. Consider uploading reports for the commit 20d8aae to get more accurate results
Thanks for helping on this @gnoeley <3 In the meantime, I think it makes more sense to keep runtime hooks section under the plugins because that's how mainly they should be registered and used. I have made few changes to merge, added link and list of available routes an additional example for error.
When adding two new (upcoming) hooks for global request, after-reponse, i will extend your current work on new section (therefore merging your PR sooner to iterate. You are more than welcome to open more to improve.)
pi0
changed the title
docs(plugins): improve hooks section
docs(plugins): add hooks section
Aug 6, 2023
pi0
changed the title
docs(plugins): add hooks section
docs(plugins): add runtime hooks section
Aug 6, 2023
documentationImprovements or additions to documentation
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
pi0
changed the title
🔗 Linked issue
#1301
❓ Type of change
📚 Description
Attempt to more explicitly outline the available hooks and their usage for Nitro server. This could help support the addition of more hooks in the linked issue.
📝 Checklist