Skip to content

Conversation

Kolobok12309
Copy link

fix #1977
image
image

@netlify
Copy link

netlify bot commented Jan 9, 2022

✔️ Deploy Preview for nuxt ready!

🔨 Explore the source changes: 675e5ac

🔍 Inspect the deploy log: https://app.netlify.com/sites/nuxt/deploys/61dafbba0bcdce0007fb7179

😎 Browse the preview: https://deploy-preview-1978--nuxt.netlify.app

@smarroufin smarroufin added the docs needs to be documented label Jan 10, 2022 — with Volta.net
@Uzere
Copy link

Uzere commented Jan 11, 2022

As I spent a lot of time figuring out what happens in nuxt/nuxt#10139, I would like to see watchParam menitioned on https://nuxtjs.org/docs/components-glossary/key#the-key-property because that is how one can get watchParam effect.
Also I think it would be good idea to explicitly menition when data() is called in https://nuxtjs.org/docs/concepts/nuxt-lifecycle#nuxt-lifecycle

@Kolobok12309
Copy link
Author

Kolobok12309 commented Jan 11, 2022

@Uzere, Actually agree with you, unfortunately, I have no experience in graphic design and will not be able to update hooks.
Also i test behaviour when set watchParam: false and changed param and query, by source code i think this have bug and if true, this needs to be added to the documentation
UPD: If i set watchParam: false, watchQuery: true and change param AND query of route, nothing happens. Tests for it

Kolobok12309 added a commit to Kolobok12309/nuxt.js that referenced this pull request Jan 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs needs to be documented

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Doc] watchParam component property

3 participants