-
Notifications
You must be signed in to change notification settings - Fork 327
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
Move /connect/in-app-wallets to app router #4894
Move /connect/in-app-wallets to app router #4894
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @MananTank and the rest of your teammates on Graphite |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4894 +/- ##
=======================================
Coverage 47.61% 47.61%
=======================================
Files 1057 1057
Lines 57221 57221
Branches 3924 3924
=======================================
Hits 27246 27246
Misses 29286 29286
Partials 689 689
*This pull request uses carry forward flags. Click here to find out more. |
size-limit report 📦
|
Your org has enabled the Graphite merge queue for merging into mainAdd the label “merge-queue” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix. You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link. |
f95fa52
to
f20b046
Compare
d2d37d1
to
d898e86
Compare
d898e86
to
9f2b083
Compare
9382059
to
22ee12e
Compare
9f2b083
to
ff5b14a
Compare
apps/dashboard/src/app/(dashboard)/dashboard/connect/in-app-wallets/[clientId]/page.tsx
Outdated
Show resolved
Hide resolved
6e09ef9
to
7b11876
Compare
b70a030
to
615dd1e
Compare
7b11876
to
23cd49d
Compare
23cd49d
to
d8ff07e
Compare
Merge activity
|
## Problem solved Short description of the bug fixed or feature added <!-- start pr-codex --> --- ## PR-Codex overview This PR focuses on enhancing the `in-app-wallets` feature by adding new components and functionalities, including API key management and layout improvements. ### Detailed summary - Added `getInAppWalletSupportedAPIKeys` function to fetch and filter API keys. - Created `Layout` component with footer sections. - Implemented `InAppWalletsAPIKeysMenu` for API key selection. - Added `PageHeader` with description and link. - Enhanced `Page` component to handle redirects and display API keys. - Updated `EmbeddedWallets` component to use `defaultTab` prop instead of search params. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex -->
d8ff07e
to
b82011e
Compare
Problem solved
Short description of the bug fixed or feature added
PR-Codex overview
This PR introduces a new
in-app-wallets
feature to the dashboard, including support for API keys, a layout for the wallet interface, and a header component. It also refines navigation and user experience for managing in-app wallets.Detailed summary
getInAppWalletSupportedAPIKeys
function to fetch API keys.Layout
component for the wallet interface.InAppWalletsAPIKeysMenu
for API key selection.PageHeader
component for in-app wallets page.Page
to handle authentication and API key redirects.EmbeddedWallets
component to use a default tab.