PR #60

121 votes ยท 118 up ยท 3 down

View on GitHub
121
Total Votes
+118
Upvotes
-3
Downvotes
+118-3

Comments(4)

๐Ÿ’ฌ
bigintersmindReview comment#60

Good catch. sort=updated returns recently active closed PRs, but doesn't guarantee merge order. Added explicit client-side sorting by merged_at to ensure true chronological merge order.

๐Ÿ’ฌ
lens0021Review comment#60

Why sort=updated here?

โ–ธ
bigintersmindComment#60Add Hall of Chaos - Past Winners section

We're now IE6 ready!

Screenshot 2026-01-19 at 7 29 06โ€ฏPM

edit: removed skridlevsky's PRs from the list, since they are maintenance fixes and not voted in.

โ–ธ
openchaos-bot[bot]Comment#60Add Hall of Chaos - Past Winners section

๐Ÿค– OpenChaos Bot

Summary: This PR adds a "Hall of Chaos" section to the homepage, displaying a list of merged pull requests. It fetches merged PRs from the GitHub API and displays them in a visually appealing card format.

Files changed: 4 (src/app/page.tsx, src/components/HallOfChaos.tsx, src/components/HallOfChaosCard.tsx, src/lib/github.ts)

Impact: Medium - Adds new feature to display past winners (merged PRs), enhancing user engagement and providing a historical record.


openchaos-bot

All Activity(129)

๐Ÿ’ฌ
bigintersmindReview comment#60

Good catch. sort=updated returns recently active closed PRs, but doesn't guarantee merge order. Added explicit client-side sorting by merged_at to ensure true chronological merge order.

๐Ÿ’ฌ
lens0021Review comment#60

Why sort=updated here?

โ–ธ
bigintersmindComment#60Add Hall of Chaos - Past Winners section

We're now IE6 ready!

Screenshot 2026-01-19 at 7 29 06โ€ฏPM

edit: removed skridlevsky's PRs from the list, since they are maintenance fixes and not voted in.

โšก
blopaReaction#60๐Ÿ‘
โšก
nobkdReaction#60๐Ÿ‘Ž
โ–ธ
openchaos-bot[bot]Comment#60Add Hall of Chaos - Past Winners section

๐Ÿค– OpenChaos Bot

Summary: This PR adds a "Hall of Chaos" section to the homepage, displaying a list of merged pull requests. It fetches merged PRs from the GitHub API and displays them in a visually appealing card format.

Files changed: 4 (src/app/page.tsx, src/components/HallOfChaos.tsx, src/components/HallOfChaosCard.tsx, src/lib/github.ts)

Impact: Medium - Adds new feature to display past winners (merged PRs), enhancing user engagement and providing a historical record.


openchaos-bot

โšก
sei0oReaction#60๐Ÿ‘
โšก
robpcReaction#60๐Ÿ‘
โšก
d4mrReaction#60๐Ÿ‘
โ—‘

Add Hall of Chaos - Past Winners section

Summary

  • Adds a Hall of Chaos section that displays previously merged PRs
  • Each winner is immortalized with their PR number, title, author, and merge date
  • Shows the evolution of the site through community votes

What's Added

  • getMergedPRs() function in /src/lib/github.ts - fetches closed+merged PRs from GitHub API
  • HallOfChaos.tsx - async server component that renders the list of past winners
  • HallOfChaosCard.tsx - individual card component with ๐Ÿ† trophy and green "MERGED" badge
  • New section on homepage below the open PRs voting section

Why

The site is about its own evolution through community voting. Showing that history:

  • Makes the site self-documenting
  • Motivates contributors (their work gets immortalized)
  • Lets visitors trace how the site got to its current chaotic state

Screenshot

Screenshot 2026-01-19 at 7 29 06โ€ฏPM