Open
Conversation
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Joaquim Verges <joaquim.verges@gmail.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
* Dashboard: Update bridge supported tokens count (thirdweb-dev#8602) <!-- ## title your PR with this format: "[SDK/Dashboard/Portal] Feature/Fix: Concise title for the changes" If you did not copy the branch name from Linear, paste the issue tag here (format is TEAM-0000): ## Notes for the reviewer Anything important to call out? Be sure to also clarify these in your comments. ## How to test Unit tests, playground, etc. --> <!-- start pr-codex --> --- ## PR-Codex overview This PR updates the `supportedTokens` value in the `data.ts` file for the bridge statistics, reflecting an increase in the number of supported tokens. ### Detailed summary - Updated `supportedTokens` from `"14,000+"` to `"17,000+"`. - Changed `supportedTokens` from `14000` to `17000` in the `bridgeStatsNumbers` object. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Updates** * Increased supported tokens count from 14,000+ to 17,000+ on the bridge dashboard. <sub>✏️ Tip: You can customize this high-level summary in your review settings.</sub> <!-- end of auto-generated comment: release notes by coderabbit.ai --> * [MNY-344] Dashboard: Add swap-widget iframe (thirdweb-dev#8603) <!-- ## title your PR with this format: "[SDK/Dashboard/Portal] Feature/Fix: Concise title for the changes" If you did not copy the branch name from Linear, paste the issue tag here (format is TEAM-0000): ## Notes for the reviewer Anything important to call out? Be sure to also clarify these in your comments. ## How to test Unit tests, playground, etc. --> <!-- start pr-codex --> --- ## PR-Codex overview This PR introduces a new `NEXT_PUBLIC_SWAP_IFRAME_CLIENT_ID` constant and enhances the swap functionality with a `SwapWidgetEmbed` component, while refactoring the existing `Providers` component to `BridgeProvidersLite`. It also adds necessary headers for security and updates layout components. ### Detailed summary - Added `NEXT_PUBLIC_SWAP_IFRAME_CLIENT_ID` constant in `public-envs.ts`. - Updated `next.config.ts` to include security headers for swap widget routes. - Introduced `BridgeProvidersLite` component in `Providers.client.tsx`. - Created `SwapWidgetEmbed` component for swap functionality. - Refactored `page.tsx` to utilize `BridgeProvidersLite` and `SwapWidgetEmbed`. - Removed the old `Providers` component logic. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **New Features** * Added a cross‑chain swap widget with page, embed component and layout, URL-prefill support, theming, and fiat currency options. * Added a lightweight provider wrapper for embed usage. * **Changes** * Checkout widget no longer auto-connects preconfigured wallets; provider usage unified to the lightweight wrapper. * **Chores** * Added CSP headers for swap widget routes. * Introduced public env var for swap iframe client ID. <sub>✏️ Tip: You can customize this high-level summary in your review settings.</sub> <!-- end of auto-generated comment: release notes by coderabbit.ai --> * [MNY-351] Portal: Add swap widget iframe docs (thirdweb-dev#8604) <!-- ## title your PR with this format: "[SDK/Dashboard/Portal] Feature/Fix: Concise title for the changes" If you did not copy the branch name from Linear, paste the issue tag here (format is TEAM-0000): ## Notes for the reviewer Anything important to call out? Be sure to also clarify these in your comments. ## How to test Unit tests, playground, etc. --> <!-- start pr-codex --> --- ## PR-Codex overview This PR enhances the `Swap Widget` functionality by adding an `iframe` integration option and a new `React Component`. It updates documentation and examples for better usability and expands features like dark/light mode support and customizable token selections. ### Detailed summary - Added `links` for `React Component` and `Iframe` in `sidebar.tsx`. - Introduced `IframeCodePreview` component in `iframe-code-preview.tsx`. - Updated `page.mdx` for `iframe` integration with features and examples. - Revised `page.mdx` for `react` with improved descriptions and features. - Enhanced API reference and examples for token selection configurations. - Added iframe event listening instructions for swap results. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex --> * [MNY-352] Playground: add swap widget iframe (thirdweb-dev#8605) <!-- ## title your PR with this format: "[SDK/Dashboard/Portal] Feature/Fix: Concise title for the changes" If you did not copy the branch name from Linear, paste the issue tag here (format is TEAM-0000): ## Notes for the reviewer Anything important to call out? Be sure to also clarify these in your comments. ## How to test Unit tests, playground, etc. --> <!-- start pr-codex --> --- ## PR-Codex overview This PR introduces enhancements to the `SwapWidgetPlayground`, allowing for integration with both `iframe` and `react` types. It modifies how themes and options are handled, improves URL management, and updates iframe handling for better user experience. ### Detailed summary - Added `integrationType` to `SwapWidgetPlaygroundOptions`. - Conditional rendering of `ColorFormGroup` based on `integrationType`. - Created `getPreviewSrc` function to manage iframe URLs. - Updated `persistTokenSelections` handling in various components. - Introduced `SwapWidgetPlaygroundAsync` for async loading. - Refactored `RightSection` to conditionally render `iframe` or `SwapWidget`. - Implemented URL update logic based on selected integration type. - Enhanced code generation for iframe and react integration types. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex --> * [MNY-353] SDK: TransactionWidget UI improvements (thirdweb-dev#8607) <!-- ## title your PR with this format: "[SDK/Dashboard/Portal] Feature/Fix: Concise title for the changes" If you did not copy the branch name from Linear, paste the issue tag here (format is TEAM-0000): ## Notes for the reviewer Anything important to call out? Be sure to also clarify these in your comments. ## How to test Unit tests, playground, etc. --> <!-- start pr-codex --> --- ## PR-Codex overview This PR focuses on UI improvements for the `TransactionWidget` in the `thirdweb` package, enhancing user experience with better layout, spacing, and loading indicators. ### Detailed summary - Changed the `chain` in `simpleBuyRequest` from `baseSepolia` to `base`. - Updated the story names and added new stories for `TransactionWidget`. - Improved layout and spacing in `TransactionPayment`. - Replaced static loading elements with `Skeleton` components for better loading states. - Enhanced button styles and added a loading spinner. - Refined contract info and address display for better clarity. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **UI Improvements** * Improved TransactionWidget loading states with clearer visual spinner components * Enhanced visual presentation and layout of transaction details, addresses, and network information * Refined typography, spacing, and styling throughout the widget for better readability and consistency <sub>✏️ Tip: You can customize this high-level summary in your review settings.</sub> <!-- end of auto-generated comment: release notes by coderabbit.ai --> * [MNY-319] SDK: Add token details screen in token selection UI (thirdweb-dev#8608) <!-- ## title your PR with this format: "[SDK/Dashboard/Portal] Feature/Fix: Concise title for the changes" If you did not copy the branch name from Linear, paste the issue tag here (format is TEAM-0000): ## Notes for the reviewer Anything important to call out? Be sure to also clarify these in your comments. ## How to test Unit tests, playground, etc. --> <!-- start pr-codex --> --- ## PR-Codex overview This PR focuses on enhancing the token selection UI by adding a token details screen and improving the display of token information within the `SwapWidget` and `BridgeWidget`. ### Detailed summary - Added a token details screen in the token selection UI. - Enhanced the `SelectedTokenButton` to include a more flexible display of the token symbol. - Introduced `useTokenPrice` hook for fetching token prices. - Updated `TokenButton` to include an info button for showing token details. - Created `TokenInfoScreen` and `TokenInfoRow` components for displaying detailed token information. - Integrated token info functionality into `TokenSelectionScreen`. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex --> * Etherlink shadownet chain (thirdweb-dev#8611) <!-- ## title your PR with this format: "[SDK/Dashboard/Portal] Feature/Fix: Concise title for the changes" --> [SDK] Feature: Add Etherlink Shadownet Testnet chain ## Notes for the reviewer Adds the Etherlink Shadownet testnet chain (ID 127823) to the predefined chains in the SDK. This includes its native currency (XTZ) and block explorer URL. ## How to test - Verify the `etherlinkShadownet` chain can be imported from `thirdweb/chains`. - Check that the chain object contains the correct `id`, `name`, `nativeCurrency`, and `blockExplorers` properties. - Run `pnpm build` and `pnpm lint` to ensure no new issues are introduced. --- [Slack Thread](https://thirdwebdev.slack.com/archives/C09DS2CKGP2/p1767789368607579?thread_ts=1767789368.607579&cid=C09DS2CKGP2) <a href="https://cursor.com/background-agent?bcId=bc-a0f088a7-057b-46f0-b6a6-3ba957d1e261"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/open-in-cursor-dark.svg"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/open-in-cursor-light.svg"><img alt="Open in Cursor" src="https://cursor.com/open-in-cursor.svg"></picture></a> <a href="https://cursor.com/agents?id=bc-a0f088a7-057b-46f0-b6a6-3ba957d1e261"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/open-in-web-dark.svg"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/open-in-web-light.svg"><img alt="Open in Web" src="https://cursor.com/open-in-web.svg"></picture></a> <!-- start pr-codex --> --- ## PR-Codex overview This PR introduces the `Etherlink Shadownet` testnet chain to the project, defining its properties and adding it to the exports. ### Detailed summary - Added a new chain definition for `etherlinkShadownet` in `etherlink-shadownet.ts`. - Defined properties such as `id`, `name`, `nativeCurrency`, and `blockExplorers`. - Exported `etherlinkShadownet` in `chains.ts` for use in the application. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **New Features** * Added support for Etherlink Shadownet testnet chain, enabling development and testing on this network with proper block explorer integration and native currency configuration (XTZ). <sub>✏️ Tip: You can customize this high-level summary in your review settings.</sub> <!-- end of auto-generated comment: release notes by coderabbit.ai --> * [MNY-342] Dashboard: Add Buy Widget iframe (thirdweb-dev#8613) <!-- ## title your PR with this format: "[SDK/Dashboard/Portal] Feature/Fix: Concise title for the changes" If you did not copy the branch name from Linear, paste the issue tag here (format is TEAM-0000): ## Notes for the reviewer Anything important to call out? Be sure to also clarify these in your comments. ## How to test Unit tests, playground, etc. --> <!-- start pr-codex --> --- ## PR-Codex overview This PR introduces a new `BuyWidget` feature, allowing users to purchase crypto tokens using various payment methods. It adds necessary environment variables, updates configuration for security policies, and implements the layout and functionality for the `BuyWidgetEmbed` component. ### Detailed summary - Added `NEXT_PUBLIC_BUY_IFRAME_CLIENT_ID` to `public-envs.ts`. - Updated `next.config.ts` with new Content Security Policy headers for `/bridge/buy-widget`. - Created `BuyWidgetLayout` component in `layout.tsx` for the buy widget page. - Implemented `BuyWidgetEmbed` component in `BuyWidgetEmbed.client.tsx` for handling crypto purchases. - Enhanced `page.tsx` to manage search parameters and integrate `BuyWidgetEmbed`. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit ## Release Notes * **New Features** * Added a new buy widget enabling direct cryptocurrency purchases with configurable payment methods (crypto and card) and multi-currency support * Implemented personalization options including custom titles, descriptions, images, and button labels * Enhanced security infrastructure to support the new widget functionality <sub>✏️ Tip: You can customize this high-level summary in your review settings.</sub> <!-- end of auto-generated comment: release notes by coderabbit.ai --> * SDK: token details screen UI tweaks (thirdweb-dev#8612) <!-- ## title your PR with this format: "[SDK/Dashboard/Portal] Feature/Fix: Concise title for the changes" If you did not copy the branch name from Linear, paste the issue tag here (format is TEAM-0000): ## Notes for the reviewer Anything important to call out? Be sure to also clarify these in your comments. ## How to test Unit tests, playground, etc. --> <!-- start pr-codex --> --- ## PR-Codex overview This PR focuses on improving the user interface of the `TokenInfoScreen` component in the `select-token-ui.tsx` file by enhancing loading states, adjusting layout heights, and refining token display logic. ### Detailed summary - Removed initial loading and token not found states. - Increased `Container` minHeight from `350px` to `450px`. - Refactored conditional rendering for token details. - Maintained structure for displaying token name, symbol, price, market cap, volume, and contract address. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Refactor** * Consolidated rendering logic for the token information view to centralize pending, not-found, and content states. * Reorganized layout so detailed token sections render only when token data is present. * Adjusted minimum container height for token information screens in the bridge swap widget. <sub>✏️ Tip: You can customize this high-level summary in your review settings.</sub> <!-- end of auto-generated comment: release notes by coderabbit.ai --> * [MNY-354] Portal: Add Buy Widget iframe docs (thirdweb-dev#8614) <!-- ## title your PR with this format: "[SDK/Dashboard/Portal] Feature/Fix: Concise title for the changes" If you did not copy the branch name from Linear, paste the issue tag here (format is TEAM-0000): ## Notes for the reviewer Anything important to call out? Be sure to also clarify these in your comments. ## How to test Unit tests, playground, etc. --> <!-- start pr-codex --> --- ## PR-Codex overview This PR introduces the `Buy Widget`, enhancing the portal with a new iframe component for purchasing crypto. It includes documentation, features, and examples for users to integrate the widget easily. ### Detailed summary - Added `IframeCodePreview` component in `iframe-code-preview.tsx`. - Updated sidebar links to reflect the new `Buy Widget` and `Swap Widget`. - Created documentation for the `Buy Widget` in `page.mdx`, detailing features and integration. - Added example usage of the `BuyWidget` component in `react/page.mdx`. - Developed `iframe/page.mdx` with customization options and event listening for the widget. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **New Features** * Added an iframe embed preview with code/preview tabs for trying the Buy widget. * **Documentation** * Added comprehensive Buy widget docs (overview, React and iframe guides, examples, customization, event listening, live playgrounds). * **Chores** * Updated site navigation: Widgets section reorganized to list Swap, Buy, and Checkout widgets with updated links. <sub>✏️ Tip: You can customize this high-level summary in your review settings.</sub> <!-- end of auto-generated comment: release notes by coderabbit.ai --> * [MNY-355] Playground: Add Buy Widget iframe playground (thirdweb-dev#8615) <!-- ## title your PR with this format: "[SDK/Dashboard/Portal] Feature/Fix: Concise title for the changes" If you did not copy the branch name from Linear, paste the issue tag here (format is TEAM-0000): ## Notes for the reviewer Anything important to call out? Be sure to also clarify these in your comments. ## How to test Unit tests, playground, etc. --> <!-- start pr-codex --> --- ## PR-Codex overview This PR focuses on enhancing the `buy` widget functionality by introducing iframe support and improving the handling of integration types. It also adds new components for rendering iframe previews and updates the URL management for better integration experience. ### Detailed summary - Updated `LeftSection.tsx` to allow both `checkout` and `buy` widgets for iframe integration. - Modified `Page` component in `page.tsx` to handle async `searchParams` and pass `defaultTab` to `BuyPlayground`. - Enhanced `CodeGen` to support widget type in iframe code generation. - Added `CheckoutIframePreview` and `BuyWidgetIframePreview` components in `RightSection.tsx` for rendering respective iframes. - Created `buildBuyIframeUrl` function in `buildBuyIframeUrl.ts` for generating URLs for the buy widget. - Updated `BuyPlayground` to manage state and URL for integration type selection using tabs. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex --> * Enable multiple ref params for bytes input (thirdweb-dev#8610) <!-- ## title your PR with this format: "[SDK/Dashboard/Portal] Feature/Fix: Concise title for the changes" If you did not copy the branch name from Linear, paste the issue tag here (format is TEAM-0000): ## Notes for the reviewer Anything important to call out? Be sure to also clarify these in your comments. ## How to test Unit tests, playground, etc. --> <!-- start pr-codex --> --- ## PR-Codex overview This PR focuses on modifying the `Button` component in the `decoded-input-set.tsx` file to enhance its functionality based on certain conditions related to `param.type` and `fields`. ### Detailed summary - Updated the `Button` component to disable it when `param.type` is "bytes" and `fields.length` is 1 or more. - Maintained the existing `onClick` functionality to append a new input with a default value of an empty string. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Bug Fixes** * Fixed an issue in the contract publish form where the Add Parameter button was incorrectly disabled when working with bytes parameters. Users can now add multiple bytes parameters without restriction. <sub>✏️ Tip: You can customize this high-level summary in your review settings.</sub> <!-- end of auto-generated comment: release notes by coderabbit.ai --> * Preserve wallet address on vault credential rotation (thirdweb-dev#8618) * Add self-managed vault recovery option for lost secret key (thirdweb-dev#8619) * [MNY-356] Add tokenEditable and amountEditable props on BuyWidget (thirdweb-dev#8621) <!-- ## title your PR with this format: "[SDK/Dashboard/Portal] Feature/Fix: Concise title for the changes" If you did not copy the branch name from Linear, paste the issue tag here (format is TEAM-0000): ## Notes for the reviewer Anything important to call out? Be sure to also clarify these in your comments. ## How to test Unit tests, playground, etc. --> <!-- start pr-codex --> --- ## PR-Codex overview This PR introduces two new properties, `amountEditable` and `tokenEditable`, to enhance the `BuyWidget` component, allowing control over editing options for token selection and amount. This improves user experience by providing flexibility in the widget's functionality. ### Detailed summary - Added `amountEditable` and `tokenEditable` properties to `BuyWidget`. - Updated `BuyPlayground.tsx`, `CheckoutPlayground.tsx`, and `TransactionPlayground.tsx` to use new props. - Modified `CodeGen.tsx` to incorporate editability options. - Updated URL generation in `buildBuyIframeUrl.ts` for new props. - Added editability options in `LeftSection.tsx`. - Enhanced `BuyWidgetEmbed` to accept new props. - Implemented stories in `BuyWidget.stories.tsx` for editable and non-editable states. - Added documentation in `iframe/page.mdx` for new features. - Adjusted `FundWallet.tsx` and `TokenSection.tsx` to respect new editability options. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit ## Release Notes * **New Features** * Added `amountEditable` and `tokenEditable` props to the BuyWidget component, allowing developers to disable token selection and/or amount editing in the buy flow. * Enhanced UI controls to prevent interaction with locked fields when these options are disabled. * Updated documentation with examples of how to use the new editability options. * **Tests** * Added story variants demonstrating token and amount lock scenarios. <sub>✏️ Tip: You can customize this high-level summary in your review settings.</sub> <!-- end of auto-generated comment: release notes by coderabbit.ai --> * [SDK] Fix: Parse x402 payment header (thirdweb-dev#8617) Co-authored-by: Joaquim Verges <joaquim.verges@gmail.com> * [MNY-233] SDK: Add Last Used badge in Connect UI (thirdweb-dev#8631) * support x402 v2 headers (thirdweb-dev#8623) * Version Packages (thirdweb-dev#8622) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * Add fallback for gas fee estimation in zkSync (thirdweb-dev#8637) * Version Packages (thirdweb-dev#8638) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * Fix getCallStatus via EIP1193 provider (thirdweb-dev#8639) * Version Packages (thirdweb-dev#8640) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * Support linking accounts with redirect mode (thirdweb-dev#8643) * Add base account sdk (thirdweb-dev#8634) Co-authored-by: Joaquim Verges <joaquim.verges@gmail.com> * Version Packages (thirdweb-dev#8644) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * portal: landing page ui improvements (thirdweb-dev#8647) --------- Co-authored-by: MananTank <manantankm@gmail.com> Co-authored-by: Yash094 <67926590+Yash094@users.noreply.github.com> Co-authored-by: kumaryash90 <kumaryashcse@gmail.com> Co-authored-by: Firekeeper <0xFirekeeper@gmail.com> Co-authored-by: Jelilat Anofiu <anofiujelailah@gmail.com> Co-authored-by: Joaquim Verges <joaquim.verges@gmail.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Montana <montanawong@gmail.com>
This comment was marked as resolved.
This comment was marked as resolved.
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
There was a problem hiding this comment.
Sorry @Dargon789, your pull request is larger than the review limit of 150000 diff characters
Summary of ChangesHello @Dargon789, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request introduces a new 'Year in Review' feature to the dashboard, enhancing user engagement by summarizing their yearly activity. It also updates the x402 payment protocol to support a new header, integrates the Base Account SDK for new wallet options, and refines the Connect Wallet UI with 'Last used' indicators. Additionally, it includes improvements for zkSync fee estimation and in-app wallet linking. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces a "Rewind" feature with new UI components, API routes, and analytics. It also adds support for the Base Account SDK wallet, a new account linking flow, and a "Last Used" badge for wallet selection, which improves user experience. Additionally, it updates the x402 payment protocol implementation to support v2 headers. The code is generally well-structured. I've identified a few issues to address: an incorrect redirect in an API route, a magic number for animation timing that should be a constant for better maintainability, and an unused variable. My review comments provide more details on these points. Overall, this is a solid set of updates.
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com> Signed-off-by: Dargon789 <64915515+Dargon789@users.noreply.github.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![https://cursor.com/open-in-cursor-dark.svg"><source](https://cursor.com/open-in-cursor-dark.svg"><source){kind=link}
![https://cursor.com/open-in-cursor-light.svg"><img](https://cursor.com/open-in-cursor-light.svg"><img){kind=link}
![https://cursor.com/open-in-cursor.svg"></picture></a> <a](https://cursor.com/open-in-cursor.svg"></picture></a> <a){kind=link}
![https://cursor.com/open-in-web-dark.svg"><source](https://cursor.com/open-in-web-dark.svg"><source){kind=link}
![https://cursor.com/open-in-web-light.svg"><img](https://cursor.com/open-in-web-light.svg"><img){kind=link}
![https://cursor.com/open-in-web.svg"></picture></a](https://cursor.com/open-in-web.svg"></picture></a){kind=link}
This was
linked to
issues
Feb 1, 2026
Closed
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
5 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.
No description provided.