Skip to content

Conversation

@alan-agius4
Copy link
Collaborator

This commit fixes a regression which causes a pre-transform error when using vite with ssr. The request.url is now passed to the index transformer instead of request.originalUrl. This is because the request.url will have a value of the index.html.

Closes #26897

@alan-agius4 alan-agius4 added action: review The PR is still awaiting reviews from at least one requested reviewer target: patch This PR is targeted for the next patch release labels Jan 22, 2024
@alan-agius4 alan-agius4 requested a review from clydin January 22, 2024 08:24
…te with SSR

This commit fixes a regression which causes a pre-transform error when using vite with ssr. The `request.url` is now passed to the index transformer instead of `request.originalUrl`. This is because the `request.url` will have a value of the `index.html`.

Closes angular#26897
@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Jan 22, 2024
@alan-agius4 alan-agius4 merged commit 0b0df5a into angular:main Jan 22, 2024
@alan-agius4 alan-agius4 deleted the vite-pre-transform branch January 22, 2024 15:05
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Feb 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Vite Pre-transform error

2 participants