feat: Add support for client-side prerequisite events. #593
Conversation
| if (value === undefined || value === null) { | ||
| this.logger.debug('Result value is null. Providing default value.'); | ||
| successDetail.value = defaultValue; | ||
| } | ||
|
|
||
| successDetail.prerequisites?.forEach((prereqKey) => { |
There was a problem hiding this comment.
Send prerequisite events in order before the event for the variation. This matches server behavior.
| }, | ||
| eventFactory, | ||
| ); | ||
| this.evaluateCb(flag, context, resolve, eventFactory); |
There was a problem hiding this comment.
Unrelated change, but originally the cb version was added after the non-cb version. The interior code was the same, so the async eval can be implemented in terms of the callback eval.
There was a problem hiding this comment.
This file updates because of unfortunate shared data. Not related to required updates.
There was a problem hiding this comment.
This file updates because of unfortunate shared data. Not related to required updates.
There was a problem hiding this comment.
This file updates because of unfortunate shared data. Not related to required updates.
|
We have decided to go with version B. |
This PR contains the server-side implementation for allFlagsState evaluation for bootstrap, server-side LDEvaluationDetail, client-side LDEvaluationDetail, and client-side events for prerequisites.
BEGIN_COMMIT_OVERRIDE
feat: Add support for client-side prerequisite events.
feat: Add support for prerequisite details to evaluation detail.
feat: Add prerequisite information to server-side allFlagsState.
END_COMMIT_OVERRIDE