[telemetry] Update dependencies, and optimize SSR handling #17008
Conversation
hasdfa
added
dependencies
|
Multiple type labels found: dependencies, enhancement Only one is allowed. Please remove the extra type labels to ensure the PR is categorized correctly. |
3 similar comments
|
Multiple type labels found: dependencies, enhancement Only one is allowed. Please remove the extra type labels to ensure the PR is categorized correctly. |
|
Multiple type labels found: dependencies, enhancement Only one is allowed. Please remove the extra type labels to ensure the PR is categorized correctly. |
|
Multiple type labels found: dependencies, enhancement Only one is allowed. Please remove the extra type labels to ensure the PR is categorized correctly. |
|
Deploy preview: https://deploy-preview-17008--material-ui-x.netlify.app/ |
github-actions
bot
added
the
PR: out-of-date
|
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
Signed-off-by: Vadym Raksha <[email protected]>
github-actions
bot
removed
the
PR: out-of-date
|
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
github-actions
bot
added
the
PR: out-of-date
There was a problem hiding this comment.
Nice improvements. 👍
Sidenote: Should we create a label for this package and add a separate entry in the changelog generation? 🤔
cc @michelengelen
IMHO, I think we could start with just a label, but keeping entries in the Core section.
P.S. A more detailed PR title wouldn't hurt. 😉
hasdfa
changed the title
Signed-off-by: Vadym Raksha <[email protected]>
github-actions
bot
removed
the
PR: out-of-date
hasdfa
changed the title
Enhancements
@fingerprintjs/fingerprintjsto use v3 because of licensingesmversion for post-install scriptUpdated dependencies
conf@^5.0.0->conf@^11.0.0(we have to limit the updates to before v12, where they increased min node version to 18)is-docker@^2.2.1->is-docker@^3.0.0(latest)I have followed (at least) the PR section of the contributing guide.