Skip to content

fix: Changelog - #123

Merged
perkinsjr merged 3 commits into
mainfrom
part-2
Apr 10, 2026
Merged

perkinsjr merged 3 commits into
mainfrom
part-2

Conversation

@perkinsjr

Copy link
Copy Markdown
Member

What does this PR do?

Fixes # (issue)

If there is not an issue for this, please create one first. This is used to tracking purposes and also helps use understand why this PR exists

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • Chore (refactoring code, technical debt, workflow improvements)
  • Blog post
  • Changelog
  • Enhancement (small improvements)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

How should this be tested?

  • Test A
  • Test B

Checklist

Required

  • Filled out the "How to test" section in this PR
  • Read Contributing Guide
  • Self-reviewed my own code
  • Commented on my code in hard-to-understand areas
  • Ran pnpm build
  • Ran pnpm fmt
  • Checked for warnings, there are none
  • Removed all console.logs
  • Merged the latest changes from main onto my branch with git pull origin main
  • My changes don't cause any responsiveness issues

Appreciated

  • If a UI change was made: Added a screen recording or screenshots to this PR

@perkinsjr
perkinsjr requested a review from MichaelUnkey as a code owner April 10, 2026 17:04
@vercel

vercel Bot commented Apr 10, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
ratelimit Ready Ready Preview, Comment Apr 10, 2026 5:04pm
www Ready Ready Preview, Comment Apr 10, 2026 5:04pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
play Skipped Skipped Apr 10, 2026 5:04pm

Request Review

@perkinsjr
perkinsjr merged commit a0f23cf into main Apr 10, 2026
5 of 7 checks passed
@perkinsjr
perkinsjr deleted the part-2 branch April 10, 2026 17:04
@coderabbitai

coderabbitai Bot commented Apr 10, 2026 •

Copy link
Copy Markdown
Contributor

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: d3ebb848-9cff-48d5-8c9c-352e098aa251

📥 Commits

Reviewing files that changed from the base of the PR and between d438a48 and f75dc30.

📒 Files selected for processing (3)
  • apps/www/app/api/revalidate/route.ts
  • apps/www/app/changelog/data.ts
  • apps/www/app/changelog/page.tsx

Walkthrough

Three files modified to refactor changelog caching and component structure. Cache invalidation switched from path-based to tag-based revalidation with error handling. Changelog data function now uses Next.js cache directives. Page component simplified by removing the separate async boundary component and inlining data fetching directly.

Changes

Cohort / File(s) Summary
Cache Configuration
apps/www/app/api/revalidate/route.ts, apps/www/app/changelog/data.ts
Cache invalidation strategy changed from revalidatePath() to revalidateTag(). Authorization checks enhanced to validate process.env.REVALIDATION_TOKEN is set. Revalidation wrapped in try/catch with error responses. Changelog data function now includes "use cache", cacheLife("days"), and cacheTag("changelogs"); removed next: { revalidate: 86400 } from fetch calls.
Component Structure
apps/www/app/changelog/page.tsx
Removed separate ChangelogFeed async component and inlined its logic directly into the page. Page component converted from synchronous to async default export, eliminating the Suspense wrapper layer while preserving MDX rendering behavior.

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

This branch was successfully deployed

2 active and 1 inactive deployments
Preview – www — f75dc30b Deployed Apr 10, 2026 by vercel[bot]
Preview – ratelimit — f75dc30b Deployed Apr 10, 2026 by vercel[bot]
Preview – play — f75dc30b Deployed Apr 10, 2026 by vercel[bot]
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