thirdweb-dev / js

Best in class web3 SDKs for Browser, Node and Mobile apps
https://thirdweb.com
Apache License 2.0
379 stars 259 forks source link

feature(dashboard): ecosystem wallets #3537

Closed gregfromstl closed 3 days ago

gregfromstl commented 1 week ago

PR-Codex overview

This PR introduces new error classes, updates routing, adds UI components, and refactors user authentication logic.

Detailed summary

The following files were skipped due to too many changes: apps/dashboard/src/@3rdweb-sdk/react/hooks/useLoggedInUser.ts, apps/dashboard/src/app/(dashboard)/dashboard/connect/ecosystem/[slug]/(active)/hooks/use-ecosystem.ts, apps/dashboard/src/app/(dashboard)/dashboard/connect/ecosystem/hooks/use-ecosystem-list.ts, apps/dashboard/src/app/components/Header/DashboardHeader.tsx, apps/dashboard/src/app/(dashboard)/dashboard/connect/ecosystem/[slug]/(active)/layout.tsx, apps/dashboard/src/app/(dashboard)/dashboard/components/sidebar/sidebar.client.tsx, apps/dashboard/src/@/components/ui/form.tsx, apps/dashboard/src/@/components/ui/ConfirmationDialog.tsx, apps/dashboard/src/app/components/DashboardFooter.tsx, apps/dashboard/src/app/(dashboard)/dashboard/connect/ecosystem/[slug]/(active)/hooks/use-delete-partner.ts, apps/dashboard/src/@/components/ui/image-upload.tsx, apps/dashboard/src/app/(dashboard)/dashboard/connect/ecosystem/[slug]/(active)/hooks/use-update-ecosystem.ts, apps/dashboard/src/app/(dashboard)/dashboard/connect/ecosystem/create/page.tsx, apps/dashboard/src/app/(dashboard)/(chain)/chainlist/components/server/chainlist-card.tsx, apps/dashboard/src/app/(dashboard)/dashboard/connect/ecosystem/create/hooks/use-create-ecosystem.ts, apps/dashboard/src/@/components/ui/radio-group.tsx, apps/dashboard/src/app/(dashboard)/dashboard/connect/ecosystem/[slug]/(active)/hooks/use-add-partner.ts, apps/dashboard/src/app/(dashboard)/(chain)/chainlist/components/client/filters.tsx, apps/dashboard/src/app/(dashboard)/dashboard/connect/ecosystem/[slug]/(active)/components/client/integration-permissions-toggle.client.tsx, apps/dashboard/src/app/(dashboard)/dashboard/connect/ecosystem/[slug]/(active)/components/client/ecosystem-header.client.tsx, apps/dashboard/src/app/(dashboard)/dashboard/connect/ecosystem/page.tsx, apps/dashboard/src/app/(dashboard)/dashboard/connect/ecosystem/[slug]/(active)/components/server/partners-table.tsx, apps/dashboard/src/@/components/ui/alert-dialog.tsx, apps/dashboard/src/app/(dashboard)/dashboard/connect/ecosystem/create/components/client/create-ecosystem-form.client.tsx, apps/dashboard/src/app/(dashboard)/dashboard/connect/ecosystem/[slug]/(active)/components/client/add-partner-form.client.tsx, apps/dashboard/src/@/components/ui/dropdown-menu.tsx, pnpm-lock.yaml

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

vercel[bot] commented 1 week ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
thirdweb_playground ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 2, 2024 9:32pm
thirdweb-www ✅ Ready (Inspect) Visit Preview 15 resolved Jul 2, 2024 9:32pm
wallet-ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 2, 2024 9:32pm
changeset-bot[bot] commented 1 week ago

⚠️ No Changeset found

Latest commit: 4786c2da72346d6cb2f18e3c3f9ac2989238388e

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

graphite-app[bot] commented 1 week ago

Your org requires the Graphite merge queue for merging into main

Add 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.

gregfromstl commented 1 week ago

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @gregfromstl and the rest of your teammates on Graphite Graphite

codecov[bot] commented 1 week ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 62.75%. Comparing base (6b919e1) to head (4786c2d).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #3537 +/- ## ========================================== - Coverage 62.77% 62.75% -0.03% ========================================== Files 897 897 Lines 68735 68735 Branches 3687 3675 -12 ========================================== - Hits 43151 43132 -19 - Misses 24895 24915 +20 + Partials 689 688 -1 ``` | [Flag](https://app.codecov.io/gh/thirdweb-dev/js/pull/3537/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thirdweb-dev) | Coverage Δ | | *Carryforward flag | |---|---|---|---| | [legacy_packages](https://app.codecov.io/gh/thirdweb-dev/js/pull/3537/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thirdweb-dev) | `65.59% <ø> (ø)` | | Carriedforward from [6b919e1](https://app.codecov.io/gh/thirdweb-dev/js/commit/6b919e13859e146f572619bfb315d5c357808a0e?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thirdweb-dev) | | [packages](https://app.codecov.io/gh/thirdweb-dev/js/pull/3537/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thirdweb-dev) | `62.21% <ø> (-0.04%)` | :arrow_down: | | *This pull request uses carry forward flags. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thirdweb-dev) to find out more. [see 5 files with indirect coverage changes](https://app.codecov.io/gh/thirdweb-dev/js/pull/3537/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thirdweb-dev)
codspeed-hq[bot] commented 1 week ago

CodSpeed Performance Report

Merging #3537 will not alter performance

Comparing feature/ecosystem-create-form (4786c2d) with main (6b919e1)

Summary

✅ 9 untouched benchmarks

github-actions[bot] commented 1 week ago

size-limit report 📦

Path Size Loading time (3g) Running time (snapdragon) Total time
thirdweb (esm) 40.52 KB (0%) 811 ms (0%) 3.6 s (+1.84% 🔺) 4.5 s
thirdweb (cjs) 89.96 KB (0%) 1.8 s (0%) 7.3 s (+21.04% 🔺) 9.1 s
thirdweb (minimal + tree-shaking) 4.79 KB (0%) 96 ms (0%) 395 ms (+131.53% 🔺) 491 ms
thirdweb/chains (tree-shaking) 423 B (0%) 10 ms (0%) 84 ms (-13.64% 🔽) 94 ms
thirdweb/react (minimal + tree-shaking) 13.52 KB (0%) 271 ms (0%) 409 ms (-30.58% 🔽) 680 ms
jnsdls commented 3 days ago

only blocker is API needs to be rolled out to prod first

graphite-app[bot] commented 3 days ago

Merge activity