Alpha
Channels
  • general
  • design
  • engineering
  • product
  • random
Direct Messages
  • SCSarah Chen
  • MPMaya Patel

Technical discussions and code reviews

Today
AR
Alex R.8:20 AM

Heads up — upgrading to Next.js 16 this morning. PR is up for review.

MP
Maya Patel8:22 AM

Nice. Any breaking changes we should watch for?

AR
Alex R.8:23 AM

Mostly the new async params in route handlers. I've already migrated our routes.

8:24

Also removed the legacy middleware — the new auth pattern is way cleaner.

MP
Maya Patel8:50 AM

Can someone take a look at PR #142? Auth redirect is acting weird on the callback route.

AR
Alex R.8:52 AM

On it. Kairo already flagged it in the code review thread — looks like a missing callback param.

MP
Maya Patel8:53 AM

That was fast!

PD
Priya Desai8:58 AM

While we're on auth — we should add rate limiting to the login endpoint. I'll create a task.

AR
Alex R.9:10 AM

Fix is up. The redirect was missing the callback param when the session expired mid-flow.

9:11

Also added a regression test so this doesn't sneak back in.

MP
Maya Patel9:15 AM

Approved ✅ Great fix, merging now.

JK
Jamie Kim9:20 AM

Is the staging deploy pipeline still green? I want to preview the new wizard.

AR
Alex R.9:21 AM

Yep, all green. Preview URL is in the PR description.

PD
Priya Desai9:30 AM

Rate limit task created: ENG-287. Targeting next sprint.

Message engineering