Skip to content

fix(seo): use derived noindex for shadow-banned profiles#6019

Merged
nimrodkra merged 2 commits into
mainfrom
fix/vordr-profile-noindex
May 10, 2026
Merged

fix(seo): use derived noindex for shadow-banned profiles#6019
nimrodkra merged 2 commits into
mainfrom
fix/vordr-profile-noindex

Conversation

@nimrodkra
Copy link
Copy Markdown
Member

@nimrodkra nimrodkra commented May 10, 2026

Summary

Companion to dailydotdev/daily-api#3864.

The profile page's getStaticProps currently uses noindex: user.reputation <= 10, which misses shadow-banned (vordr) users with reputation > 10 — they get indexed and inbound links pass equity. This PR consumes the new server-derived User.noindex field from the API instead.

  • Selects noindex in USER_BY_ID_STATIC_FIELDS_QUERY.
  • Adds noindex?: boolean to PublicProfile (optional to avoid touching unrelated test/storybook fixtures; the API guarantees a value at runtime).
  • Replaces noindex: user.reputation <= 10 with noindex: !!user.noindex in ProfileLayout/getStaticProps.

The visible page content is unchanged — only the <meta name="robots"> tag changes, preserving the shadow-ban illusion.

Test plan

  • Deploy with the API PR.
  • curl https://app.daily.dev/<vordr-user-slug> -> 200 with <meta name="robots" content="noindex,nofollow">.
  • curl https://app.daily.dev/<vordr-user-slug> | grep robots for a non-vordr rep>10 user -> no robots meta tag (or only the default).

Made with Cursor

Preview domain

https://fix-vordr-profile-noindex.preview.app.daily.dev

Selects the new server-derived `noindex` field from the API on the
public profile page, so shadow-banned (vordr) users get
`<meta name="robots" content="noindex,nofollow">` without changing the
visible page content.

Co-authored-by: Cursor <cursoragent@cursor.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented May 10, 2026

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

Project Deployment Actions Updated (UTC)
daily-webapp Ready Ready Preview May 10, 2026 7:48pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
storybook Ignored Ignored May 10, 2026 7:48pm

Request Review

Touched by the noindex change, so the strict-changed guard surfaces
pre-existing errors. Narrow types via guards/optionals without changing
runtime behavior.

Co-authored-by: Cursor <cursoragent@cursor.com>
@nimrodkra nimrodkra merged commit 6fc10dc into main May 10, 2026
14 checks passed
@nimrodkra nimrodkra deleted the fix/vordr-profile-noindex branch May 10, 2026 19:53
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