angular-toolkit icon indicating copy to clipboard operation
angular-toolkit copied to clipboard

component generator does not respect --changeDetection Flag

Open distante opened this issue 6 years ago • 12 comments

Steps to Reproduce:

ionic start
npm install @angular/[email protected]
ng g component Test --changeDetection=OnPush --entryComponent

Output: Generated component:

import { Component, OnInit } from '@angular/core';

@Component({
  selector: 'app-training-options',
  templateUrl: './training-options.component.html',
  styleUrls: ['./training-options.component.scss'],
})
export class TrainingOptionsComponent implements OnInit {

  constructor() { }

  ngOnInit() {}

}

My ionic info:

Ionic:

   ionic (Ionic CLI)             : 4.10.3 (C:\Users\distante\AppData\Roaming\npm\node_modules\ionic)
   Ionic Framework               : @ionic/angular 4.0.2
   @angular-devkit/build-angular : 0.12.4
   @angular-devkit/schematics    : 7.3.4
   @angular/cli                  : 7.3.4
   @ionic/angular-toolkit        : 1.4.0

System:

   NodeJS : v10.14.1 (C:\Program Files\nodejs\node.exe)
   npm    : 6.5.0
   OS     : Windows 10

Other Information: Using a Pure angular Project ng g generates the correct component.

distante avatar Mar 06 '19 15:03 distante

Thanks for the issue! We'll get this fixed it asap

mhartington avatar Mar 06 '19 20:03 mhartington

@mhartington Looks like this is erroring in 2.0.0? https://github.com/ionic-team/angular-toolkit/issues/177

imhoffd avatar Aug 30 '19 16:08 imhoffd

Seems like your schematics for components also doesnt support the changeDetection default in angular.json just want to make sure that gets fixed as well since they are related.

Lindsor avatar May 06 '20 00:05 Lindsor

Hi @dwieeb Will we have this soon or?

Sampath-Lokuge avatar Jun 08 '20 19:06 Sampath-Lokuge

Also looking for a fix.

GaryLoop avatar Aug 28 '20 04:08 GaryLoop

Is there any workaround until this is fixed?

Edit: Simply remove "defaultCollection": "@ionic/angular-toolkit" from angular.json, if you don't need any of ionics collection

bagbag avatar Jul 21 '21 18:07 bagbag

Hi, i am too using Ionic5 and using the workaround of putting this.changeRef.detectChanges() on my backend calls.... never had to do that in Ionic3 (which i used until a little time ago).

I also tried to remove "defaultCollection" from angular.json, but that does not change anything... i am still forced to use detectChanges...

P.S: this happens more in the iOS version of my app; under 'ionic serve' i do not observe this behaviour.... (and all works as in ionic3, without detectchanges instruction

this is my ionic info:


Ionic:

   Ionic CLI                     : 5.4.16 (/usr/local/lib/node_modules/ionic)
   Ionic Framework               : @ionic/angular 5.9.1
   @angular-devkit/build-angular : 12.1.4
   @angular-devkit/schematics    : 12.1.4
   @angular/cli                  : 12.1.4
   @ionic/angular-toolkit        : 4.0.0

Cordova:

   Cordova CLI       : 10.0.0 ([email protected])
   Cordova Platforms : ios 6.2.0
   Cordova Plugins   : cordova-plugin-ionic-keyboard 2.2.0, cordova-plugin-ionic-webview 5.0.0, (and 6 other plugins)

Utility:

   cordova-res : not installed
   native-run  : not installed

System:

   Android SDK Tools : 26.1.1 (/Users/demetriomortelliti/Library/Android/sdk)
   ios-sim           : 8.0.2
   NodeJS            : v14.15.5 (/usr/local/bin/node)
   npm               : 6.14.11
   OS                : macOS Big Sur
   Xcode             : Xcode 13.0 Build version 13A233

demym avatar Nov 29 '21 07:11 demym

Any updates on this?

pseudooriginal avatar Jun 15 '22 14:06 pseudooriginal

I use angular cli 18 rc4, ng g component test-push --changeDetection=OnPush generate same prob : changeDetection is not recognize

evan-boissonnot avatar Apr 26 '24 08:04 evan-boissonnot