Support i18n localeDetection with value false

[alacroix]

Hey there,

We found that the locale detection is forced and doesn't support the option localeDetection to false as linked below:

https://nextjs.org/docs/pages/building-your-application/routing/internationalization#disabling-automatic-locale-detection

Since it's an opt-out option, I forced the exact check with false

Thanks!

[changeset-bot]

🦋 Changeset detected

Latest commit: 92ec92b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages

Name	Type
@opennextjs/aws	Patch
app-pages-router	Patch
app-router	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[pkg-pr-new]

Open in Stackblitz

pnpm add https://pkg.pr.new/@opennextjs/aws@585

commit: 92ec92b

[khuezy]

Hey thanks! Would you be open to adding some unit tests around the i18 stuff? eg: https://github.com/opennextjs/opennextjs-aws/tree/main/packages/tests-unit/tests/core/routing

I want to add the i18 to our e2e apps too since support for it is still iffy, but that's for another day.

[alacroix]

Done @khuezy 👍

I didn't test the accept-header file since it's copied from NextJS

// Copied from Next.js source code
// https://github.com/vercel/next.js/blob/canary/packages/next/src/server/accept-header.ts

[conico974]

LGTM Thanks for that