-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
fix(perf): Fix HTTP widget documentation link #66011
Conversation
The function is _slightly_ different than the name implies.
- change the copy a bit - redirect to Set Up Performance page - rename variable
Thoughts on this adjustment so that it's more actionable but still links to the performance setup? "No transactions with HTTP spans found. You may need to add integrations to your performance monitoring setup to instrument spans." |
@vivianyentran I like that! What do you think about "to capture these spans" instead of "to instrument spans"? I think we usually say that spans are "captured", and instrumentation is "added" to a codebase or application |
Good suggestion! "Capture spans" definitely makes more sense in this case! |
@vivianyentran done 👍🏻 |
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.
copy LGTM! 👍
Fixes #65071 Directs users with no HTTP span to the "Set Up Performance" page rather than the "Integrations" page which doesn't even exist for a lot of platforms.
Fixes #65071
Directs users with no HTTP span to the "Set Up Performance" page rather than the "Integrations" page which doesn't even exist for a lot of platforms.
Maybe the widget copy is a little awkward?
e.g.,