-
Notifications
You must be signed in to change notification settings - Fork 190
Fix: Update starknet kit and starknet js #1195
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
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
@akintewe is attempting to deploy a commit to the LFG Labs Team on Vercel. A member of the Team first needs to authorize it. |
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including You can disable this status message by setting the WalkthroughThis update modifies wallet naming from "Argent X" and "Argent (mobile)" to "Ready" in both configuration and tests, updates related connector instantiation, and upgrades the Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant UI
participant NotificationManager
participant WalletConfig
User->>UI: Initiate wallet connection
UI->>WalletConfig: Fetch wallet options
WalletConfig-->>UI: Return wallet list (with "Ready" as name)
UI->>User: Display wallet options ("Ready" shown)
User->>UI: Submit transaction
UI->>NotificationManager: Monitor transaction status
NotificationManager->>UI: Notify only if transaction is reverted (not rejected)
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related issues
✨ Finishing Touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Marchand-Nicolas
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey it doesn't build, you can debug using npm run build. You can also try deleting the package-lock.json file
you can try it now |
Marchand-Nicolas
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great job :D
Lgtm!!
|
Thanks for notifying me @akintewe I forgot to merge! |
Pull Request type
Please add the labels corresponding to the type of changes your PR introduces:
Resolves: #1194
Description
This PR updates the Starknet.js and StarknetKit packages to their latest stable versions and implements the new "Ready" branding for Argent wallet in the connection menu.
Changes Made:
Package Updates:
starknet.jsfrom^6.11.0to7.6.4starknetkitfrom^2.6.1to3.0.1Wallet Name Updates:
Breaking Changes Fixed:
isRejected()method calls toisReverted()in:components/UI/notifications/notificationDetail.tsxhooks/useNotificationManager.tsTesting:
Files Modified:
package.json- Updated dependenciespackage-lock.json- Updated lock fileutils/walletConfig.ts- Updated wallet namescomponents/UI/notifications/notificationDetail.tsx- Fixed breaking changeshooks/useNotificationManager.ts- Fixed breaking changesOther information
This PR addresses issue #1194 to update the wallet connection menu to display "Ready" instead of "Argent" for the Argent wallet provider, ensuring the updated branding is reflected in the UI when users choose a wallet provider.
Close #1194
Summary by CodeRabbit