Skip to content

fix: enable SSR for dynamic admin routes#262

Merged
eric-martin-gg merged 6 commits intomainfrom
fix/admin-dynamic-routes
Apr 8, 2026
Merged

fix: enable SSR for dynamic admin routes#262
eric-martin-gg merged 6 commits intomainfrom
fix/admin-dynamic-routes

Conversation

@eric-martin-gg
Copy link
Copy Markdown
Collaborator

Summary

  • Removes output: "export" from next.config.mjs — static export was generating a literal [pid].html file instead of handling dynamic route segments, causing 404s on /admin/student-details/:id
  • Updates Netlify publish directory from out to .next so the existing @netlify/plugin-nextjs plugin serves SSR routes

Test plan

  • Verify /admin still loads and lists applications
  • Verify /admin/student-details/<id> resolves correctly (no more 404)
  • Verify Netlify deploy preview works

🤖 Generated with Claude Code

Remove static export mode (`output: "export"`) which was causing 404s
on /admin/student-details/[pid] routes. Update Netlify publish dir to
.next so the existing @netlify/plugin-nextjs handles SSR.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 7, 2026

Deploy Preview for uwblueprint ready!

Name Link
🔨 Latest commit 185030d
🔍 Latest deploy log https://app.netlify.com/projects/uwblueprint/deploys/69d5bcddb373d400086ae155
😎 Deploy Preview https://deploy-preview-262--uwblueprint.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Upgrade @typescript-eslint packages from v4 to v5 for ESLint 8
compatibility. Fix pre-existing prettier and eslint errors (unescaped
entity, auto-generated next-env.d.ts). Add out/ to .prettierignore.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@eric-martin-gg eric-martin-gg marked this pull request as ready for review April 8, 2026 00:15
@sthuray sthuray self-requested a review April 8, 2026 01:50
</p>
<p className="text-xs text-charcoal-500">
Don't worry, a backup is generated in the code!
Don&apos;t worry, a backup is generated in the code!
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

weird change

@eric-martin-gg eric-martin-gg merged commit 6dc53f2 into main Apr 8, 2026
5 checks passed
@eric-martin-gg eric-martin-gg deleted the fix/admin-dynamic-routes branch April 8, 2026 13:54
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