FlagMetadata and EventMetadata are shaped the same in the SDK:
I think for consistency, that means we should use the same type for these, which I guess would mean it should be a Dictionary<string, object>() unless there's some way we can bring them in line with your new implementation in a non-breaking way... maybe with extension methods?
Originally posted by @toddbaert in #223 (comment)
FlagMetadataandEventMetadataare shaped the same in the SDK:I think for consistency, that means we should use the same type for these, which I guess would mean it should be a
Dictionary<string, object>()unless there's some way we can bring them in line with your new implementation in a non-breaking way... maybe with extension methods?Originally posted by @toddbaert in #223 (comment)