Skip to content

docs: set correct default value for deep option in usefetch #32724

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

Merged
merged 1 commit into from
Jul 22, 2025

Conversation

mmeester
Copy link

🔗 Linked issue

📚 Description

This pull request updates the documentation for the deep option in useFetch to set the default value to true instead of false. This change aligns the documentation with the actual implementation in the source code and ensures consistency for users referencing the docs.

Motivation
Previously, there was a discrepancy between the Nuxt 3.x documentation and the codebase regarding the default value of the deep option. The documentation stated the default was false, while the implementation and expected behavior is true. This update corrects the documentation, helping prevent user confusion and improving clarity.

Summary of changes

  • Set the documented default value of deep to true.
  • Clarified related explanations to reflect this default.

@mmeester mmeester requested a review from danielroe as a code owner July 22, 2025 11:54
Copy link

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@danielroe
Copy link
Member

as with the other PR, the default is false for v4

what made you think otherwise?

@danielroe danielroe closed this Jul 22, 2025
@mmeester
Copy link
Author

@danielroe I know, I made the misstake of adjusting it to v4 first, this is a change for v3 docs

@danielroe danielroe reopened this Jul 22, 2025
@danielroe
Copy link
Member

thank you!

@danielroe danielroe merged commit 31c7667 into nuxt:3.x Jul 22, 2025
13 of 15 checks passed
@github-actions github-actions bot mentioned this pull request Jul 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants