Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(core): computation of outDir for SSR builds #12249

Closed
wants to merge 2 commits into from

Conversation

ematipico
Copy link
Member

Changes

Closes #12248
Closes PLT-2594

The bug was happening only for SSR builds.

Testing

Added a new test

Docs

N/A

Copy link

changeset-bot bot commented Oct 17, 2024

🦋 Changeset detected

Latest commit: 49a32e3

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 Oct 17, 2024
@bluwy
Copy link
Member

bluwy commented Oct 17, 2024

I don't know if we can fix this in main now. I discovered that it's kinda a breaking change if someone else relied on the old behaviour (for workaround reasons), that's why I sent the fix to next at #11916

Copy link
Member

@bluwy bluwy left a comment

Choose a reason for hiding this comment

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

Blocking due to my previous comment

@ematipico
Copy link
Member Author

I don't know if we can fix this in main now. I discovered that it's kinda a breaking change if someone else relied on the old behaviour (for workaround reasons), that's why I sent the fix to next at #11916

The issue is only for SSR, which means that it worked for SSG builds. I would agree with you if both were affected by the incorrect behaviour but in this case it's not.

Plus, I don't think we have ever suggested a workaround in our docs like we did for pagination

@ematipico ematipico closed this Oct 17, 2024
@bluwy
Copy link
Member

bluwy commented Oct 17, 2024

I think there's still a chance we could break SSR adapters if we change this in main. We don't explicitly suggest a workaround, but it's the only workaround users can do when they're hit with this quirk and it goes unreported. And the behaviour had existed for a long while that it's as risky to fix now like the pagination fix.

It's probably safer to handle this in next so we don't cause potential regression now IMO.

@ematipico ematipico deleted the fix/output-dir-server branch October 21, 2024 14:53
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.

Specifying config.outDir results in broken directory structure
3 participants