• Supprimer public listEstablishment;
• Renommer listEstablishment en establishments et typé correctement
• Supprimer public categorie = this.service.result; et récupérer la catégorie a partir de la route (voir issue #32, #34 )
• Supprimer la méthode etaDetail non utilisé
• Factoriser les deux <div class=".container"> en un, en utilisant une fonction .filter en TS ce qui permet de ne plus avoir besoin de estaDefault et estaCategorie
• Utiliser CardComponent voir issue #50

• Créer une méthode isLogin qui renvoie true (si le token existe) ou false (si le token n'existe pas).
• Utiliser la methode isLogin dans HeaderComponent (voir issue #35) et dans AuthGuard (voir issue #45)

• Supprimer public profil: Profile; et utiliser .patchValue directement avec la réponse de l'API.
• Renommer myForm en profile

Template HTML de CategorieComponent :

      <div class="card m-4">
        <div class="hovereffect">
          <img class="img-responsive border border-dark col" src="assets/img/{{ categorie }}.jpg" alt="" />
          <div class="overlay">
            <h2 class="font-weight-bolder">{{ categorie }}</h2>
            <a class="info" (click)="categorieRest(categorie)" routerLink="profiles/establishments">Show restaurants</a>
          </div>
        </div>

Template HTML de ListEstablishmentComponent :

      <div class="card m-4">
        <div class="hovereffect">
          <img class="img-responsive border border-dark col" src="{{ establishment.medias[0]?.url }}" />

          <div class="overlay">
            <h2 class="font-weight-bolder">{{ establishment.name }}</h2>
            <a class="info btn btn-danger" [routerLink]="['/establishments/details', establishment._id]">See
              restaurant</a>
          </div>
        </div>
      </div>

Le code HMLT est très similaire donc création d'un component !!
Créer un component CardComponent qui mutualise les templates ci dessus.
Ajouter un @Input pour l'url de l'image, un @Input pour le nom, un @Input pour l'id de l’établissement. BONUS utiliser <ng-content> pour le contenu de <h2>.
Utiliser ce component dans ListEstablishmentComponent et CategorieComponent

• Supprimer public messageForm: Message; et utiliser .patchValue directement avec la réponse de l'API.
• Renommer myForm en message
• Supprimer les propriétés id: string; et messageService: Message;

Pensez a supprimer les console.log, le code mort (si il y en a), les variables, propriétés inutilisé ainsi que les imports et services injecté inutilisé

• ajouter un paramètre id a la route profiles/establishments

environment-specific-variables

• Un peu longue la ligne html de 370 caractères...
• Supprimer public listBooking;
• Renommer public tableLists = []; en bookings et typé correctement

• Supprimer la propriété loading = false
• Supprimer la propriété returnUrl
  A la place de

this.submitted = true;
    if (this.loginForm.invalid) {
      return;
    }

Vous devriez utilisé la variable de template de ngForm https://angular.io/api/forms/NgForm#template-variable-references ce qui vous permet d'accéder à l'état de votre formulaire directement depuis le html :

<form #f="ngForm" (ngSubmit)="f.valid && onSubmit()" [formGroup]='loginForm'>

• Utiliser 'this.router.navigate(['']);' au lieu de 'this.router.navigate(['#']);'

• Supprimer public id; et public result;
• Supprimer la méthode getEstablishmentBySort(), les valeurs doivent être passé en paramètre des routes.

• Supprimer config: NgbCarouselConfig si vous ne l'utilisez pas
• Si un restaurant n'a pas d'horaire renseigné l'erreur _co.time.open is undefined est affichée dans la console. Vous devez tester si le restaurant possède bien des horaires avant de les afficher (ngIf ou le safe navigation operator)

• Supprimer la propriété bookingForm
• Renommer la propriété userForm en booking
• Supprimer la propriété title
• Supprimer la propriété cacheId
• NgOnInit : stocker params.get('id') dans la propriété this.id
• Submit : supprimer le subscribe this.activetedroute.paramMap et utilser this.id dans la condition
• Il y a une erreur dans la console a l'affichage du formulaire "Cannot find control with name: 'address'". Le FormGroup adress a comme FormGroup parent owner et non userForm vérifié le code HTML.

Utiliser la directive routerLink sur <a class="navbar-brand"> au lieu de href pour évité de recharger toute la page

A rendre fonctionnel

• ngOnInit : ajouter une propriete this.id et l'utiliser pour stocker l'id récupérer avec params.get('id');
• onSubmit : supprimer l'appel this.activatedRoute.paramMap.subscribe et utiliser directement la propriete this.id dans la condition
• Il y a une erreur a l'initialisation du formulaire dans la console "Cannot find control with name: 'message'".

<textarea
          class="form-control"
          formControlName="message"
          placeholder="Describe your establishment"
></textarea>

Il n'y a pas de FormControl message dans le FormGroup this.establishmentForm.

• Un peu longue la ligne html de 320 caractères...
• Supprimer public listMessage;
• Renommer public tableLists = []; en message et type correctement

Appeler la méthode isLogin au lieu de faire appel au localStorage (voir issue #46)

• Supprimer public listEstablishment; utiliser a la place public establishments: []; et typé la correctement.
• Supprimer public listMessage; utiliser a la place public messages: []; et typé la correctement.
• NgOnInit paramètre etam comme le magasin ?

Lorsque l'on navigue de la vue "restaurant par categorie" (pizza > show restaurant) on a bien deux restaurants d’affiché mais si on clique sur "all restaurants" rien ne se passe. Sûrement liée
aux issues #38, #32, #34

• Supprimer la méthode CategorieRest(), pourquoi Rest ? (voir issue #33 )
• Passer l'id de la categorie en parametre a la directive routerLink (voir issue #32 )
• Utiliser CardComponent voir issue #50