Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

roekatz
Copy link
Contributor

@roekatz roekatz commented Aug 15, 2024

connect method is important to be able to retry connections (Specifically from opal server) without using the cumbersome __aenter__

@roekatz roekatz force-pushed the roe/per-10361-fix-opal-server-eventbroadcaster-integration branch 2 times, most recently from 804da6a to d75208d Compare September 4, 2024 15:20
@roekatz roekatz marked this pull request as ready for review September 4, 2024 15:21
@roekatz roekatz changed the title Refactor EventBroadcaster Simplify EventBroadcaster's interface Sep 4, 2024
@roekatz roekatz force-pushed the roe/per-10361-fix-opal-server-eventbroadcaster-integration branch 2 times, most recently from 7f7b9d8 to 7bcbcaa Compare September 5, 2024 10:11
@roekatz roekatz changed the title Simplify EventBroadcaster's interface EventBroadcaster: Introduce connect & close + Clean code a bit Sep 5, 2024
@roekatz roekatz requested a review from asafc September 5, 2024 10:13
@roekatz roekatz force-pushed the roe/per-10361-fix-opal-server-eventbroadcaster-integration branch from 7bcbcaa to ee3c2ee Compare September 9, 2024 12:16
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.

1 participant

Morty Proxy This is a proxified and sanitized view of the page, visit original site.