Skip to content
This repository has been archived by the owner on Jul 31, 2024. It is now read-only.

fixed peerDependency newrelic #89

Merged
merged 1 commit into from
Oct 11, 2022
Merged

fixed peerDependency newrelic #89

merged 1 commit into from
Oct 11, 2022

Conversation

siuvdlec
Copy link
Contributor

@siuvdlec siuvdlec commented Oct 11, 2022

Proposed Release Notes

  • Updated newrelic peer dependency to be >= 8.14.0. This makes the hasToRemoveScriptWrapper property available for api.getBrowserTimingHeader.

Details

In order to be able to use api.getBrowserTimingHeader with hasToRemoveScriptWrapper: true the minimum version of newrelic is 8.14.0 https://github.com/newrelic/node-newrelic/blob/main/NEWS.md#v8140-2022-06-06

@CLAassistant
Copy link

CLAassistant commented Oct 11, 2022

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@bizob2828 bizob2828 left a comment

Choose a reason for hiding this comment

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

Thanks for updating @siuvdlec. You can just install the agent version you need without this fix landing. But we will get it shipped eventually.

@bizob2828 bizob2828 merged commit c06f391 into newrelic:main Oct 11, 2022
@github-actions github-actions bot mentioned this pull request Oct 17, 2022
bizob2828 added a commit to bizob2828/newrelic-node-nextjs that referenced this pull request Jul 26, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants