Skip to content

fix(landing): redirect /privacy → /policy to unblock CWS extension publish - #4

Merged
Smetools merged 1 commit into
masterfrom
hassan-privacy-redirect
May 17, 2026
Merged

fix(landing): redirect /privacy → /policy to unblock CWS extension publish#4
Smetools merged 1 commit into
masterfrom
hassan-privacy-redirect

Conversation

@Smetools

@Smetools Smetools commented May 17, 2026

Copy link
Copy Markdown
Collaborator

Why

Chrome Web Store has `https://knowcap.ai/privacy\` registered as the
Knowcap extension's privacy URL, but the actual privacy policy page
lives at `/policy` (created in 988833f, never at /privacy). Google's
publish-time reachability check 404s on the URL and blocks every
extension publish with:

```
"Publish condition not met: Privacy policy link is not reachable."
```

Has silently blocked every extension publish since 1.21.6 (May 6).
1.21.19 has been sitting unpublished in the CWS dashboard for days.

Fix

Next.js permanent redirects `/privacy` and `/privacy-policy` → `/policy`
in `next.config.js`. Both URLs now return 200 (via 308), Google's
reachability check passes, and anyone googling "knowcap privacy" also
lands on the policy.

Test Plan

Summary by CodeRabbit

  • Chores
    • Added permanent URL redirects from /privacy and /privacy-policy to /policy.

Review Change Stack

The Chrome Web Store has https://knowcap.ai/privacy registered as the
Knowcap extension's privacy URL, but the actual privacy policy page
lives at /policy (built in 988833f, never at /privacy). Google's
publish-time reachability check 404s on the URL and blocks every
extension publish with:

  "Publish condition not met: Privacy policy link is not reachable."

This has been silently breaking publishes since 1.21.6 (May 6) —
1.21.19 has been sitting unpublished in the CWS dashboard for days.

Fix: Next.js permanent redirects for /privacy and /privacy-policy →
/policy. Both URLs now return 200 (via 308), Google's check passes,
and anyone googling "knowcap privacy" also lands on the policy.
@vercel

vercel Bot commented May 17, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
app Building Building Preview, Comment May 17, 2026 11:20pm

@coderabbitai

coderabbitai Bot commented May 17, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 4f3c748a-c032-4266-8a5d-4071a205f2ae

📥 Commits

Reviewing files that changed from the base of the PR and between f57c8d3 and 89ab843.

📒 Files selected for processing (1)
  • app/next.config.js

📝 Walkthrough

Walkthrough

The PR adds an async redirects() configuration hook to the Next.js config that permanently redirects two legacy privacy routes (/privacy and /privacy-policy) to a unified /policy endpoint.

Changes

Privacy Policy Redirects

Layer / File(s) Summary
Privacy policy redirect rules
app/next.config.js
An async redirects() function with permanent redirect rules mapping /privacy and /privacy-policy to /policy, including comments explaining the publish-time behavior.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

Poem

🐰 Two paths once lost in time,
Now guide where policies align,
From /privacy to /policy bright,
A hop, a redirect, all feels right!
The rabbit hops through cleaner routes. ✨

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch hassan-privacy-redirect

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

@Smetools
Smetools merged commit 36a087d into master May 17, 2026
1 of 3 checks passed
@Smetools
Smetools deleted the hassan-privacy-redirect branch May 17, 2026 23:20
@Smetools
Smetools restored the hassan-privacy-redirect branch June 1, 2026 08:57
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