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
fix(vue-app): use app.context.route
for resolving components
#9050
Conversation
If someone could have a look at the tests suite? On my side, most of the time I get a timeout from puppeteer on each e2e tests: |
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. Thanks!
app.context.route
for resolving components
Hi. Why was the if statement removed I'm able to reproduce it https://codesandbox.io/s/small-browser-ek8m5 Try switching between nuxt 2.15.3 and 2.15.4 and ull see that with latest the page keeps refreshing. @enwin ^ |
Hi @patrioticcow! The if statement was moved further down so when From what I'm seing from your code example, the issue resides in the fact that the component resolution is triggered using The best of both world would be for nuxt to use Will work on that ASAP 👍 |
I'll need your feedback on this fix since I'm calling the async component resolution used for the SSR. It might not be the best idea but it works so far 🤷
Types of changes
Description
Resolves: #9049
Update
vue-app/template/index.js
to call async component resolver on client side too.Update
vue-app/template/client.js
to useapp.context.route
in theresolveComponents
method to match the components based on the final route.Checklist: