Skip to content

Conversation

@alan-agius4
Copy link
Contributor

@alan-agius4 alan-agius4 commented Apr 28, 2023

Using @angular/platform-browser-dynamic is no longer required for JIT scenarios with Ivy. Instead @angular/compiler should be imported instead.

This change is part of the effort to reduce the server bundles sizes, which is needed to support cloud workers.

BREAKING CHANGE:
Users that are using SSR with JIT mode will now need to add import to @angular/compiler before bootstrapping the application.

NOTE: this does not effect users using the Angular CLI.

@alan-agius4 alan-agius4 added target: rc This PR is targeted for the next release-candidate area: server Issues related to server-side rendering labels Apr 28, 2023
@ngbot ngbot bot added this to the Backlog milestone Apr 28, 2023
@alan-agius4 alan-agius4 force-pushed the platform-server-dynamic-ssr branch from 0c84f78 to 95430bd Compare April 28, 2023 12:10
@alan-agius4 alan-agius4 added the action: review The PR is still awaiting reviews from at least one requested reviewer label Apr 28, 2023
@alan-agius4 alan-agius4 marked this pull request as ready for review April 28, 2023 16:27
@AndrewKushnir AndrewKushnir requested a review from alxhub April 28, 2023 16:27
Copy link
Contributor

@AndrewKushnir AndrewKushnir left a comment

Choose a reason for hiding this comment

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

Reviewed-for: public-api

@alxhub could you please also take a look at the change before the merge?

@alan-agius4 alan-agius4 force-pushed the platform-server-dynamic-ssr branch from 95430bd to 0136848 Compare April 28, 2023 16:33
@pullapprove pullapprove bot requested a review from jessicajaniuk April 28, 2023 16:33
@angular-robot angular-robot bot added the detected: breaking change PR contains a commit with a breaking change label Apr 28, 2023
…-dynamic`

Using `@angular/platform-browser-dynamic` is no longer required for JIT scenarios with Ivy. Instead `@angular/compiler` should be imported instead.

This change is part of the effort to reduce the server bundles sizes, which is needed to support cloud workers.

BREAKING CHANGE:
Users that are using SSR with JIT mode will now need to add  `import to @angular/compiler` before bootstrapping the application.

**NOTE:** this does not effect users using the Angular CLI.
@alan-agius4 alan-agius4 force-pushed the platform-server-dynamic-ssr branch from 0136848 to 82b51e9 Compare April 28, 2023 16:34
Copy link
Contributor

@jessicajaniuk jessicajaniuk left a comment

Choose a reason for hiding this comment

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

reviewed-for: public-api

@alan-agius4
Copy link
Contributor Author

Caretaker note: Presubmit was green prior to the commit message change.

@alan-agius4 alan-agius4 added the merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note label Apr 28, 2023
@jessicajaniuk jessicajaniuk removed the request for review from pkozlowski-opensource April 28, 2023 16:35
@alan-agius4 alan-agius4 modified the milestones: Backlog, v16-final Apr 28, 2023
@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 Apr 28, 2023
@alxhub
Copy link
Member

alxhub commented May 1, 2023

This PR was merged into the repository by commit 06452af.

alxhub pushed a commit that referenced this pull request May 1, 2023
…-dynamic` (#50064)

Using `@angular/platform-browser-dynamic` is no longer required for JIT scenarios with Ivy. Instead `@angular/compiler` should be imported instead.

This change is part of the effort to reduce the server bundles sizes, which is needed to support cloud workers.

BREAKING CHANGE:
Users that are using SSR with JIT mode will now need to add  `import to @angular/compiler` before bootstrapping the application.

**NOTE:** this does not effect users using the Angular CLI.

PR Close #50064
@alxhub alxhub closed this in 06452af May 1, 2023
@alan-agius4 alan-agius4 deleted the platform-server-dynamic-ssr branch May 1, 2023 15:01
sr5434 pushed a commit to sr5434/angular that referenced this pull request May 3, 2023
…-dynamic` (angular#50064)

Using `@angular/platform-browser-dynamic` is no longer required for JIT scenarios with Ivy. Instead `@angular/compiler` should be imported instead.

This change is part of the effort to reduce the server bundles sizes, which is needed to support cloud workers.

BREAKING CHANGE:
Users that are using SSR with JIT mode will now need to add  `import to @angular/compiler` before bootstrapping the application.

**NOTE:** this does not effect users using the Angular CLI.

PR Close angular#50064
@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 Jun 1, 2023
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 area: server Issues related to server-side rendering detected: breaking change PR contains a commit with a breaking change merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note target: rc This PR is targeted for the next release-candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants