docs: add SEO cross-links to extension guides#8
Open
hermes-alby wants to merge 1 commit into
Open
Conversation
📝 WalkthroughWalkthroughThis PR systematically improves cross-product navigation and discoverability across Alby Browser Extension documentation by adding explicit links to the Alby Browser Extension installation page, Alby Hub, the centralized app directory, and Alby Go, while clarifying product positioning and entry points throughout the guides. ChangesAlby Browser Extension Documentation Cross-Linking
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment Warning |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (1)
lightning-browser-extension/features/manage-connected-websites.md (1)
4-4: ⚡ Quick winInconsistent link formatting in frontmatter description.
The URL
getalby.com/discoveris presented as plain text in the frontmatter description, while the same URL is properly formatted as a markdown link[Alby app directory](https://getalby.com/discover)on line 12.For consistency, consider either:
- Using the full markdown link format in the frontmatter (if supported by your documentation system)
- Or using the plain URL format throughout
Also note: Line 4 is missing the
https://protocol prefix while line 12 includes it.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@lightning-browser-extension/features/manage-connected-websites.md` at line 4, Update the frontmatter description so the discover URL is consistent with the body: replace the plain "getalby.com/discover" with the full "https://getalby.com/discover" and, if your docs system supports Markdown, use the existing markdown link syntax used later (e.g. replace getalby.com/discover with [Alby app directory](https://getalby.com/discover)); ensure the exact text matches the link used on line 12 to keep formatting and protocol consistent.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@lightning-browser-extension/features/create-and-change-budgets.md`:
- Line 7: The sentence in create-and-change-budgets.md narrows budgets to "apps
from the Alby app directory" which is inaccurate; update the copy so it states
budgets apply to any website or app that requests payment through Alby while
still promoting the app directory—for example, replace "When you want to pay in
apps from the [Alby app directory](https://getalby.com/discover), Alby asks to
confirm the payment" with wording like "When a website or app requests a payment
through Alby (including apps listed in the [Alby app
directory](https://getalby.com/discover)), Alby asks you to confirm the payment"
so the scope is correct but the directory is still promoted.
In `@lightning-browser-extension/features/manage-connected-websites.md`:
- Line 12: The sentence "Use connected website permissions to stay in control
while trying apps from the [Alby app directory](https://getalby.com/discover)."
incorrectly narrows the feature; update that line in
manage-connected-websites.md to state that connected website permissions apply
to any website that connects to Alby while still promoting the directory—for
example, replace the sentence with a broader variant like "Use connected website
permissions to stay in control when any website connects to Alby; discover apps
in the [Alby app directory](https://getalby.com/discover)." Make sure to edit
the exact sentence text shown in the diff.
---
Nitpick comments:
In `@lightning-browser-extension/features/manage-connected-websites.md`:
- Line 4: Update the frontmatter description so the discover URL is consistent
with the body: replace the plain "getalby.com/discover" with the full
"https://getalby.com/discover" and, if your docs system supports Markdown, use
the existing markdown link syntax used later (e.g. replace getalby.com/discover
with [Alby app directory](https://getalby.com/discover)); ensure the exact text
matches the link used on line 12 to keep formatting and protocol consistent.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: e25ed753-0f3a-4eb9-819f-0b232a571ae6
📒 Files selected for processing (17)
lightning-browser-extension/README.mdlightning-browser-extension/discover-apps.mdlightning-browser-extension/faq/does-the-extension-work-on-mobile.mdlightning-browser-extension/faq/how-can-i-run-my-own-lightning-node.mdlightning-browser-extension/faq/how-do-i-install-the-alby-browser-extension.mdlightning-browser-extension/faq/how-to-enable-alby-only-on-specific-websites.mdlightning-browser-extension/faq/implementing-payments-and-authentication.mdlightning-browser-extension/features/create-and-change-budgets.mdlightning-browser-extension/features/manage-connected-websites.mdlightning-browser-extension/features/manage-multiple-wallets.mdlightning-browser-extension/features/nostr.mdlightning-browser-extension/features/send-and-receive-bitcoin.mdlightning-browser-extension/link-wallet/README.mdlightning-browser-extension/link-wallet/alby-hub-via-nwc-link.mdlightning-browser-extension/link-wallet/alby-web-account.mdlightning-browser-extension/link-wallet/no-limits-quick-and-easy.mdlightning-browser-extension/link-wallet/voltage.md
| # 💵 Create and change budgets | ||
|
|
||
| When you want to pay, Alby asks to confirm the payment | ||
| When you want to pay in apps from the [Alby app directory](https://getalby.com/discover), Alby asks to confirm the payment |
There was a problem hiding this comment.
Wording may incorrectly limit the scope of the budgets feature.
The new text "When you want to pay in apps from the Alby app directory" suggests that budgets only apply when paying through apps listed in the Alby app directory. However, the budgets feature works with any website that requests payment through Alby, not just apps from the directory.
Consider revising to promote the app directory without limiting the feature's applicability:
📝 Suggested revision
-When you want to pay in apps from the [Alby app directory](https://getalby.com/discover), Alby asks to confirm the payment
+When you want to pay (try apps from the [Alby app directory](https://getalby.com/discover)), Alby asks to confirm the paymentor
-When you want to pay in apps from the [Alby app directory](https://getalby.com/discover), Alby asks to confirm the payment
+When you want to pay in bitcoin apps (find them in the [Alby app directory](https://getalby.com/discover)), Alby asks to confirm the payment📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| When you want to pay in apps from the [Alby app directory](https://getalby.com/discover), Alby asks to confirm the payment | |
| When you want to pay (try apps from the [Alby app directory](https://getalby.com/discover)), Alby asks to confirm the payment |
Suggested change
| When you want to pay in apps from the [Alby app directory](https://getalby.com/discover), Alby asks to confirm the payment | |
| When you want to pay in bitcoin apps (find them in the [Alby app directory](https://getalby.com/discover)), Alby asks to confirm the payment |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@lightning-browser-extension/features/create-and-change-budgets.md` at line 7,
The sentence in create-and-change-budgets.md narrows budgets to "apps from the
Alby app directory" which is inaccurate; update the copy so it states budgets
apply to any website or app that requests payment through Alby while still
promoting the app directory—for example, replace "When you want to pay in apps
from the [Alby app directory](https://getalby.com/discover), Alby asks to
confirm the payment" with wording like "When a website or app requests a payment
through Alby (including apps listed in the [Alby app
directory](https://getalby.com/discover)), Alby asks you to confirm the payment"
so the scope is correct but the directory is still promoted.
|
|
||
| ## Manage permissions on sites | ||
|
|
||
| Use connected website permissions to stay in control while trying apps from the [Alby app directory](https://getalby.com/discover). |
There was a problem hiding this comment.
Wording may incorrectly limit the scope of the permissions feature.
The phrase "while trying apps from the Alby app directory" suggests that website permission management only applies when using apps from the directory. However, this feature manages permissions for any website that connects to Alby, not just apps listed in the directory.
Consider revising to promote the app directory without limiting the feature's applicability:
📝 Suggested revision
-Use connected website permissions to stay in control while trying apps from the [Alby app directory](https://getalby.com/discover).
+Use connected website permissions to stay in control when connecting to bitcoin apps (discover them in the [Alby app directory](https://getalby.com/discover)).📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| Use connected website permissions to stay in control while trying apps from the [Alby app directory](https://getalby.com/discover). | |
| Use connected website permissions to stay in control when connecting to bitcoin apps (discover them in the [Alby app directory](https://getalby.com/discover)). |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@lightning-browser-extension/features/manage-connected-websites.md` at line
12, The sentence "Use connected website permissions to stay in control while
trying apps from the [Alby app directory](https://getalby.com/discover)."
incorrectly narrows the feature; update that line in
manage-connected-websites.md to state that connected website permissions apply
to any website that connects to Alby while still promoting the directory—for
example, replace the sentence with a broader variant like "Use connected website
permissions to stay in control when any website connects to Alby; discover apps
in the [Alby app directory](https://getalby.com/discover)." Make sure to edit
the exact sentence text shown in the diff.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Scope
lightning-browser-extension/dev/Verification
git diff --checklightning-browser-extension/, nodev/files changed, and no new GitHub/GitBook guide cross-links were addedpython3 /home/hermes/.hermes/skills/software-development/docs-link-maintenance/scripts/validate_changed_markdown_links.py --scope lightning-browser-extension --forbid-scope devSummary by CodeRabbit