Skip to content

Conversation

mnoman09
Copy link
Contributor

@mnoman09 mnoman09 commented Oct 18, 2022

Summary

  • Added functions of clearAllNotifications, clearAllNotifications with parameter of listenerType and removeNotificationListener to remove callbacks and listeners using id.

Test plan

All Tests should pass

@mnoman09 mnoman09 marked this pull request as ready for review October 21, 2022 12:14
@mnoman09 mnoman09 changed the title feat(removeAllNotificationListeners): Adds support for removing all notification listeners feat(clearAllNotificationListeners): Adds support for removing all notification listeners Oct 24, 2022
@yasirfolio3 yasirfolio3 self-requested a review October 25, 2022 09:22
Copy link
Contributor

@yasirfolio3 yasirfolio3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@msohailhussain msohailhussain merged commit d7886b7 into master Oct 26, 2022
@msohailhussain msohailhussain deleted the mnoman/removeAllForcedDecision branch October 26, 2022 05:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants