Skip to content

fix(wordpress): remove anonymous telemetry ping (review blocker)#38

Open
asafamos wants to merge 1 commit into
mainfrom
fix/wp-remove-telemetry
Open

fix(wordpress): remove anonymous telemetry ping (review blocker)#38
asafamos wants to merge 1 commit into
mainfrom
fix/wp-remove-telemetry

Conversation

@asafamos

@asafamos asafamos commented Jun 8, 2026

Copy link
Copy Markdown
Owner

WordPress.org review flagged the /api/track scan-complete ping as phoning home without opt-in (Guidelines 7 & 9). GA-style tracking isn't allowed even opt-in, so removed it entirely (JS + PHP config + readme disclosure). Bumped to 1.2.1. Plugin now makes zero outbound requests by default; the opt-in daily cron is the only remaining optional network feature.

🤖 Generated with Claude Code

WordPress.org review flagged the scan-complete ping to /api/track as
"Phoning Home / Collecting User Data Without Opt-In Consent" (Guidelines
7 & 9). Per their guidance, GA-style usage tracking isn't permitted even
opt-in — so remove it entirely rather than gate it.

- axle-scan.js: drop pingTelemetry() + its call.
- axle.php: drop the telemetryUrl from wp_localize_script.
- readme.txt: remove the "anonymous usage ping" disclosure + FAQ mentions;
  the opt-in daily cron (a service the user enables) is the only remaining
  optional network feature. Bump to 1.2.1 + changelog.

The plugin now makes zero outbound requests by default.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@vercel

vercel Bot commented Jun 8, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
axle Ready Ready Preview, Comment Jun 8, 2026 10:11am

@github-actions

github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown

✅ No accessibility violations detected

Scanned http://localhost:3000 at 2026-06-08T10:11:11.554Z

axe-core automated checks catch roughly 57% of WCAG issues. Manual review still recommended.

Powered by axle · Remediation assistance, not compliance certification.

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