@jamescrosswell I'm sorry for this late review but I just realized there's a huge bug here.
What about breadcrumb.ExtraData? I think you completely missed it.
Also I don't find the ExtraData type very convenient.
I think that this could be more convenient:
public record BreadcrumbEvent(
object? Sender,
string EventName,
params (string Key, string Value)[] ExtraData
);
Originally posted by @albyrock87 in #3997 (comment)
@jamescrosswell I'm sorry for this late review but I just realized there's a huge bug here.
What about
breadcrumb.ExtraData? I think you completely missed it.Also I don't find the
ExtraDatatype very convenient.I think that this could be more convenient:
Originally posted by @albyrock87 in #3997 (comment)