-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
ref(tracing): Record transaction name changes #5723
Conversation
/** Setter for `name` property, which also sets `source` as custom */ | ||
public set name(newName: string) { | ||
this._name = newName; | ||
this.metadata.source = 'custom'; | ||
this.setName(newName); | ||
} |
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.
Q: Is there a reason we're maintaining this setter and setName? Always seemed a bit redundant to me.
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.
to be backwards compatible - we should maybe fix this for v8, will add a todo
size-limit report 📦
|
0ef9393
to
372b798
Compare
Part of #5679
Dependent on #5714 merging
This PR makes transaction record name changes internally on the transaction class. This allows for this metadata to be added to outgoing transaction events.