Mike Chu comments

Results 87 comments of


                                            Mike Chu

[ENHANCEMENT] Reduce the bundle size impact of upgrading from 2.9.2 to v3

Agree. 💯 Now that our functional requirements are satisified for this new version, we'll be diving into reducing the underlying [JavaScript SDK size](https://bundlephobia.com/package/@optimizely/[email protected]) and then find opportunities to reduce the...

Regression leads to missing `autoUpdate` hook updates.

@epatey Thanks for the detailed filing. I've created a ticket (FSSDK-9625) to review and implement a solution.

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

I created an internal bug ticket (FSSDK-9622) to review.

added useCallback to getCurrentDecision in useDecision hook

This seems to relate to #196 and (now) internal ticket FSSDK-9624

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

I've created internal work item (FSSDK-9629) to cover this and set it for async story pointing.

Add ability to conditionally render experimentation hooks

This is a solid addition. I've added a Jira ticket for this (FSSDK-9627; need to add tests, etc) I see you've already signed the Contributor License Agreement. Thanks.

Feature Request: `useDecisions` hook

I've created an internal work ticket (FSSDK-9620) for this. Question (and please forgive my in-progress React knowledge): could we separate out by component where useDecision() is used ie react inside...

[BUG] Akamai EW doesnt have TextEncoder and crypto as Globals

Thanks. We're looking into a similar report at https://github.com/optimizely/react-sdk/issues/232

sending SDK to createInstance() in Server-side context causes memory leaks

We released version v5.3.0. Can you review this version in an SSR test environment?

[BUG] queueMicrotask is not supported on rather recent TVs breaking clients

Hey @ToxicBakery. This is a fantastic find and edge case we'd not looked at/considered. Thank you so much for the very clear report. I think we'll need to enhance the...