Conversation
🦋 Changeset detectedLatest commit: 2af98f7 The changes in this PR will be included in the next version bump. This PR includes changesets to release 23 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
10 Skipped Deployments
|
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
Coverage Report
File Coverage
|
||||||||||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment.
Bug: Memoization Ignores Prop Changes
The memoizeCreateAppKit function uses a module-level variable to memoize the AppKit instance. It only checks if the instance is null, causing it to ignore subsequent AppKitProvider prop changes (e.g., projectId, networks). This results in a single AppKit instance created with initial props, leading to stale configuration and incorrect behavior.
packages/appkit/src/library/react/providers.ts#L10-L26
appkit/packages/appkit/src/library/react/providers.ts
Lines 10 to 26 in f81bab3
Was this report helpful? Give feedback by reacting with 👍 or 👎
Description
Introduced
AppKitProviderReact component for easy AppKit integration in React appsExample usage
Type of change
Associated Issues
For Linear issues: Closes APKT-3303
Showcase (Optional)
Checklist