Skip to content

Conversation

@lunika
Copy link
Member

@lunika lunika commented Dec 3, 2025

Purpose

The / location is not trying the $uri/index.html file. We should try
this instad of $uri/ because when a new static page is added, we always
have this pattern.

Proposal

  • 🐛(nginx) fix / location to handle new static pages

@lunika lunika requested a review from AntoLC December 3, 2025 08:24
@lunika lunika self-assigned this Dec 3, 2025
@lunika lunika added the bug Something isn't working label Dec 3, 2025
@github-actions
Copy link

github-actions bot commented Dec 3, 2025

Size Change: +3 B (0%)

Total Size: 4.11 MB

Filename Size Change
apps/impress/out/_next/static/4f12416a/_buildManifest.js 0 B -906 B (removed) 🏆
apps/impress/out/_next/static/3862608f/_buildManifest.js 906 B +906 B (new file) 🆕

compressed-size-action

@lunika lunika force-pushed the fix/nginx_new_location branch 2 times, most recently from 25af715 to 536afc9 Compare December 3, 2025 10:59
@lunika lunika changed the title 🐛(nginx) add missing /docs/new location 🐛(nginx) fix / location to handle new static pages Dec 3, 2025
@AntoLC AntoLC added the preview label Dec 3, 2025
@github-actions
Copy link

github-actions bot commented Dec 3, 2025

🚀 Preview will be available at https://1682-docs.ppr-docs.beta.numerique.gouv.fr/

You can use the existing account with these credentials:

  • username: docs
  • password: docs

You can also create a new account if you want to.

Once this Pull Request is merged, the preview will be destroyed.

The / location is not trying the $uri/index.html file. We should try
this instad of $uri/ because when a new static page is added, we always
have this pattern.
@lunika lunika force-pushed the fix/nginx_new_location branch from 536afc9 to 54fe70d Compare December 3, 2025 11:14
Copy link
Collaborator

@AntoLC AntoLC left a comment

Choose a reason for hiding this comment

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

Perfect ✅

@lunika lunika merged commit 54fe70d into main Dec 3, 2025
27 of 28 checks passed
@lunika lunika deleted the fix/nginx_new_location branch December 3, 2025 11:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working preview

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants