Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

Conditionally include New Relic Instrumentation #176

Closed
tangollama opened this issue Sep 25, 2015 · 4 comments
Closed

Conditionally include New Relic Instrumentation #176

tangollama opened this issue Sep 25, 2015 · 4 comments
Labels
help wanted indicates that an issue is open for contributions
Milestone
HospitalRun 1.0

Comments

@tangollama
Copy link
Member

Setup a config to include (or not) the ability to instrument the frontend using New Relic Insights. https://blog.newrelic.com/2015/04/03/new-relic-insights-browser-instrumentation-js-api/
@tangollama tangollama self-assigned this Sep 25, 2015
@tangollama
Copy link
Member Author

Actually, the server agent automatically includes the JS api. Now I just need to conditionally (i.e. check if the API is available) call events.

@tangollama tangollama modified the milestone: HospitalRun 1.0 Oct 14, 2015
@tangollama tangollama removed their assignment Oct 16, 2015
@tangollama tangollama added the help wanted indicates that an issue is open for contributions label Oct 16, 2015
@jkleinsc
Copy link
Member

@tangollama I added New Relic support with c8c3601 (which pulls in New Relic if so configured in your config.js file. See https://github.com/HospitalRun/hospitalrun-server-routes/releases/tag/0.9.2 for more detail. I think that means we can close this, but can you verify?

@tangollama
Copy link
Member Author

Will do. Thank you @jkleinsc.

@tangollama tangollama removed the help wanted indicates that an issue is open for contributions label Mar 15, 2016
@tangollama tangollama self-assigned this Mar 15, 2016
@tangollama
Copy link
Member Author

@dswanson not to pigeon-hole you, but would you be cool verifying this?

@tangollama tangollama removed their assignment Mar 15, 2016
@tangollama tangollama self-assigned this Mar 22, 2016
@tangollama tangollama added the in progress indicates that issue/pull request is currently being worked on label Mar 22, 2016
@tangollama tangollama added help wanted indicates that an issue is open for contributions and removed in progress indicates that issue/pull request is currently being worked on labels Jun 2, 2016
@tangollama tangollama removed their assignment Mar 14, 2017
matteovivona pushed a commit that referenced this issue Jan 15, 2021
@JacobMGEvans
Merge pull request #176 from codyarose/codyarose_ImportantPatientInfo
f760ef8 
Convert ImportantPatientInfo.test.tsx to RTL
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
help wanted indicates that an issue is open for contributions
Projects
None yet
Milestone
HospitalRun 1.0
Development

No branches or pull requests
2 participants
@jkleinsc @tangollama