Skip to content

Blog Post: Advancements in Lightning Infrastructure#432

Merged
dergigi merged 28 commits into
masterfrom
impact-report-advancements-in-lightning
Apr 17, 2025
Merged

Blog Post: Advancements in Lightning Infrastructure#432
dergigi merged 28 commits into
masterfrom
impact-report-advancements-in-lightning

Conversation

@Arvin21M
Copy link
Copy Markdown
Contributor

@Arvin21M Arvin21M commented Apr 1, 2025

Build Preview:

Header Image:

  • image

@Arvin21M Arvin21M requested a review from niftynei April 1, 2025 17:11
@Arvin21M Arvin21M self-assigned this Apr 1, 2025
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 1, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
os-website ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 17, 2025 11:46am

@Arvin21M
Copy link
Copy Markdown
Contributor Author

Arvin21M commented Apr 1, 2025

@niftynei Would really appreciate a direct quote once you get a chance to review 🙏

@Arvin21M
Copy link
Copy Markdown
Contributor Author

Arvin21M commented Apr 1, 2025

@OpenSats/volunteers if you have a moment to spare, would you please do a quick sanity check on the writeup? 🙏

Comment thread data/blog/advancements-in-lightning-infrastructure.mdx Outdated
Comment thread data/blog/advancements-in-lightning-infrastructure.mdx Outdated
Comment thread data/blog/advancements-in-lightning-infrastructure.mdx Outdated
Comment thread data/blog/advancements-in-lightning-infrastructure.mdx Outdated
Comment thread data/blog/advancements-in-lightning-infrastructure.mdx Outdated
@DanielFeichtinger
Copy link
Copy Markdown

@Arvin21M Many thanks, looking great!

A couple of tweaks from our end:

  • Could we remove reference to the microcontroller work? It's on the back burner while we close out some other stuff and we don't want to give the impression that it's ready to deploy.
  • Could we replace the looking forward section with some of the detail from our doc? (It supersedes the milestones on the website.) Something like: "Looking ahead, VLS aims for full mainnet readiness within a year, with plans to add support for new Lightning features like splicing and dual funding. The team is focusing on enabling enterprises to confidently manage millions in channel funds."

@Arvin21M
Copy link
Copy Markdown
Contributor Author

Arvin21M commented Apr 2, 2025

@DanielFeichtinger
Thanks for those suggestions.
Please take another look to see if I made those edits as your comment intended.
LMK if we still need to make another tweak or two to make things more precise 🙏

Copy link
Copy Markdown
Contributor

@Bayernatoor Bayernatoor left a comment

Choose a reason for hiding this comment

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

Re-reviewed changes, sanity check done. Looks good to me! 👏

Copy link
Copy Markdown
Contributor

@vincenzopalazzo vincenzopalazzo left a comment

Choose a reason for hiding this comment

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

Sorry to be late to the party, I left some comments.

Overall LGTM :)

Comment thread data/blog/advancements-in-lightning-infrastructure.mdx Outdated
Comment thread data/blog/advancements-in-lightning-infrastructure.mdx
Comment thread data/blog/advancements-in-lightning-infrastructure.mdx Outdated
Arvin21M and others added 2 commits April 3, 2025 12:03
Co-authored-by: Vincenzo Palazzo <vincenzopalazzodev@gmail.com>
Co-authored-by: Vincenzo Palazzo <vincenzopalazzodev@gmail.com>
Copy link
Copy Markdown
Contributor

@optout21 optout21 left a comment

Choose a reason for hiding this comment

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

From Splicing/Optout part, I would suggest 3 minor edits -- see them commented inline.

  • The paragraph "This work has required ... interrupting active transactions" should be dropped altogether, as it contains minute details that don't add much info in this context.
  • In the following paragraph, some small changes:
  • Remove ldk-node from the list
  • Rephrase next sentence to "Although several foundational pull requests have been
    merged, further work is needed to finalize integration into LDK and
    ensure interoperability before Splicing is fully production-ready within LDK."

Otherwise good job, thank you gentlemen!

Comment thread data/blog/advancements-in-lightning-infrastructure.mdx Outdated
Comment thread data/blog/advancements-in-lightning-infrastructure.mdx Outdated
Comment thread data/blog/advancements-in-lightning-infrastructure.mdx Outdated
Co-authored-by: optout <13562139+optout21@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

@Arvin21M Arvin21M left a comment

Choose a reason for hiding this comment

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

  • The review process for this draft by the grantees with their respective projects included in this draft (via change requests, or go-ahead with no change requests) appears to be complete at this point.
  • One final review by OS board member and adding quote in opening section are the final steps.
  • Thanks to everyone who spent time reviewing.

🙏

Comment thread data/blog/advancements-in-lightning-infrastructure.mdx Outdated
Comment thread data/blog/advancements-in-lightning-infrastructure.mdx Outdated
Copy link
Copy Markdown
Contributor

@jonatack jonatack left a comment

Choose a reason for hiding this comment

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

Read this quickly. One nit that came to mind: variants of "enabl(e|ed|ing)" are used 11 times and "leverag(e|ing)" 3 times. To my mind, these are MBA consultant-style words that are easy to overuse. Feel free to ignore!

Edit: otherwise looks great!

@jonatack
Copy link
Copy Markdown
Contributor

jonatack commented Apr 7, 2025

Update commit a596508 looks good 👍

@Arvin21M Arvin21M removed the request for review from niftynei April 15, 2025 15:10
Copy link
Copy Markdown
Member

@dergigi dergigi left a comment

Choose a reason for hiding this comment

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

Looks good to me!

Thanks everyone for helping to get this out the door. 🙏 --- :shipit:

@dergigi dergigi merged commit fdfa726 into master Apr 17, 2025
3 checks passed
@dergigi dergigi deleted the impact-report-advancements-in-lightning branch April 17, 2025 11:48
@dergigi
Copy link
Copy Markdown
Member

dergigi commented Apr 17, 2025

Live now:

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.

8 participants