Skip to content

[codex] fix plugin tutorial cta destinations#596

Merged
riderx merged 1 commit into
codex/plugin-tutorial-actions-statsfrom
codex/plugin-tutorial-doc-links
Apr 29, 2026
Merged

[codex] fix plugin tutorial cta destinations#596
riderx merged 1 commit into
codex/plugin-tutorial-actions-statsfrom
codex/plugin-tutorial-doc-links

Conversation

@riderx
Copy link
Copy Markdown
Member

@riderx riderx commented Apr 29, 2026

Summary

  • keep the docs CTA on Capgo docs instead of falling back to GitHub or npm
  • render the three shared CTAs as View Docs, View GitHub, and View npm
  • keep pointer cursors on the button-style links in the shared plugin tutorial page

Notes

Summary by CodeRabbit

  • Bug Fixes

    • Quick links (repository and NPM) now consistently display with proper destinations in all scenarios.
    • Improved fallback behavior for documentation links when not explicitly configured.
  • Style

    • Enhanced cursor styling to better indicate clickable elements throughout the page.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 29, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 76998f91-3619-41bd-abe5-61ef9a4ff346

📥 Commits

Reviewing files that changed from the base of the PR and between f52b208 and 147eb68.

📒 Files selected for processing (1)
  • apps/web/src/pages/plugins/[slug].astro

📝 Walkthrough

Walkthrough

The plugins page component updates URL computation logic for NPM and docs links, ensuring both always resolve to valid destinations. The quick-links UI now unconditionally displays repository and NPM links instead of conditionally rendering them. Cursor styling is applied to anchor elements throughout the page.

Changes

Cohort / File(s) Summary
Plugin Page Quick Links
apps/web/src/pages/plugins/[slug].astro
Modified npmHref to always compute a value (either direct NPM link or search URL), changed docsHref fallback behavior to use docs/plugins base URL instead of npmHref, removed conditional rendering of repo/NPM links in quick-links section (now always visible), and added cursor-pointer styling to multiple anchor elements.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~18 minutes

Possibly related PRs

Poem

🐰 Links now hop where once they'd hide,
Always ready, never shy—
Docs and npm, side by side,
No more conditional goodbye! ✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/plugin-tutorial-doc-links

Review rate limit: 0/5 reviews remaining, refill in 59 minutes and 31 seconds.

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

@sonarqubecloud
Copy link
Copy Markdown

@riderx riderx marked this pull request as ready for review April 29, 2026 16:14
@riderx riderx merged commit dcc2f34 into codex/plugin-tutorial-actions-stats Apr 29, 2026
8 of 9 checks passed
@riderx
Copy link
Copy Markdown
Member Author

riderx commented Apr 29, 2026

@coderabbitai review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 29, 2026

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@riderx
Copy link
Copy Markdown
Member Author

riderx commented Apr 29, 2026

@coderabbitai review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 29, 2026

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

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.

1 participant