feat: Add support for per-context summary events. #859
Conversation
kinyoklion
force-pushed
the
rlamb/prototype-multi-summary
branch
from
d6b4b45 to
5eee3b3
Compare
|
@launchdarkly/js-client-sdk-common size report |
|
@launchdarkly/js-client-sdk size report |
There was a problem hiding this comment.
JSON files intentionally do not have newlines at the end for test purposes.
There was a problem hiding this comment.
We have a local canonicalization implementation, but we use standardized test data. Our implementation handles cycles.
|
@launchdarkly/js-sdk-common size report |
| @@ -41,4 +41,4 @@ jobs: | |||
| target_file: 'packages/sdk/browser/dist/index.js' | |||
| package_name: '@launchdarkly/js-client-sdk' | |||
| pr_number: ${{ github.event.number }} | |||
| size_limit: 21000 | |||
| size_limit: 25000 | |||
There was a problem hiding this comment.
Both this PR and the client-side identify PR need a little breathing room. This PR exceeded the limit by 174 bytes.
Adds support for per-context summary events. Per-context summary events allow for enhanced analytics to know which contexts evaluate which flags, even when those flags are not part of an experiment.