Skip to content

Update nextjs.md#2998

Merged
dai-shi merged 3 commits intopmndrs:mainfrom
theperfectpunk:patch-1
Feb 8, 2025
Merged

Update nextjs.md#2998
dai-shi merged 3 commits intopmndrs:mainfrom
theperfectpunk:patch-1

Conversation

@theperfectpunk
Copy link
Copy Markdown
Contributor

add mandatory argument for useRef

Related Bug Reports or Discussions

Fixes #

Summary

Two code snippets in the documentation have useRef hook without initialValue mandatory argument, added null as the initialValue for both of the hooks

Check List

  • pnpm run fix for formatting and linting code and docs

add mandatory argument for useRef
@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 5, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
zustand-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 8, 2025 9:58am

@codesandbox-ci
Copy link
Copy Markdown

codesandbox-ci Bot commented Feb 5, 2025

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Copy link
Copy Markdown
Member

@dai-shi dai-shi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch! Do we have other places in docs that need to be fixed too?

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Feb 5, 2025

Open in Stackblitzdemostarter

npm i https://pkg.pr.new/zustand@2998

commit: 32e87fb

added initialValue for useRef
@theperfectpunk
Copy link
Copy Markdown
Contributor Author

Nice catch! Do we have other places in docs that need to be fixed too?

updated one more useRef in the testing section

Copy link
Copy Markdown
Member

@dai-shi dai-shi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@dai-shi dai-shi merged commit 57608bb into pmndrs:main Feb 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants