Skip to content

Conversation

ConorOkus
Copy link
Contributor

@ConorOkus ConorOkus commented Sep 27, 2022

@netlify
Copy link

netlify bot commented Sep 27, 2022

Deploy Preview for lightningdevkit ready!

Name Link
🔨 Latest commit d5aba95
🔍 Latest deploy log https://app.netlify.com/sites/lightningdevkit/deploys/633c4eede4b9e90008368eb0
😎 Deploy Preview https://deploy-preview-167--lightningdevkit.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

- A [`ChannelManager`](https://docs.rs/lightning/*/lightning/ln/channelmanager/index.html) that enables it to notify peers when it wants to do things such as open channels.
- A [`P2PGossipSync`](https://docs.rs/lightning/*/lightning/routing/gossip/struct.P2PGossipSync.html) that understands when peers you are connected to send you gossip messages, enables you to forward messages, announce channel opens and more.

Since LDK knows nothing about your networking stack, you'll need to implement this module in order to connect to peers on the network. Provide LDK with raw TCP/IP socket data and the library will handle the rest for you.
Copy link
Contributor

@tnull tnull Sep 28, 2022

Choose a reason for hiding this comment

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

nit: A bit unclear what this module refers to in this sentence. I believe the networking stack?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry for late response, updated

Conor Okus and others added 3 commits October 4, 2022 16:17
Co-authored-by: Elias Rohrer <dev@tnull.de>
Co-authored-by: Elias Rohrer <dev@tnull.de>
@ConorOkus ConorOkus merged commit aafd380 into lightningdevkit:main Oct 5, 2022
@ConorOkus ConorOkus deleted the peer-management-overview branch October 5, 2022 15:18
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.

2 participants