edwardgou-sentry

Results 10 comments of edwardgou-sentry

> > As for your other ask to overwrite the span op and span description, I don't believe there is a way to do so right now in the SDKs....

> Nice, The light vs non light is super uninutitive imo, and we run into this situation a good amount. > > We can probably add a comment here why...

Hi @SouthLink , can you try updating to the [7.114.0](https://github.com/getsentry/sentry-javascript/releases/tag/7.114.0) version of the sdk? There are some fixes in the newer versions that may help with capturing webvitals. Also, can...

Hi @SouthLink , I don't see anything wrong with the sentry configuration you posted and seems like you've updated to a more recent version of the sdk. There's a couple...

Hi @SouthLink , thanks for sharing your webpage. I viewed your page on a desktop browser and was able to trigger some INP events by clicking around, so I don't...

Planning to use this as a route name tag on inp spans.

Resulting envelope should look something like this ``` {"sent_at":"2024-02-13T15:14:01.049Z"} {"type":"span"} { "data": { "sentry.origin": "manual", "sentry.op": "ui.interaction.click", "measurements": { "inp": { "value": 120, "unit": "millisecond" } }, "release": "frontend@d9135c6e4543124bb23930bb8054db3a96a6069e", "environment":...

Some context around why the changes in this PR are needed Decision Doc on sending INP using standalone spans https://www.notion.so/sentry/Ingesting-and-Storing-INP-8c665f20577f44268a3a06e0f4e4d1c2 Relay changes were needed to consume standalone spans including extracting...

Closing this to break out into smaller prs and base off `v7` branch: The following 3 PRs can probably be reviewed individually: https://github.com/getsentry/sentry-javascript/pull/10702 https://github.com/getsentry/sentry-javascript/pull/10704 https://github.com/getsentry/sentry-javascript/pull/10706 They feature some smaller fixes...