Skip to content

Feat: Change CTA Link - #13

Merged
JosephMaynard merged 2 commits into
masterfrom
feat/change-cta-link
Feb 22, 2026
Merged

JosephMaynard merged 2 commits into
masterfrom
feat/change-cta-link

Conversation

@JosephMaynard

@JosephMaynard JosephMaynard commented Feb 22, 2026 •

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • Bug Fixes

    • Updated CTA link destinations to route to a new next-steps page instead of the previous path.
  • Chores

    • Enhanced code documentation and refactored internal asset management for improved maintainability.

@coderabbitai

coderabbitai Bot commented Feb 22, 2026 •

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉


📝 Walkthrough

Walkthrough

Changes update CTA URL endpoints from the base path to a /next-steps path across HTML and TypeScript configuration files. Additionally, JS_CONTENT is now exported as a public constant from the assets module. SVG formatting in HTML is updated without semantic changes.

Changes

Cohort / File(s) Summary
CTA URL Updates
report-ui/index.html, src/cta.ts
Updated two CTA link hrefs from dependency-radar.com/?source=standalone-report to dependency-radar.com/next-steps?source=standalone-report. Updated CTA_BASE_URL constant accordingly and added JSDoc documentation for buildCtaUrl function.
Asset Export
src/report-assets.ts
Exported JS_CONTENT as a public constant, enabling external module access to the embedded JavaScript payload.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

🐰 The paths are cleared, the assets freed,
Next-steps await where users lead,
A rabbit hops through code so clean,
With exports bright and URLs keen! 🚀

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change in the changeset: updating CTA links from dependency-radar.com to dependency-radar.com/next-steps across multiple files.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ 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/change-cta-link

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

@coderabbitai

coderabbitai Bot commented Feb 22, 2026 •

Copy link
Copy Markdown
Contributor

Note

Docstrings generation - SUCCESS
Generated docstrings and committed to branch feat/change-cta-link (commit: 0876b7d1cb9850db6cec1fbc5fd495b6f9e99cff)

Docstrings generation was requested by @JosephMaynard.

The following files were modified:

* `src/cta.ts`

These file types are not supported:
* `report-ui/index.html`
@JosephMaynard
JosephMaynard merged commit f2112de into master Feb 22, 2026
1 check passed
@JosephMaynard
JosephMaynard deleted the feat/change-cta-link branch February 22, 2026 21:58
@coderabbitai coderabbitai Bot mentioned this pull request Jul 3, 2026
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