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

refactor: import zone.js/testing instead of zone-testing-bundle #2163

Merged
merged 1 commit into from
Sep 13, 2023

Conversation

JiaLiPassion
Copy link
Contributor

Close #2162

From zone.js 0.13.2, the original
zone.js/bundles/zone-testing-bundle.js is not easily import and will eventually removed from zone.js 0.14.0.

The proper way to import zone.js and zone.js/testing is

import 'zone.js';
import 'zone.js/testing';

So this PR use this way to import zone.js related bundles.

Close thymikee#2162

From zone.js 0.13.2, the original
`zone.js/bundles/zone-testing-bundle.js` is not easily import and will
eventually removed from zone.js 0.14.0.

The proper way to import zone.js and zone.js/testing is

```
import 'zone.js';
import 'zone.js/testing';
```

So this PR use this way to import zone.js related bundles.
Copy link

@alan-agius4 alan-agius4 left a comment

Choose a reason for hiding this comment

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

LGTM

@ahnpnl ahnpnl merged commit e47ae67 into thymikee:main Sep 13, 2023
10 checks passed
@ocombe
Copy link

ocombe commented Sep 14, 2023

@ahnpnl could you make a new release with that fix please?

@ghiscoding
Copy link

@ahnpnl we really need a patch release for this since as pointed out by this comment, the temp fix that was introduced in Angular zone.js was reverted in zone.js@0.14.0 and now the issue #2162 came back and is again a problem

zone.js version 0.13.3 has been just released which should get around this issue.
Deep imports will eventually no longer be allowed from zone.js version 0.14.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: latest zone.js v0.13.2 is causing import error
5 participants