Skip to content

feat: Update usage reporting to v2 pattern #847

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

Closed
wants to merge 1 commit into from
Closed

Conversation

arcoraven
Copy link
Contributor

@arcoraven arcoraven commented Feb 9, 2025

PR-Codex overview

This PR focuses on updating dependencies, enhancing analytics reporting functionality, and improving the structure of the usage reporting system in the project.

Detailed summary

  • Updated @thirdweb-dev/service-utils to a nightly version.
  • Added ENABLE_USAGE_V2_ANALYTICS to environment variables.
  • Refactored usage reporting functions to support both V1 and V2 analytics.
  • Renamed ReportUsageParams to ReportTransactionUsageParams.
  • Enhanced error handling and logging in reporting functions.
  • Introduced reportUsageV2 function for new analytics reporting.

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

Copy link

zeet-co bot commented Feb 9, 2025

We're building your pull request over on Zeet.
Click me for more info about your build and deployment.
Once built, this branch can be tested at: https://tw-unreal-demo-engine-vwn0-ph-usagev2.engine-aws-usw2.zeet.app before merging 😉

@arcoraven arcoraven requested a review from d4mr February 9, 2025 10:29
Copy link

New, updated, and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@thirdweb-dev/service-utils@0.8.8-nightly-cad9c74befe33bef2cc6a12a8ade8d9c996ffe41-20250209100117 🔁 npm/@thirdweb-dev/service-utils@0.4.52 None 0 210 kB thirdweb-bot
npm/kafkajs@2.2.4 environment, network 0 732 kB tulios
npm/lz4js@0.2.0 None 0 36.4 kB johnwchadwick

View full report↗︎

Copy link

This PR is stale because it has been open for 7 days with no activity. Remove stale label or comment or this PR will be closed in 3 days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants