Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add removeAllChangeListeners method #861

Merged
merged 2 commits into from
Mar 13, 2025

Conversation

eXon
Copy link
Collaborator

@eXon eXon commented Mar 13, 2025

Summary

  • Add the ability to remove all change listeners at once with a single method call
  • Simplifies cleanup code in components that use cookie change listeners

Test plan

  • Added a comprehensive test to verify that all change listeners are properly removed
  • Verified that the implementation correctly cleans up all related event handlers

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Adds the ability to clean up all cookie change listeners at once, allowing for easier garbage collection of objects and proper cleanup.

🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
@eXon eXon force-pushed the feature/remove-all-change-listeners branch from e67ee0e to d902bb9 Compare March 13, 2025 13:33
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
@eXon eXon merged commit de06778 into main Mar 13, 2025
3 checks passed
@eXon eXon deleted the feature/remove-all-change-listeners branch March 13, 2025 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant