Skip to content

feat: Add new syntax candy fn to send a stream of updates to the server...#64

Open
rklaehn wants to merge 2 commits into
mainfrom
notify-streaming
Open

feat: Add new syntax candy fn to send a stream of updates to the server...#64
rklaehn wants to merge 2 commits into
mainfrom
notify-streaming

Conversation

@rklaehn

@rklaehn rklaehn commented Sep 2, 2025

Copy link
Copy Markdown
Collaborator

…without any return message.

Basically a streaming version of notify.

I am using this in my blobs PR that turns provider events into an irpc protocol. See n0-computer/iroh-blobs#142 . In the PR I use an ..Ext trait, but this might as well go into irpc itself.

@rklaehn rklaehn requested a review from Frando September 2, 2025 07:48
@Frando

Frando commented Sep 2, 2025

Copy link
Copy Markdown
Member

Looks good! I think with this we have all interactions pattern covered :)

Can you add some docs? Then I think this is good to go.

@rklaehn

rklaehn commented Sep 2, 2025

Copy link
Copy Markdown
Collaborator Author

Looks good! I think with this we have all interactions pattern covered :)

Can you add some docs? Then I think this is good to go.

I think there are a very large number of possible patterns, but this seems quite useful. Will add some docs.

@dignifiedquire

Copy link
Copy Markdown
Contributor

ping @rklaehn still missing some docs, so we can merge this

rklaehn and others added 2 commits October 21, 2025 15:41
…hout any return message.

Basically a streaming version of notify.
@Frando

Frando commented Oct 21, 2025

Copy link
Copy Markdown
Member

Added a doc comment, will approve and merge.

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