knocklabs / javascript

Official JavaScript packages for interacting with Knock
https://knock.app/
MIT License
10 stars 2 forks source link

fix: remove visibility change event listener on feed teardown #111

Closed connorlindsey closed 2 months ago

connorlindsey commented 2 months ago

Description

This PR makes sure we remove the visibilitychanged event listener when we tear down the feed instance.

linear[bot] commented 2 months ago

KNO-5710 Too many event listeners being created by React providers

changeset-bot[bot] commented 2 months ago

🦋 Changeset detected

Latest commit: b08aa62bda13af4c6bf27cbe2c748239517941e7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 7 packages | Name | Type | | ------------------------------- | ----- | | @knocklabs/client | Patch | | client-example | Patch | | @knocklabs/react-core | Patch | | @knocklabs/react-native | Patch | | @knocklabs/react | Patch | | @knocklabs/react-native-example | Patch | | nextjs-example | Patch |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

vercel[bot] commented 2 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
javascript-slack-connect-example ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 9, 2024 5:39pm
javascript-slack-kit-example ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 9, 2024 5:39pm