Skip to content

Conversation

@ematipico
Copy link
Member

Changes

This PR adds build.format to the SSR Manifest.

This information is eventually needed by the i18n middleware to correctly compute the redirect.

Testing

Existing tests should pass

Docs

N/A

@changeset-bot
Copy link

changeset-bot bot commented Jan 19, 2024

🦋 Changeset detected

Latest commit: 7375ebf

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

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

@github-actions github-actions bot added the pkg: astro Related to the core `astro` package (scope) label Jan 19, 2024
@ematipico ematipico changed the title fix(i18n): pass build.format when computing the direct fix(i18n): pass build.format when computing the redirect Jan 19, 2024
@lilnasy
Copy link
Contributor

lilnasy commented Jan 19, 2024

Could you elaborate on how build.format affects deployed paths?

@matthewp
Copy link
Contributor

Can a test be added for this? Does it fix an existing known bug?

@ematipico
Copy link
Member Author

Could you elaborate on how build.format affects deployed paths?

https://docs.astro.build/en/reference/configuration-reference/#buildformat

@ematipico
Copy link
Member Author

Can a test be added for this? Does it fix an existing known bug?

Yeah, as you can see there's a regression that actually makes sense and needs to be updated.

I'll add more test cases

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pkg: astro Related to the core `astro` package (scope)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants