Repository navigation
feat: Implement SDK spec v0.7.0 - #154
Closed
flagsmithsmith[bot] wants to merge 3 commits into
Closed
flagsmithsmith[bot] wants to merge 3 commits into
flagsmithsmith[bot] wants to merge 3 commits into
Conversation
…osures Implements the events steps of SDK spec v0.6.0. - `Flagsmith::getExperimentFlag()` gets an identity's flag and records a `$flag_exposure` event for the variant served, when the identity is enrolled in a running experiment on the feature. - `Flagsmith::trackEvent()` and `Flagsmith::trackExposureEvent()` record custom and exposure events; `Flagsmith::flushEvents()` sends them now. - `EventProcessor` buffers events, deduplicates exposures, and posts them to the Flagsmith events API on flush, when full, after the flush interval, or when destroyed. Events are opt-in via `enableEvents`. - `Flag` exposes the `variant` and `experiment` the identities API returns. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> (cherry picked from commit 0dad745)
- Reject `withEventProcessor()` on offline instances, as the constructor already does for `enableEvents`. - Remove `EventProcessor`'s unused `$timeout` parameter: PSR-18 has no per-request timeout, so document passing a client configured with one. - Read the SDK version from composer.json once per process. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> (cherry picked from commit c3b3474)
1 task
|
Important Review skippedBot user detected. To trigger a single review, invoke the
 |
Contributor
Author
|
Superseded by #156, which carries this pull request's implementation work, for the SDK spec's latest version. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Implements the SDK spec's v0.7.0 in this SDK.
This pull request was opened by an AI agent (OpenHands).