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
Update Migration-Guide-V4.md #4091
Update Migration-Guide-V4.md #4091
Conversation
better error reporting for route definition. As a result if you specify an `onRoute` hook | ||
in a plugin you should either: | ||
* wrap your routes in a plugin (recommended) | ||
* use `await register(...)` |
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 would add the code example to let the user understand better the implications:
any onRoute
hooks will not be executed when a plugin adds them if it is not awaited or the routes are wrapped in a plugin
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 added the code example used in the linked issue before and after refactoring. Could you have a look please if that was what you had in mind?
Co-authored-by: Matteo Collina <matteo.collina@gmail.com>
Add code examples to the Synchronous route definitions section
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
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Extending the V4 migration guide to highlight the changes to route definitions.
Resolves #4085
Checklist
and the Code of conduct