FredrikOseberg
commented
2 years ago Looks good to me! I've left a text change comment, aside from that, this is great!
Just a quick double check because my Golang isn't strong - this should still parse responses from the API when the segments property isn't set?
Edit: Just saw this work with empty responses, great!
I added a test for testing the old response with the new interface:
This PR will implement global segments as a part of performance optimizations related to segment use. This will ensure that segments are sent as part of the payload from the client API, parsed correctly into a map on the SDK side and combined with the regular strategy constraints.
Fixes #115