Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

feat: addunstable_defaultUse#3111

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Draft
himself65 wants to merge1 commit intopmndrs:main
base:main
Choose a base branch
Loading
fromhimself65:himself65/2025/07/29/use

Conversation

@himself65
Copy link
Collaborator

From v3 discussion#2889

Do not use use in useAtomValue (migration path: add { use: true } option which is true by default in v2, and warns it)

@vercel
Copy link

vercelbot commentedJul 29, 2025•
edited
Loading

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

NameStatusPreviewCommentsUpdated (UTC)
jotai✅ Ready (Inspect)Visit Preview💬Add feedbackJul 29, 2025 11:56pm

@codesandbox-ci
Copy link

This pull request is automatically built and testable inCodeSandbox.

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

@github-actions
Copy link

LiveCodes Preview inLiveCodes

Latest commit:0438ff3
Last updated: Jul 29, 2025 11:56pm (UTC)

PlaygroundLink
React demohttps://livecodes.io?x=id/444XAH6WH

Seedocumentations for usage instructions.

@pkg-pr-new
Copy link

More templates

npm i https://pkg.pr.new/jotai@3111

commit:0438ff3

@dai-shi
Copy link
Member

I would like to wait for#2889 (comment) a little more.

Once we decided our plan, we don't needunstable_ prefix. btw, what's the implication behinddefaultUse instead ofuse?

himself65 reacted with thumbs up emoji

const{
delay,
unstable_promiseStatus:promiseStatus=!React.use,
unstable_defaultUse=true,
Copy link
Member

Choose a reason for hiding this comment

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

We should warn if it's undefined for the migration. So, it's too early to merge this PR.

himself65 reacted with thumbs up emoji
@dai-shidai-shi marked this pull request as draftJuly 30, 2025 00:02
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@dai-shidai-shidai-shi left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@himself65@dai-shi
[8]ページ先頭
©2009-2025 Movatter.jp