Skip to content

Conversation

@joaquim-verges
Copy link
Member

@joaquim-verges joaquim-verges commented Jan 22, 2025

Fixes TOOL-3174 TOOL-3175 TOOL-3176


PR-Codex overview

This PR focuses on enhancing transaction analytics by introducing new functionalities for tracking and displaying client transactions. It modifies existing components and introduces new interfaces to support these changes.

Detailed summary

  • Changed title in TotalSponsoredCard from "Total Sponsored" to "Gas Sponsored".
  • Added TransactionStats interface in analytics.ts.
  • Introduced getClientTransactions function for fetching client transaction stats.
  • Updated PieChartCard to support link for labels.
  • Integrated TransactionsChartCardUI in analytics pages to display client transactions.
  • Created ChainDistributionCard and ContractDistributionCard for visualizing transaction data by chain and contract.
  • Modified TotalSponsoredCard to reflect new transaction metrics.

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

@vercel
Copy link

vercel bot commented Jan 22, 2025

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 Jan 22, 2025 6:08pm
thirdweb_playground ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 22, 2025 6:08pm
thirdweb-www ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 22, 2025 6:08pm
wallet-ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 22, 2025 6:08pm

@github-actions github-actions bot added Dashboard Involves changes to the Dashboard. packages SDK Involves changes to the thirdweb SDK labels Jan 22, 2025
@joaquim-verges joaquim-verges changed the title feat: Add transaction analytics and tracking [WIP] feat: Add transaction analytics and tracking Jan 22, 2025
Copy link
Member Author


How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • merge-queue - adds this PR to the back of the merge queue
  • hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

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

@joaquim-verges joaquim-verges marked this pull request as ready for review January 22, 2025 01:55
@joaquim-verges joaquim-verges requested review from a team as code owners January 22, 2025 01:56
@github-actions
Copy link
Contributor

github-actions bot commented Jan 22, 2025

size-limit report 📦

Path Size Loading time (3g) Running time (snapdragon) Total time
thirdweb (esm) 46.68 KB (0%) 934 ms (0%) 2.7 s (-1.08% 🔽) 3.7 s
thirdweb (cjs) 116.87 KB (0%) 2.4 s (0%) 7.1 s (+32.2% 🔺) 9.4 s
thirdweb (minimal + tree-shaking) 5.59 KB (0%) 112 ms (0%) 201 ms (-37.11% 🔽) 313 ms
thirdweb/chains (tree-shaking) 506 B (0%) 10 ms (0%) 107 ms (-22% 🔽) 117 ms
thirdweb/react (minimal + tree-shaking) 19.22 KB (0%) 385 ms (0%) 850 ms (+25.66% 🔺) 1.3 s

@codecov
Copy link

codecov bot commented Jan 22, 2025

Codecov Report

Attention: Patch coverage is 69.44444% with 11 lines in your changes missing coverage. Please review.

Project coverage is 54.89%. Comparing base (8bbee03) to head (3d76705).
Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
packages/thirdweb/src/wallets/smart/index.ts 69.44% 10 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6019      +/-   ##
==========================================
- Coverage   54.89%   54.89%   -0.01%     
==========================================
  Files        1151     1151              
  Lines       61175    61209      +34     
  Branches     5155     5159       +4     
==========================================
+ Hits        33580    33598      +18     
- Misses      26868    26883      +15     
- Partials      727      728       +1     
Flag Coverage Δ *Carryforward flag
legacy_packages 65.68% <ø> (ø) Carriedforward from 8bbee03
packages 52.55% <69.44%> (+<0.01%) ⬆️

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

Files with missing lines Coverage Δ
packages/thirdweb/src/wallets/smart/index.ts 57.94% <69.44%> (+0.83%) ⬆️

... and 2 files with indirect coverage changes

@changeset-bot
Copy link

changeset-bot bot commented Jan 22, 2025

⚠️ No Changeset found

Latest commit: 3d76705

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

@joaquim-verges joaquim-verges force-pushed the feat_Add_transaction_analytics_and_tracking branch from 8bf79ed to ab6bd0e Compare January 22, 2025 08:17
@joaquim-verges joaquim-verges changed the title [WIP] feat: Add transaction analytics and tracking [Dashboard] feat: Add transaction analytics and tracking Jan 22, 2025
@linear
Copy link

linear bot commented Jan 22, 2025

CardFooter,
CardHeader,
} from "@/components/ui/card";
import { Link } from "../../../../tw-components";
Copy link
Member

@MananTank MananTank Jan 22, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use Link from next/link package instead

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@MananTank MananTank requested a review from gregfromstl January 22, 2025 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Dashboard Involves changes to the Dashboard. packages SDK Involves changes to the thirdweb SDK

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants