Feature: Migrate Universal Bridge Analytics #6988
Conversation
|
|
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. |
size-limit report 📦
|
gregfromstl
force-pushed
the
greg/tool-4164-migrate-dashboard-analytics
branch
from
6c9621d to
becd948
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #6988 +/- ##
=======================================
Coverage 54.60% 54.60%
=======================================
Files 903 903
Lines 57934 57934
Branches 3970 3968 -2
=======================================
+ Hits 31632 31636 +4
+ Misses 26202 26198 -4
Partials 100 100
🚀 New features to boost your workflow:
|
| const isEmpty = | ||
| !props.data || | ||
| props.data.length === 0 || | ||
| props.data.every((x) => x.developerFeeUsdCents === 0); |
There was a problem hiding this comment.
did you tranform it to cents somewhere? remember in usage its USD, not cents
There was a problem hiding this comment.
Yeah its transformed to cents in the analytics API to be consistent with the amount field
|
|
||
| data = (() => { | ||
| if (successType === "fail") { | ||
| return data.filter((x) => x.status === "failed"); |
There was a problem hiding this comment.
just a nit: noticed that on an empty dashboard, it shows the bar fully red, with 0 success 0 failed
| </div> | ||
| ); | ||
| } | ||
| import {} from "@/components/ui/select"; |
gregfromstl
force-pushed
the
greg/tool-4164-migrate-dashboard-analytics
branch
from
85e9a2d to
478ff96
Compare
PR-Codex overview
This PR focuses on refactoring the
PayAnalyticscomponent and related functionalities to integrate universal bridge statistics, while removing deprecated code and enhancing the data handling for various analytics components.Detailed summary
PayAnalytics.NEXT_PUBLIC_PAY_URLconstant.getUniversalBridgeFiltersFromSearchParamsfunction.UniversalBridgeStatsandUniversalBridgeWalletStats.PayAnalyticsto fetch universal bridge data.PaymentsSuccessRate,TotalVolumePieChart,Payouts, andPayCustomersTableto use new data structures.PayAnalyticsFilterto utilize universal bridge filters.