js icon indicating copy to clipboard operation
js copied to clipboard

feat(wallet-ui): initial app

Open gregfromstl opened this issue 1 year ago • 7 comments

  1. Created a new .env.example file specifying the NEXT_PUBLIC_ROOT_DOMAIN variable.
  2. Updated .gitignore to include .env*.local instead of .env*.
  3. Modified the Layout component in [ecosystem]/layout.tsx to accept children and params properties and display the ecosystem and children elements.
  4. Updated the Page component in [ecosystem]/page.tsx to return a simple div with the text Page.
  5. Added a new middleware.ts file to handle dynamic routing based on the request host, matching all paths except for specific routes (e.g., /api, /_next, /_static). Rewrites the URL to point to either the root application or the appropriate ecosystem dynamic route.


PR-Codex overview

This PR adds Thirdweb integration, tailwind config updates, UI components, and ecosystem API calls.

Detailed summary

  • Added Thirdweb integration for client and components
  • Updated tailwind config for dark mode and animations
  • Introduced UI components like Skeleton, Card, Toaster, Label, Input
  • Implemented ecosystem API calls for info retrieval

The following files were skipped due to too many changes: apps/wallet-ui/package.json, apps/wallet-ui/src/components/ui/badge.tsx, apps/wallet-ui/src/app/[ecosystem]/(ui)/layout.tsx, apps/wallet-ui/src/middleware.ts, apps/wallet-ui/src/components/ui/popover.tsx, apps/wallet-ui/src/app/[ecosystem]/(ui)/wallet/[address]/layout.tsx, apps/wallet-ui/src/lib/assets/erc20.ts, apps/wallet-ui/src/components/ui/button.tsx, apps/wallet-ui/src/components/NftGallery.tsx, apps/wallet-ui/src/components/NftCard.tsx, apps/wallet-ui/src/lib/assets/erc721.ts, apps/wallet-ui/src/util/simplehash.ts, apps/wallet-ui/src/components/ChainCombobox.tsx, apps/wallet-ui/src/components/TransactionToastProvider.tsx, apps/wallet-ui/src/components/ui/use-toast.ts, apps/wallet-ui/src/app/globals.css, apps/wallet-ui/src/components/ui/dialog.tsx, apps/wallet-ui/src/components/ChainIcon.tsx, apps/wallet-ui/src/components/ui/form.tsx, apps/wallet-ui/src/components/NftModal.tsx, apps/wallet-ui/src/components/ui/toast.tsx, apps/wallet-ui/src/components/ui/command.tsx, pnpm-lock.yaml

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

gregfromstl avatar Jun 25 '24 00:06 gregfromstl

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

Name Status Preview Comments Updated (UTC)
docs-v2 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 17, 2024 8:31am
thirdweb_playground ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 17, 2024 8:31am
thirdweb-www ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 17, 2024 8:31am
wallet-ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 17, 2024 8:31am

vercel[bot] avatar Jun 25 '24 00:06 vercel[bot]

⚠️ No Changeset found

Latest commit: 2e99057f90ebed481152ba0036a103b9d38fb73b

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

changeset-bot[bot] avatar Jun 25 '24 00:06 changeset-bot[bot]

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.

graphite-app[bot] avatar Jun 25 '24 00:06 graphite-app[bot]

  • #3498 Graphite: 2 dependent PRs (#3499 Graphite, #3511 Graphite)
  • #3478 Graphite 👈
  • #3477 Graphite
  • main

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

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

gregfromstl avatar Jun 25 '24 00:06 gregfromstl

Codecov Report

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

Project coverage is 59.11%. Comparing base (13764f1) to head (2e99057). Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3478      +/-   ##
==========================================
- Coverage   59.12%   59.11%   -0.02%     
==========================================
  Files         958      958              
  Lines       76513    76513              
  Branches     3468     3468              
==========================================
- Hits        45241    45233       -8     
- Misses      30599    30606       +7     
- Partials      673      674       +1     
Flag Coverage Δ *Carryforward flag
legacy_packages 65.68% <ø> (ø) Carriedforward from 13764f1
packages 58.02% <ø> (-0.02%) :arrow_down:

*This pull request uses carry forward flags. Click here to find out more.

see 5 files with indirect coverage changes

codecov[bot] avatar Jun 25 '24 00:06 codecov[bot]

CodSpeed Performance Report

Merging #3478 will not alter performance

Comparing 06-24-feat_wallet-ui_setup_middleware_redirects (c4eb27a) with main (77b852a)

Summary

✅ 9 untouched benchmarks

codspeed-hq[bot] avatar Jun 25 '24 00:06 codspeed-hq[bot]

size-limit report 📦

Path Size Loading time (3g) Running time (snapdragon) Total time
thirdweb (esm) 43.06 KB (0%) 862 ms (0%) 2.5 s (-29.4% 🔽) 3.4 s
thirdweb (cjs) 92.24 KB (0%) 1.9 s (0%) 9.1 s (+33.81% 🔺) 10.9 s
thirdweb (minimal + tree-shaking) 4.81 KB (0%) 97 ms (0%) 221 ms (+0.41% 🔺) 318 ms
thirdweb/chains (tree-shaking) 492 B (0%) 10 ms (0%) 52 ms (-52.09% 🔽) 62 ms
thirdweb/react (minimal + tree-shaking) 13.96 KB (0%) 280 ms (0%) 370 ms (-14.91% 🔽) 650 ms

github-actions[bot] avatar Jun 25 '24 00:06 github-actions[bot]

New dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@radix-ui/[email protected] None +13 419 kB vladmoroz
npm/@tanstack/[email protected] environment +1 3.12 MB tannerlinsley
npm/[email protected] Transitive: environment +28 992 kB paco

View full report↗︎

socket-security[bot] avatar Aug 09 '24 22:08 socket-security[bot]

Merge activity

  • Aug 17, 4:04 AM EDT: The merge label 'merge-queue' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
  • Aug 17, 4:04 AM EDT: gregfromstl added this pull request to the Graphite merge queue.
  • Aug 17, 4:31 AM EDT: gregfromstl merged this pull request with the Graphite merge queue.

graphite-app[bot] avatar Aug 17 '24 08:08 graphite-app[bot]