-
Notifications
You must be signed in to change notification settings - Fork 15
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
wip: Web Vitals Autoinstrumentation #108
wip: Web Vitals Autoinstrumentation #108
Conversation
Hi @pkanal, do you have a timeline on this package? We're looking to incorporate the web-vitals library in otel as well and wanted to know whether we should pick it up in parallel, as we'll need it in the near future. Also, for the CLS (and FID) metric, are we planning on keeping the default implementations for them? I know that they're dependent on user interactions, and so might never get sent until very late in the webpage's lifespan. Just wanted to know if there was any discussion on this edge case. |
@shumone There's no specific timeline but I am actively working on this. The plan with the OTel Client Side SIG is to create a proof of concept in this sandbox repo and once we're happy with the API here, we will contribute this to the opentelemetry-js-contrib repo. If you're planning on working on this, we'd definitely be happy to collaborate! At the moment we're planning on keeping the default implementation for CLS and FID. If you have any feedback for how to handle them differently feel free to comment on the PR for how they could be handled! |
c53b34f
to
ebed67b
Compare
f53a76a
to
c45f0aa
Compare
7dba1f0
to
4ed5d6b
Compare
62f8681
into
open-telemetry:auto-instrumentation-poc
No description provided.