react-sdk

react-sdk copied to clipboard

A `useEffect`'s within `useDecision` will flap the auto update listeners on every render.

1

[This](https://github.com/optimizely/react-sdk/blob/da4627c1e45f2a1f92861468f06828a8990e288d/src/hooks.ts#L467) `useEffect` will mount and execute on every single render. The root cause is that `getCurrentDecision` in `useDecision` is not wrapped in `useCallback` like it is in the other hooks....

epatey

Allow `default` to be set on an OptimizelyVariation that has `variation` also set

5

This would be a nice quality of life improvement to have as we often have experiment code that looks like this: ```tsx ``` It would be nice to be able...

grug

fix user overrides: merge overrides with current user attributes

2

This pull request addresses the issue that is laid out by my colleague @dtothefp here: optimizely#120 It will merge the existing user attributes with the current user's attributes to prevent...

Sillhouette

fix(hooks): update useFeature decisions when forced variation updates

2

Update feature decisions when forced variations update

jscarmona

Add ability to conditionally render experimentation hooks

1

## Description We'd like to be able to render `useExperiment()` and the other hooks conditionally. This PR will add the ability for people to run `useExperiment('experiment_key', { disabled: true })`...

PepijnSenders

added useCallback to getCurrentDecision in useDecision hook

1

adding useCallback here fixed a few things: 1. Bug: if you used multiple useDecision hooks with { autoUpdate: true }, the second one would not autoUpdate (you had to fully...

monahans

Regression leads to missing `autoUpdate` hook updates.

1

There is a timing bug in all three hooks' (`useFeature`, `useExperiment`, and `useDecision`) that leads to `autoUpdate: true` render triggers getting lost. The code in #125 introduced a window of...

epatey

Can't perform a React state update on an unmounted component

1

Hi. I'm getting this `Can't perform a React state update on an unmounted component` Warning when I'm using optimizely. In a simple example, I created a hook like this: ```...

ManuViola77

Feature Request: `useDecisions` hook

2

The current `useDecision` hook can only work with one feature flag and return only one OptimizelyDecision. In situation where more than one decision is needed, a useDecisions hook can be...

ivanngbowtie

Merge Attributes on Override Attributes and/or Pass User Attributes as Context

2

We have a base set of attributes that are passed to our OptimizelyProvider upon initialization. A common use case is in a hook to pass the `overrideAttributes`. Generally we would...

dtothefp