IzabelaJas / todo

0 stars 0 forks source link

Zbiorczy angular - related issue #11

Open wkulczi opened 1 year ago

wkulczi commented 1 year ago
  1. Muszę pochwalić pomysł z budową pliku app.component.html. Dzięki temu ułożeniu tabela tasks-list zawsze jest widoczna a do góry może być wyświetlona dowolna ścieżka. Wiąże się to jednak z pewnym downsidem - komponenty nie wiedzą o sobie nawzajem i musiałaś umieścić location.reload() żeby dane były ponownie załadowane z backendu. Rozwiązań jest kilka. Umieszczam jedno z nich jako gist: https://gist.github.com/wkulczi/26a0e110ddcdb359d9112a618ba24aeb.
  2. Jeśli chcesz, by pod ścieżką '' nie było żadnego komponentu to możesz po prostu tę ścieżkę pominąć, wtedy pod <router-outlet> nie podstawi się nic i widoczny będzie tylko komponent tasks-list. tl;dr - coś takiego też działa:
    //app-routing.module.ts
    const routes: Routes = [
    {path: 'update-task/:id', component: UpdateTaskComponent},
    {path: 'create-task', component: CreateTaskComponent},
    //wyrzuciłem {path:'', component: TopComponent}
    ];
  3. Dobra robota ze wstrzykiwaniem serwisów do konstruktora i reużywaniem pojedynczej instancji za pomocą providedIn:'root'
  4. Fajnie, że używasz hooków angularowych, OnInit i OnDestroy to chyba najpopularniejsze z nich i bardzo przydatne. Pamiętaj jednak, ze gdy chcesz użyć funkcji ngOnInit to klasa w której realizujesz tę funkcję koniecznie musi mieć implements OnInit. Bez tego funkcja chyba nie rusza
wkulczi commented 1 year ago

Also - [(ngModel)] niestety jest już passé image https://angular.io/api/forms/FormControlName#use-with-ngmodel-is-deprecated

Zamiast tego używa się FormControl ale to jest cała nowa historia i to niemała