chore: develop -> main #413
Conversation
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
feat: add porto and config
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Pull request overview
This pull request merges the develop branch into main, bringing updates for version 2.0.2. The changes focus on enhancing wallet connectivity options and fixing configuration issues.
Key Changes
- Added multiple new wallet connector options (@rainbow-me/rainbowkit, @reown/appkit, porto)
- Updated core Web3 dependencies (viem from 2.29.1 to 2.37.9, wagmi from 2.15.2 to 2.17.5)
- Implemented Porto wallet connector with conditional initialization based on environment variable
- Fixed TypeScript configuration by removing the unused
customConditionsproperty
Reviewed changes
Copilot reviewed 10 out of 13 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| package.json | Version bump to 2.0.2, added new wallet connector dependencies, updated viem/wagmi versions, attempted to fix typos in onlyBuiltDependencies |
| tsconfig.json | Removed unused customConditions compiler option |
| src/providers/Web3Provider.tsx | Added import for Porto wallet initialization |
| src/lib/wallets/web3modal.config.tsx | Activated web3modal configuration by uncommenting code and updating to use @reown/appkit |
| src/lib/wallets/rainbowkit.config.tsx | Minor comment formatting updates |
| src/lib/wallets/portoInit.ts | New file implementing Porto wallet initialization with error handling |
| src/lib/wallets/connectkit.config.tsx | Added blank line for formatting |
| src/env.ts | Added PUBLIC_ENABLE_PORTO environment variable validation |
| src/components/sharedComponents/ui/Header/MobileMenu/MobileMenu.tsx | Wrapped ConnectWalletButton in Box component for better layout control |
| pnpm-lock.yaml | Updated lockfile with new dependencies and version changes |
| README.md | Updated documentation link to match new URL structure |
| .gitignore | Added .tool-versions to ignored files |
| .env.example | Added PUBLIC_ENABLE_PORTO configuration option with default value |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.
| "onlyBuiltDependencies": [ | ||
| "@biomejs/biome", | ||
| "@parcel/watcher", | ||
| "@reown/appkit", | ||
| "@swc/core", | ||
| "ufferutil", | ||
| "sbuild", | ||
| "bufferutil", | ||
| "eccak", | ||
| "tf-8-validate" | ||
| "esbuild", | ||
| "keccak", | ||
| "sbuild", | ||
| "tf-8-validate", | ||
| "ufferutil", | ||
| "utf-8-validate" | ||
| ] |
There was a problem hiding this comment.
The onlyBuiltDependencies array contains typos from the previous code. Several entries are missing their first character (e.g., "ufferutil" instead of "bufferutil", "sbuild" instead of "esbuild", "eccak" instead of "keccak", "tf-8-validate" instead of "utf-8-validate"). While this PR fixes some of these typos, it still contains duplicates ("sbuild" remains alongside the corrected "esbuild") and maintains the incorrect entries.
develop -> main