Skip to content

feat: add lnvps to app store#2100

Merged
rolznz merged 1 commit intomasterfrom
feat/lnvps
Feb 27, 2026
Merged

feat: add lnvps to app store#2100
rolznz merged 1 commit intomasterfrom
feat/lnvps

Conversation

@im-adithya
Copy link
Copy Markdown
Member

@im-adithya im-adithya commented Feb 27, 2026

Fixes #2060

Summary by CodeRabbit

  • New Features
    • Added LNVPS app to available connections and integrations. LNVPS offers VPS infrastructure with Bitcoin-native billing and automated renewals through Nostr Wallet Connect.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Feb 27, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3392e75 and ad44601.

⛔ Files ignored due to path filters (1)
  • frontend/src/assets/suggested-apps/lnvps.png is excluded by !**/*.png
📒 Files selected for processing (1)
  • frontend/src/components/connections/SuggestedAppData.tsx

📝 Walkthrough

Walkthrough

Added a new LNVPS app entry to the suggested applications data structure in SuggestedAppData.tsx, including asset import, metadata, guides, and category classification.

Changes

Cohort / File(s) Summary
LNVPS App Addition
frontend/src/components/connections/SuggestedAppData.tsx
Added import for lnvps.png asset and new AppStoreApp entry with id "lnvps" containing metadata (title, description, webLink, logo), categorization (misc), and user guides (installGuide, finalizeGuide). Entry positioned between Lightning Messageboard and Alby Extension with hideConnectionQr set to true.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A VPS so new joins our store,
With Bitcoin's might, we've got one more!
LNVPS hops in with verve,
Lightning-fast, it's what they serve! 🚀

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Linked Issues check ⚠️ Warning The linked issue #2061 requires implementing error handling for database queries, but the PR changes only add a new app store entry without any database error handling implementation. The PR changes do not address the error handling requirements from issue #2061. Verify that this PR is correctly linked to issue #2061 or link it to the appropriate issue instead.
Out of Scope Changes check ⚠️ Warning The PR adds LNVPS app data to the app store, which appears unrelated to the linked issue #2061 about database query error handling, suggesting out-of-scope changes. The LNVPS app store addition does not align with the database error handling objectives in issue #2061. Review the issue linkage and scope alignment.
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The PR title 'feat: add lnvps to app store' accurately describes the main change: adding a new app entry (LNVPS) to the app store configuration.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/lnvps

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@rolznz rolznz left a comment

Choose a reason for hiding this comment

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

tACK

@rolznz rolznz merged commit 4a924c3 into master Feb 27, 2026
12 checks passed
@rolznz rolznz deleted the feat/lnvps branch February 27, 2026 09:22
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.

Add LNVPS to the app store

2 participants