Skip to content
This repository has been archived by the owner on Nov 22, 2024. It is now read-only.

fix(@nguniversal/builders): address method Promise.prototype.then called on incompatible receiver error #2879

Merged
merged 1 commit into from
Oct 27, 2022

Conversation

alan-agius4
Copy link
Collaborator

@alan-agius4 alan-agius4 commented Oct 26, 2022

Prerendering does not work when using Node.js versions that contain the following commit nodejs/node#43728 due to loadesm is now using SafePromiseAll which is not handled properly by zone.js.

modules/builders/src/prerender/worker.ts Outdated Show resolved Hide resolved
modules/builders/src/prerender/worker.ts Outdated Show resolved Hide resolved
JiaLiPassion
JiaLiPassion previously approved these changes Oct 26, 2022
CaerusKaru
CaerusKaru previously approved these changes Oct 26, 2022
…led on incompatible receiver error

Prerendering does not work when using Node.js versions that contain the following commit nodejs/node#43728 due to `loadesm` is now using `SafePromiseAll` which is not handled properly by zone.js.
@alan-agius4 alan-agius4 added action: merge PR author is ready for this to merge target: minor target: minor This PR is targeted for the next minor release and removed action: review target: patch This PR is targeted for the next patch release labels Oct 27, 2022
@alan-agius4 alan-agius4 merged commit b7dbc25 into angular:main Oct 27, 2022
@alan-agius4 alan-agius4 deleted the zone-js-patch-prerender branch October 27, 2022 08:40
@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 Nov 27, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge PR author is ready for this to merge target: minor target: minor This PR is targeted for the next minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants