Skip to content

fix: add Open Graph type to leaderboard - #104

Merged
anurag629 merged 1 commit into
codercops:developfrom
srujankaleru2007:fix/97-leaderboard-og-type
Oct 26, 2026
Merged

anurag629 merged 1 commit into
codercops:developfrom
srujankaleru2007:fix/97-leaderboard-og-type

Conversation

@srujankaleru2007

Copy link
Copy Markdown
Contributor

What does this change?

Adds og:type=website to leaderboard pages and covers it with a regression test.

Closes #97

How I tested this

  • npm run typecheck — passed.
  • npm test — 15 files and 379 tests passed. TEMP and TMP pointed to a workspace temp directory because the sandbox's default temp path caused EPERM errors.
  • Verified the regression test fails when the metadata line is removed and passes when restored.

AI assistance

Codex assisted with code review, implementation, regression verification, and PR preparation.

Checklist

  • I was assigned the linked issue
  • npm run typecheck and npm test pass
  • New behavior has a test in test/
  • For a bug fix: the new test fails without my change
  • No tokens or secrets in the diff

@anurag629 anurag629 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @srujankaleru2007, this works. Typecheck and all 379 tests pass on top of develop, and the new test fails with develop's page.ts.

@anurag629 anurag629 added the hacktoberfest-accepted PR counts for Hacktoberfest label Oct 26, 2026
@anurag629
anurag629 merged commit e4df97c into codercops:develop Oct 26, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hacktoberfest-accepted PR counts for Hacktoberfest

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The leaderboard page has no og:type

2 participants