-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: added custom segments app #277
feat: added custom segments app #277
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The intent of replacing these examples from node-newrelic is to wrap them in a webframework so you don't have to run in background transaction. The only example that should be running in background transactions is the example to show how to use that api method. This example is showing how to record custom segments
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
verified the app works as expected. just a few suggestions in the README and an alternate way to require the functions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work
This addition aims to move the examples in node-newrelic/examples/api/segments into a single stand-alone application. The application uses the fastify web framework to showcase how segments can be used in a variety of use cases: callback-based, promise-based, asynchronously, and synchronously.