-
Notifications
You must be signed in to change notification settings - Fork 423
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
[TOOL-3007] Dashboard: Replace chainsaw with insight #5926
[TOOL-3007] Dashboard: Replace chainsaw with insight #5926
Conversation
🦋 Changeset detectedLatest commit: 4b5661b The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
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. |
cc3f53f
to
ca7016e
Compare
size-limit report 📦
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5926 +/- ##
==========================================
+ Coverage 54.81% 54.83% +0.01%
==========================================
Files 1141 1141
Lines 60903 60903
Branches 5126 5128 +2
==========================================
+ Hits 33384 33396 +12
+ Misses 26795 26781 -14
- Partials 724 726 +2
*This pull request uses carry forward flags. Click here to find out more. |
Merge activity
|
<!-- start pr-codex --> ## PR-Codex overview This PR focuses on enhancing the analytics functionality within the `dashboard` application by introducing new utility functions, updating existing API calls, and improving the data handling for contract analytics. ### Detailed summary - Added export of `toEventSelector` from `thirdweb/utils`. - Introduced `INSIGHT_SERVICE_API_KEY` in environment constants. - Modified `ContractAnalyticsPageClient` to accept new props for function and event selectors. - Created new analytics functions for total contract events, unique wallets, and transactions. - Updated `isAnalyticsSupportedForChain` to use `INSIGHT_SERVICE_API_KEY`. - Refactored analytics components to use new hooks for contract events and functions. - Improved data fetching and error handling in analytics hooks. - Enhanced UI components for displaying analytics data with loading states. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex -->
ca7016e
to
4b5661b
Compare
PR-Codex overview
This PR focuses on enhancing the analytics features of the
thirdweb
project. Key updates include the addition of new utility functions, improved data fetching for contract analytics, and adjustments to various components to support new data structures and API endpoints.Detailed summary
toEventSelector
utility function export.INSIGHT_SERVICE_API_KEY
for analytics API calls.ContractAnalyticsPageClient
to accept new props for function and event selectors.isAnalyticsSupportedForChain
to use the new API key.