- Issue created by @longwave
- First commit to issue fork.
- Merge request !23Issue #3432440 by mfb, longwave: Allow Interaction to Next Paint to be enabled in JavaScript β (Merged) created by mfb
- last update
about 1 year ago 30 pass - Status changed to Needs review
about 1 year ago 5:58pm 20 March 2024 - πΊπΈUnited States mfb San Francisco
If there is any downside to enabling INP, such as breaking older versions of self-hosted Sentry server, then we might need to hide this behind a config flag. But, I didn't check for that yet so, keeping this simple for now π
- last update
about 1 year ago 30 pass - π¬π§United Kingdom longwave UK
Yeah it's not clear from the docs why you would want to disable this at the moment. Thanks for the swift response - will patch this in on our setup and try it out soon.
- πΊπΈUnited States mfb San Francisco
In my testing, enabling this results in a separate ui.interaction.click payload being sent to Sentry; i.e. this is analogous to other JS payload types that we do have settings for, namely, auto session tracking and client reports. So I'd say it's preferable to add a config flag, so if someone doesn't want to be so "chatty" w/ Sentry they can keep it disabled.
- πΊπΈUnited States mfb San Francisco
...but these extra events weren't actually sent - at least in my testing - until I also updated the JS SDK? So I guess we also need to update that (unless I just had bad luck trying to test this feature)
- last update
about 1 year ago 30 pass - last update
about 1 year ago 30 pass - last update
about 1 year ago 30 pass - last update
about 1 year ago 30 pass - last update
about 1 year ago 30 pass - Status changed to Fixed
about 1 year ago 6:50pm 22 March 2024 - πΊπΈUnited States mfb San Francisco
This worked in my quick testing, but please re-open if there are any problems
Automatically closed - issue fixed for 2 weeks with no activity.