Skip to content

improve SDK metric reporting#68

Merged
Vadman97 merged 14 commits intomainfrom
vadim/sdk
May 30, 2025
Merged

improve SDK metric reporting#68
Vadman97 merged 14 commits intomainfrom
vadim/sdk

Conversation

@Vadman97
Copy link
Contributor

@Vadman97 Vadman97 commented May 30, 2025

Summary

  • Correct error stacktrace reporting from new opentelemetry trace-based implementation
  • Fix reporting 0-value document_load durations
  • Fix document_load metrics not foorwarding to ld client
  • Refactor metrics to live in Observability plugin
  • Default to strict privacy mode.

How did you test this change?

local deploy of react-router e2e app

Are there any deployment considerations?

changesets

Does this work require review from our design team?

no

@Vadman97 Vadman97 changed the title Vadim/sdk improve SDK metric reporting May 30, 2025
@Vadman97 Vadman97 marked this pull request as ready for review May 30, 2025 19:45
@Vadman97 Vadman97 requested a review from a team as a code owner May 30, 2025 19:45
@Vadman97 Vadman97 enabled auto-merge (squash) May 30, 2025 20:00
@Vadman97 Vadman97 force-pushed the vadim/sdk branch 2 times, most recently from faad814 to f79b8b4 Compare May 30, 2025 22:40
@Vadman97 Vadman97 merged commit c901c22 into main May 30, 2025
16 checks passed
@Vadman97 Vadman97 deleted the vadim/sdk branch May 30, 2025 22:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants