Skip to content

refactor(core): avoid invoking IntersectionObserver in defer triggers on the server#52306

Closed
AndrewKushnir wants to merge 1 commit into
angular:mainfrom
AndrewKushnir:defer_viewport_server
Closed

refactor(core): avoid invoking IntersectionObserver in defer triggers on the server#52306
AndrewKushnir wants to merge 1 commit into
angular:mainfrom
AndrewKushnir:defer_viewport_server

Conversation

@AndrewKushnir
Copy link
Copy Markdown
Contributor

This commit updates the logic to make sure that DOM-specific triggers do not produce errors on the server.

Resolves #52304.

Does this PR introduce a breaking change?

  • Yes
  • No

@AndrewKushnir AndrewKushnir added action: review The PR is still awaiting reviews from at least one requested reviewer area: core Issues related to the framework runtime target: rc This PR is targeted for the next release-candidate core: defer Issues related to @defer blocks. labels Oct 21, 2023
@ngbot ngbot Bot added this to the Backlog milestone Oct 21, 2023
… on the server

This commit updates the logic to make sure that DOM-specific triggers do not produce errors on the server.

Resolves angular#52304.
@AndrewKushnir AndrewKushnir added action: presubmit The PR is in need of a google3 presubmit and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Oct 21, 2023
@AndrewKushnir
Copy link
Copy Markdown
Contributor Author

Presubmit.

@AndrewKushnir AndrewKushnir added action: merge The PR is ready for merge by the caretaker and removed action: presubmit The PR is in need of a google3 presubmit labels Oct 21, 2023
@dylhunn
Copy link
Copy Markdown
Contributor

dylhunn commented Oct 23, 2023

This PR was merged into the repository by commit 666853f.

dylhunn pushed a commit that referenced this pull request Oct 23, 2023
… on the server (#52306)

This commit updates the logic to make sure that DOM-specific triggers do not produce errors on the server.

Resolves #52304.

PR Close #52306
@dylhunn dylhunn closed this in 666853f Oct 23, 2023
@angular-automatic-lock-bot
Copy link
Copy Markdown

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 Nov 23, 2023
ChellappanRajan pushed a commit to ChellappanRajan/angular that referenced this pull request Jan 23, 2024
… on the server (angular#52306)

This commit updates the logic to make sure that DOM-specific triggers do not produce errors on the server.

Resolves angular#52304.

PR Close angular#52306
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: core Issues related to the framework runtime core: defer Issues related to @defer blocks. target: rc This PR is targeted for the next release-candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Defer block with prefetch on viewport throws because IntersectionObserver is not defined

3 participants