You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I believe #188 attempted to solve this, however the following project does not see new values for useFlags Flagsmith/flagsmith-js-examples#8. Essentially in the current live version of the SDK events.off('event', eventListener) is removing more than 1 event handler.
I cannot figure out the issue, however maybe we need to implement a version of tinyevents that lets you remove the exact event listener that you setup.
I believe #188 attempted to solve this, however the following project does not see new values for useFlags Flagsmith/flagsmith-js-examples#8. Essentially in the current live version of the SDK events.off('event', eventListener) is removing more than 1 event handler.
To get around it, the PR has a version of flagsmith from https://github.com/Flagsmith/flagsmith-js-client/pull/210/files#diff-090ff4f85a594b9bd7b6f1a515395bd0a604ade2c55db3e5c9e7ea76021cbd7fL156 that fixes the issue, but it'd cause a leak due to the removed line.
I cannot figure out the issue, however maybe we need to implement a version of tinyevents that lets you remove the exact event listener that you setup.
i.e.
The text was updated successfully, but these errors were encountered: