TudorandClaude Opus 5
b793640507
The rendering split is dictated by CI building with no database. /blog, /blog/rss.xml and /content-sitemap.xml have no dynamic params, so Next prerenders them at build time and the build fails on a missing Payload secret — caught here, not on staging. They are force-dynamic instead: one indexed query against Postgres on the same Docker network, and a newly published post appears immediately rather than waiting on a revalidation. /blog/[slug] keeps ISR, because with no generateStaticParams there is nothing to prerender; it is generated on first request and cached, which is exactly what the collection's afterChange hook exists to invalidate. RichText takes `converters`, not `blocks`, in Payload 3.88, and the default converters must be spread or every paragraph and heading loses its renderer and the body comes out empty. BlogPosting references the Person and Organization by @id rather than repeating them, so every post and the About page resolve to one author entity instead of declaring several people with the same name. /sitemap.xml is proxied from FastAPI, which knows nothing about Payload, so the Next-owned URLs get their own sitemap and robots.txt lists both. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017YmbBhr8s7GusjDE12hrZM
77 lines
2.4 KiB
TypeScript
77 lines
2.4 KiB
TypeScript
import type { Metadata } from 'next';
|
|
import Link from 'next/link';
|
|
import { getCachedPayload } from '@/lib/payload';
|
|
import { absoluteUrl } from '@/lib/site';
|
|
import styles from './Blog.module.css';
|
|
|
|
/*
|
|
* Dynamic, not ISR.
|
|
*
|
|
* This route has no dynamic params, so Next prerenders it at build time — and
|
|
* CI builds the image with no database reachable, which fails the build. It is
|
|
* a single indexed query against Postgres on the same Docker network, so
|
|
* rendering per request is cheap, and it means a newly published post appears
|
|
* here immediately rather than waiting on a revalidation.
|
|
*/
|
|
export const dynamic = 'force-dynamic';
|
|
|
|
export const metadata: Metadata = {
|
|
title: 'Blog',
|
|
description:
|
|
'Notes on what school performance data shows, and what it does not.',
|
|
alternates: { canonical: absoluteUrl('/blog') },
|
|
};
|
|
|
|
function formatDate(value: string) {
|
|
return new Date(value).toLocaleDateString('en-GB', {
|
|
day: 'numeric',
|
|
month: 'long',
|
|
year: 'numeric',
|
|
});
|
|
}
|
|
|
|
export default async function BlogIndexPage() {
|
|
const payload = await getCachedPayload();
|
|
const { docs } = await payload.find({
|
|
collection: 'posts',
|
|
where: { _status: { equals: 'published' } },
|
|
sort: '-publishedAt',
|
|
limit: 50,
|
|
depth: 0,
|
|
});
|
|
|
|
return (
|
|
<div className={styles.page}>
|
|
<header className={styles.header}>
|
|
<p className={styles.kicker}>Blog</p>
|
|
<h1 className={styles.heading}>Notes on the numbers</h1>
|
|
<p className={styles.standfirst}>
|
|
What school performance data shows, what it doesn't, and how to
|
|
read it without being misled. Written by{' '}
|
|
<Link href="/about" className={styles.link}>Tudor</Link>.
|
|
</p>
|
|
</header>
|
|
|
|
{docs.length === 0 ? (
|
|
<p className={styles.empty}>No posts yet.</p>
|
|
) : (
|
|
<ul className={styles.list}>
|
|
{docs.map((post) => (
|
|
<li key={post.id} className={styles.item}>
|
|
<time className={styles.date} dateTime={String(post.publishedAt)}>
|
|
{formatDate(String(post.publishedAt))}
|
|
</time>
|
|
<h2 className={styles.itemTitle}>
|
|
<Link href={`/blog/${post.slug}`} className={styles.itemLink}>
|
|
{post.title}
|
|
</Link>
|
|
</h2>
|
|
<p className={styles.excerpt}>{post.excerpt}</p>
|
|
</li>
|
|
))}
|
|
</ul>
|
|
)}
|
|
</div>
|
|
);
|
|
}
|