Skip to content

feat(packages/sui-react-web-vitals): add sub inp metrics to cwv#1669

Merged
andresz1 merged 6 commits intomasterfrom
feat/update-cwv-reporter
Nov 27, 2023
Merged

feat(packages/sui-react-web-vitals): add sub inp metrics to cwv#1669
andresz1 merged 6 commits intomasterfrom
feat/update-cwv-reporter

Conversation

@andresz1
Copy link
Member

@andresz1 andresz1 commented Nov 23, 2023

Description

This PR:

  • Add INP metric sub-division tracking (input delay, processing time and presentation delay) to have better visibility of INP issues
  • Add tests
  • Remove not needed code
  • Now if there is a route id the pathname will not be tracked using the distribution method

Breaking changes:

  • Changed pathnames prop with allowed (an optional array of pathnames or route ids that you want to track). This will allow us to filter by route ids.

This change will require a new major

Screenshot 2023-11-23 at 17 16 01

Related Issue

Example

Screenshot 2023-11-23 at 17 12 46

@andresz1 andresz1 marked this pull request as ready for review November 23, 2023 16:13
Copy link
Contributor

@jordevo jordevo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔝 glad to see some new tests! 👏

...(routeid
? [
{
key: 'routeid',
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏

Copy link
Collaborator

@nucliweb nucliweb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@nucliweb
Copy link
Collaborator

👀 Remember to update the package.json to upgrade to a major version

BREAKING CHANGES:

pathname for allowed
@andresz1 andresz1 merged commit 350350f into master Nov 27, 2023
@andresz1 andresz1 deleted the feat/update-cwv-reporter branch November 27, 2023 10:16
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.

4 participants