Skip to content
This repository has been archived by the owner on Aug 7, 2021. It is now read-only.

fix: import the workaroundResolve funciton based on the Angular version (it was moved from compiler_host to utils in Angular 8) #904

Merged
merged 3 commits into from
May 29, 2019

Conversation

DimitarTachev
Copy link
Contributor

@DimitarTachev DimitarTachev commented May 21, 2019

PR Checklist

What is the current behavior?

A workaroundResolve is not a function error is thrown with Angular 8 (@ngtools/webpack@8.0.0)

What is the new behavior?

The Angular apps are properly built with Angular 8.

Related to: NativeScript/nativescript-angular#1833 NativeScript/nativescript-angular#1831

@cla-bot cla-bot bot added the cla: yes label May 21, 2019
@SvetoslavTsenov
Copy link
Contributor

SvetoslavTsenov commented May 21, 2019

This PR is tested with:
NativeScript/nativescript-angular#1831
and everything works fine

@SvetoslavTsenov
Copy link
Contributor

test --ignore api23 ios11 branch_ngapps#amiorkov/angular8-rc branch_nativescript_angular#amiorkov/angular8-rc

…on (it was moved from compiler_host to utils in Angular 8)
@SvetoslavTsenov
Copy link
Contributor

test --ignore api23 ios11 branch_ngapps#amiorkov/angular8-rc branch_nativescript_angular#amiorkov/angular8-rc

@VladimirAmiorkov VladimirAmiorkov requested a review from Fatme May 27, 2019 11:49
@VladimirAmiorkov
Copy link
Contributor

test --ignore api23 ios11 branch_ngapps#amiorkov/angular8-rc branch_nativescript_angular#amiorkov/angular8-rc

@VladimirAmiorkov
Copy link
Contributor

test --ignore api23 ios11 branch_ngapps#amiorkov/angular8-rc branch_nativescript_angular#amiorkov/angular8-rc

@DimitarTachev DimitarTachev merged commit adc36a9 into master May 29, 2019
@DimitarTachev DimitarTachev deleted the feature/angular8 branch May 29, 2019 11:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants