Skip to content

fix(internals): check if atom has onMount property before queueing processOnMount callback (performance)#3291

Open
dmaskasky wants to merge 2 commits intopmndrs:mainfrom
dmaskasky:mount-callbacks-condition
Open

fix(internals): check if atom has onMount property before queueing processOnMount callback (performance)#3291
dmaskasky wants to merge 2 commits intopmndrs:mainfrom
dmaskasky:mount-callbacks-condition

Conversation

@dmaskasky
Copy link
Copy Markdown
Collaborator

Summary

Check if atom has onMount property before queueing processOnMount callback.

Check List

  • pnpm run fix for formatting and linting code and docs

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 1, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
jotai Ready Ready Preview, Comment Apr 1, 2026 11:27pm

Request Review

@codesandbox-ci
Copy link
Copy Markdown

codesandbox-ci bot commented Apr 1, 2026

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.

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Apr 1, 2026

More templates

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

commit: 3cf2d9f

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 1, 2026

LiveCodes Preview in LiveCodes

Latest commit: 3cf2d9f
Last updated: Apr 1, 2026 11:26pm (UTC)

Playground Link
React demo https://livecodes.io?x=id/AMU75FWL9

See documentations for usage instructions.

@dmaskasky dmaskasky force-pushed the mount-callbacks-condition branch from d4cac8a to 3cf2d9f Compare April 1, 2026 23:26
@dai-shi dai-shi changed the title check if atom has onMount property before queueing processOnMount callback fix(internals): check if atom has onMount property before queueing processOnMount callback (performance) Apr 2, 2026
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