feat: add connect wallet functionality#39
Merged
Iwueseiter merged 1 commit intoChain-Library:mainfrom Jun 7, 2025
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description:
This PR introduces a new wallet connector modal that enables users to connect to multiple Starknet-compatible wallets including Braavos, Argent, and others. The modal is designed with a user-friendly and responsive interface that aligns with the project's existing design language and follows architectural conventions specified in the Predifi Frontend README.
Key Features:
🔌 Multi-Wallet Support: Supports seamless integration with popular Starknet wallets (Braavos, Argent, etc.).
🧩 Figma-Inspired UI: Modal UI adheres closely to the Figma design specs—maintaining consistent colors, typography, and spacing.
⚙️ Error Handling: Graceful handling of connection errors, including unsupported wallets and user denial of connection.
📱 Responsive Design: Modal is fully responsive across various screen sizes and provides a smooth user experience.
✅ Stable Integration: Ensured that the npm run build passes without errors post integration.
📁 Code Consistency: All code additions follow Predifi’s frontend architecture and styling conventions.
Technical Details:
Integrated wallet connectors using appropriate libraries and APIs for Starknet wallet communication.
Implemented UI feedback for different states: loading, error, and connected.
Modularized code to ensure easy future extension or addition of new wallets.
Manual and functional testing performed to validate wallet connectivity and error flows.
Screenshots:
(Include if applicable—UI preview of modal on desktop/mobile)
Checklist:
UI matches Figma design specs
Wallet connections tested (Braavos, Argent)
Error handling implemented
Follows Predifi Frontend conventions
No build-time errors (npm run build passed)
Modal is responsive and interactive
Notes:
Please review especially the wallet connection logic and let me know if there are additional wallets we should support in this phase.
closes #37