Skip to content

Commit

Permalink
Fix precision component not displayed
Browse files Browse the repository at this point in the history
  • Loading branch information
daviserez committed Jul 12, 2023
1 parent da63f2e commit 47d6119
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion client/app/shared/components/address/address.component.ts
@@ -1,6 +1,11 @@
import {Component, ElementRef, Inject, Input, NgZone, OnChanges, OnInit, ViewChild} from '@angular/core';
import {UntypedFormControl, FormsModule, ReactiveFormsModule} from '@angular/forms';
import {NaturalIconDirective, NaturalQueryVariablesManager, NaturalSelectComponent} from '@ecodev/natural';
import {
NaturalIconDirective,
NaturalQueryVariablesManager,
NaturalSelectComponent,
NaturalSelectEnumComponent,
} from '@ecodev/natural';
// Format can remove following line, that is required to prevent warnings in console
import {merge} from 'lodash-es';
import {CountryService} from '../../../countries/services/country.service';
Expand Down Expand Up @@ -32,6 +37,7 @@ import {NgIf, NgClass, AsyncPipe} from '@angular/common';
NgClass,
ExtendedModule,
NaturalSelectComponent,
NaturalSelectEnumComponent,
CdkAccordionModule,
MatIconModule,
ReactiveFormsModule,
Expand Down

4 comments on commit 47d6119

@daviserez
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@PowerKiKi Hello, je me plonge à peine dans ces "Standalone Components", as-tu une idée de pourquoi la compilation est passée sans erreur ? Le composant "natural-select-enum" n'apparaissait tout simplement plus, sans indiquer aucune erreur... 🫨

@PowerKiKi
Copy link
Contributor

Choose a reason for hiding this comment

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

J'ai eu la même mauvaise surprise. Je m'attendais évidemment à ce que la compilation errors, ou au minimum warns, en cas d'import manquant. Je n'ai pas encore pris le temps de regarder, mais peut-être qu'il y a déjà des discussions à ce sujet chez Angular...

angular/angular#46351 semble en causer, mais ton cas est encore plus simple que celui exposé là-bas, et le fait que cela ne crash pas me laisse très songeur... 🤔

@daviserez
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oui, c'est un peu inquiétant 🤕 Merci pour ta réponse.

Espérons qu'ils corrigent ça prochainement, je garderai un œil sur le sujet également.

@PowerKiKi
Copy link
Contributor

Choose a reason for hiding this comment

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

Si tu veux follow, j'ai angular/angular#51064

Please sign in to comment.