Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

MOB-917 Fix Pusher emitting multiple events on Closed Thread #107

Merged
merged 1 commit into from
Feb 13, 2024
Merged

MOB-917 Fix Pusher emitting multiple events on Closed Thread #107

merged 1 commit into from
Feb 13, 2024

Conversation

Peter-John-paystack
Copy link
Collaborator 

- After Pusher emits its success or failure response, subscribe from the event and close the pusher connection

@Peter-John-paystack
MOB-917 Fix Pusher emitting multiple events on Closed Thread
959601f 
	- After Pusher emits its success or failure response, subscribe from the event and close the pusher connection
@Peter-John-paystack Peter-John-paystack added the bug Something isn't working label Feb 12, 2024
@Peter-John-paystack Peter-John-paystack self-assigned this Feb 12, 2024
@github-actions GitHub Actions
Copy link
Contributor

Warnings
⚠️ The source files were changed, but the tests remain unmodified. Consider updating or adding to the tests to match the source changes.

Generated by 🚫 Danger Swift against 959601f

@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
33.3% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@Peter-John-paystack Peter-John-paystack marked this pull request as ready for review February 12, 2024 12:49
@Peter-John-paystack Peter-John-paystack merged commit 5b6ee3d into main Feb 13, 2024
6 checks passed
@Peter-John-paystack Peter-John-paystack deleted the fix/MOB-917-Fix-Pusher-multi-events-emitting branch February 13, 2024 07:48
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@michael-paystack michael-paystack michael-paystack approved these changes

@dami-paystack dami-paystack dami-paystack approved these changes

@nathanstasin nathanstasin Awaiting requested review from nathanstasin

Assignees

@Peter-John-paystack Peter-John-paystack

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Peter-John-paystack @michael-paystack @dami-paystack