Skip to content

Adds utm params to powered by link re SC-1901#75

Merged
dmackinn merged 1 commit intodevelopmentfrom
feaure/SC-1901-utm-params
Mar 5, 2026
Merged

Adds utm params to powered by link re SC-1901#75
dmackinn merged 1 commit intodevelopmentfrom
feaure/SC-1901-utm-params

Conversation

@dmackinn
Copy link
Contributor

@dmackinn dmackinn commented Mar 5, 2026

Summary by CodeRabbit

  • Enhancements
    • Enhanced tracking capabilities for the "Powered by Searchcraft" attribution link. The link now includes improved analytics parameters to better monitor referral sources, track campaign effectiveness, measure user engagement, and provide comprehensive insights into how visitors discover your site through the Searchcraft plugin integration.

@dmackinn dmackinn requested a review from a team March 5, 2026 05:36
@coderabbitai
Copy link

coderabbitai bot commented Mar 5, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: d39d3d87-93a3-40fe-91c4-471954d40c6d

📥 Commits

Reviewing files that changed from the base of the PR and between 44b030c and dabea9f.

📒 Files selected for processing (2)
  • public/templates/common-template-values.php
  • public/templates/search-results.php

Walkthrough

The pull request adds UTM parameter tracking to external Searchcraft plugin links. A new dynamic link with tracking parameters is constructed in the common template values file and replaces two hardcoded external links in the search results template.

Changes

Cohort / File(s) Summary
UTM Link Generation
public/templates/common-template-values.php
Introduces logic to construct a "powered by" link with UTM parameters (source, medium, campaign, content, and plugin version) appended via add_query_arg to https://searchcraft.io/.
Link References
public/templates/search-results.php
Replaces two hardcoded external links with the dynamically generated $searchcraft_powered_by_link variable, each properly escaped with esc_url.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

🚥 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 pull request title accurately describes the main change: adding UTM parameters to the powered-by link, which is reflected in both modified files.
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 feaure/SC-1901-utm-params

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.

@dmackinn dmackinn merged commit 8a8db5b into development Mar 5, 2026
5 checks passed
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.

2 participants