Skip to content

Conversation

@edusperoni
Copy link
Contributor

@edusperoni edusperoni commented Nov 4, 2025

PR Checklist

What is the current behavior?

When registering multiple listeners, it's impossible to unregister any other than the last one created. This creates leaks and weird behavor

What is the new behavior?

When creating a listener it'll now return a function you can call to unregister it. Calling unregister with only the intent name unregisters all listeners.

Additionally, this reverts a few things back to the application files

@nx-cloud
Copy link

nx-cloud bot commented Nov 4, 2025

View your CI Pipeline Execution ↗ for commit 4a5e169

Command Status Duration Result
nx test apps-automated -c=android ✅ Succeeded 4m 17s View ↗
nx run-many --target=test --configuration=ci --... ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2025-11-04 17:02:17 UTC

@NathanWalker NathanWalker merged commit fe4c9c0 into main Nov 4, 2025
7 checks passed
@NathanWalker NathanWalker deleted the feat/multiple-android-listeners branch November 4, 2025 17:29
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.

3 participants