Skip to content

Conversation

@rklaehn
Copy link
Contributor

@rklaehn rklaehn commented Jan 22, 2026

Implements #424

@vercel
Copy link

vercel bot commented Jan 22, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
iroh-computer Ready Ready Preview, Comment Jan 22, 2026 8:08pm

Request Review

Add first graphics
@n0bot n0bot bot added this to iroh Jan 22, 2026
@github-project-automation github-project-automation bot moved this to 🚑 Needs Triage in iroh Jan 22, 2026
Copy link
Member

@ramfox ramfox left a comment

Choose a reason for hiding this comment

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

Looks great. Some nits for grammar!


Implementing a custom transport is not quite as simple. But it also isn't extremely complex. Dealing with the quirks of the actual transport will certainly be more work than wiring it up into iroh.

The custom transport API consists of a number of dynable traits.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
The custom transport API consists of a number of dynable traits.
The custom transport API consists of a number of dyn-able traits.

I think this is how I've seen it written before, but feel-free to ignore.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

works for me.

Copy link
Member

@ramfox ramfox left a comment

Choose a reason for hiding this comment

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

Looks great. Some nits for grammar/punctuation!

@ramfox ramfox moved this from 🚑 Needs Triage to 🏗 In progress in iroh Jan 22, 2026
okdistribute and others added 10 commits January 22, 2026 10:57
Co-authored-by: ramfox <kasey@n0.computer>
Co-authored-by: ramfox <kasey@n0.computer>
Co-authored-by: ramfox <kasey@n0.computer>
Co-authored-by: ramfox <kasey@n0.computer>
Co-authored-by: ramfox <kasey@n0.computer>
Co-authored-by: ramfox <kasey@n0.computer>
Co-authored-by: ramfox <kasey@n0.computer>
Co-authored-by: ramfox <kasey@n0.computer>
Co-authored-by: ramfox <kasey@n0.computer>
Co-authored-by: ramfox <kasey@n0.computer>
okdistribute and others added 18 commits January 22, 2026 10:59
Co-authored-by: ramfox <kasey@n0.computer>
Co-authored-by: ramfox <kasey@n0.computer>
Co-authored-by: ramfox <kasey@n0.computer>
Co-authored-by: ramfox <kasey@n0.computer>
Co-authored-by: ramfox <kasey@n0.computer>
Co-authored-by: ramfox <kasey@n0.computer>
Co-authored-by: ramfox <kasey@n0.computer>
Co-authored-by: ramfox <kasey@n0.computer>
Co-authored-by: ramfox <kasey@n0.computer>
Co-authored-by: ramfox <kasey@n0.computer>
Co-authored-by: ramfox <kasey@n0.computer>
Co-authored-by: ramfox <kasey@n0.computer>
Co-authored-by: ramfox <kasey@n0.computer>
Co-authored-by: ramfox <kasey@n0.computer>
Co-authored-by: ramfox <kasey@n0.computer>
Co-authored-by: ramfox <kasey@n0.computer>
Co-authored-by: ramfox <kasey@n0.computer>
Co-authored-by: ramfox <kasey@n0.computer>
@okdistribute
Copy link
Contributor

@ramfox @rklaehn I incorporated your changes and updated the structure so it flows more, I didn't change much content just reorganized it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🏗 In progress

Development

Successfully merging this pull request may close these issues.

4 participants