#78 PR comments

src/app/app-routing.module.ts

@@ -20,7 +20,7 @@ const routes: Routes = [
       { path: 'time-clock', component: TimeClockComponent },
       { path: 'time-entries', component: TimeEntriesComponent },
       { path: 'activities-management', component: ActivitiesManagementComponent },
-      { path: 'customer-managment', component: CustomerComponent },
+      { path: 'customers-management', component: CustomerComponent },
       { path: '', pathMatch: 'full', redirectTo: 'time-clock' },
     ],
   },

src/app/modules/customer-managment/components/customer-info/components/create-customer/create-customer.spec.ts

@@ -18,7 +18,7 @@ describe('InputCustomerComponent', () => {
     fixture.detectChanges();
   });
 
-  it('component should create', () => {
+  it('component should be created', () => {
     expect(component).toBeTruthy();
   });
 });

src/app/modules/customer-managment/components/customer-info/components/customer-list/customer-list.component.html

@@ -12,10 +12,12 @@
     >
       <td class="col-sm-9">{{ customer.name }}</td>
       <td class="col-sm-3 text-center">
-        <button (click)="activateCustomerForm()" type="button" class="btn btn-sm btn-secondary">
+        <button type="button" class="btn btn-sm btn-secondary">
           <i class="fa fa-pencil fa-xs"></i>
         </button>
-        <button type="button" class="btn btn-sm btn-danger ml-2"><i class="fas fa-trash-alt fa-xs"></i></button>
+        <button type="button" class="btn btn-sm btn-danger ml-2">
+          <i class="fas fa-trash-alt fa-xs"></i>
+        </button>
       </td>
     </tr>
   </tbody>

src/app/modules/customer-managment/components/customer-info/components/customer-list/customer-list.component.spec.ts

@@ -20,14 +20,8 @@ describe('CustomerTableListComponent', () => {
     fixture.detectChanges();
   });
 
-  it('component should create', () => {
+  it('component should be created', () => {
     expect(component).toBeTruthy();
   });
 
-  it('changeShowCustomerForm should listen form changes', () => {
-    component.showCustomerForm = true;
-    spyOn(component.changeShowCustomerForm, 'emit');
-    component.activateCustomerForm();
-    expect(component.changeShowCustomerForm.emit).toHaveBeenCalled();
-  });
 });

src/app/modules/customer-managment/components/customer-info/components/customer-list/customer-list.component.ts

@@ -1,4 +1,4 @@
-import { Component, Input, Output, EventEmitter } from '@angular/core';
+import { Component } from '@angular/core';
 import { ITEMS_PER_PAGE } from 'src/environments/environment';
 
 @Component({
@@ -29,13 +29,6 @@ export class CustomerListComponent {
     },
   ];
 
-  @Input() showCustomerForm;
-  @Output() changeShowCustomerForm = new EventEmitter<boolean>();
-
   constructor() {}
 
-  activateCustomerForm() {
-    this.showCustomerForm = true;
-    this.changeShowCustomerForm.emit(this.showCustomerForm);
-  }
 }

src/app/modules/customer-managment/components/projects-type/components/create-project-type/create-project-type.component.html

@@ -2,13 +2,9 @@
   <form style="width: 600px;">
     <div class="form-group">
       <input type="text" class="form-control form-control-sm" id="" aria-describedby="" placeholder="Name" />
-      <textarea
-        class="form-control form-control-sm mt-2"
-        id="descriptionTextArea"
-        rows="3"
-        placeholder="Description"
-      ></textarea>
-      <button type="submit" class="btn btn-sm btn-primary mb-2 mt-2">Save</button>
+      <textarea class="form-control form-control-sm mt-2" id="descriptionTextArea" rows="3"
+        placeholder="Description"></textarea>
+      <button type="submit" class="btn btn-sm btn-primary">Save</button>
       <button type="submit" class="btn btn-sm btn-secondary mb-2 ml-2 mt-2">Cancel</button>
     </div>
   </form>

src/app/modules/customer-managment/components/projects-type/components/create-project-type/create-project-type.component.spec.ts

@@ -18,7 +18,7 @@ describe('InputProjectTypeComponent', () => {
     fixture.detectChanges();
   });
 
-  it('component should create', () => {
+  it('component should be created', () => {
     expect(component).toBeTruthy();
   });
 });

src/app/modules/customer-managment/components/projects-type/components/project-type-list/project-type-list.component.spec.ts

@@ -20,7 +20,7 @@ describe('ProjectTypeTableListComponent', () => {
     fixture.detectChanges();
   });
 
-  it('component should create', () => {
+  it('component should be created', () => {
     expect(component).toBeTruthy();
   });
 });

src/app/modules/customer-managment/components/projects/components/create-project/create-project.component.spec.ts

@@ -18,7 +18,7 @@ describe('InputProjectComponent', () => {
     fixture.detectChanges();
   });
 
-  it('component should create', () => {
+  it('component should be created', () => {
     expect(component).toBeTruthy();
   });
 });

src/app/modules/customer-managment/components/projects/components/project-list/project-list.component.spec.ts

@@ -20,7 +20,7 @@ describe('ProjectTableListComponent', () => {
     fixture.detectChanges();
   });
 
-  it('component should create', () => {
+  it('component should be created', () => {
     expect(component).toBeTruthy();
   });
 });

src/app/modules/customer-managment/components/search/search.component.spec.ts

@@ -19,7 +19,7 @@ describe('SearchComponent', () => {
     fixture.detectChanges();
   });
 
-  it('should create', () => {
+  it('component should be created', () => {
     expect(component).toBeTruthy();
   });
 });

src/app/modules/customer-managment/components/search/search.component.ts

@@ -1,12 +1,11 @@
-import { Component, OnInit } from '@angular/core';
+import { Component } from '@angular/core';
 
 @Component({
   selector: 'app-search',
   templateUrl: './search.component.html',
   styleUrls: ['./search.component.scss'],
 })
-export class SearchComponent implements OnInit {
+export class SearchComponent {
   constructor() {}
 
-  ngOnInit(): void {}
 }

src/app/modules/customer-managment/pages/customer.component.html

@@ -6,10 +6,7 @@ <h4>Customers</h4>
     </div>
     <app-search></app-search>
   </div>
-  <app-customer-list
-    [showCustomerForm]="showCustomerForm"
-    (changeShowCustomerForm)="showCustomerForm = $event"
-  ></app-customer-list>
+  <app-customer-list></app-customer-list>
   <div class="row" *ngIf="showCustomerForm">
     <div class="col">
       <app-management-customer-projects></app-management-customer-projects>

src/app/modules/customer-managment/pages/customer.component.spec.ts

@@ -18,7 +18,7 @@ describe('CustomerComponent', () => {
     fixture.detectChanges();
   });
 
-  it('should create', () => {
+  it('should be created', () => {
     expect(component).toBeTruthy();
   });
 

src/app/modules/customer-managment/pages/customer.component.ts

@@ -1,17 +1,15 @@
-import { Component, OnInit } from '@angular/core';
+import { Component } from '@angular/core';
 
 @Component({
   selector: 'app-customer',
   templateUrl: './customer.component.html',
   styleUrls: ['./customer.component.scss'],
 })
-export class CustomerComponent implements OnInit {
+export class CustomerComponent {
   showCustomerForm = false;
 
   constructor() {}
 
-  ngOnInit(): void {}
-
   activateCustomerForm(): boolean {
     this.showCustomerForm = true;
     return this.showCustomerForm;

src/app/modules/shared/components/sidebar/sidebar.component.ts

@@ -19,9 +19,8 @@ export class SidebarComponent implements OnInit {
       { route: '/time-clock', icon: 'far fa-clock', text: 'Time Clock' },
       { route: '/time-entries', icon: 'far fa-list-alt', text: 'Time Entries' },
       { route: '/reports', icon: 'fas fa-chart-pie', text: 'Reports' },
-      // { route: '/project-management', icon: 'far fa-folder-open', text: 'Projects' },
       { route: '/activities-management', icon: 'far fa-file-alt', text: 'Activities' },
-      { route: '/customer-managment', icon: 'fas fa-users-cog', text: 'Customer' },
+      { route: '/customers-management', icon: 'fas fa-users-cog', text: 'Customers' },
     ];
   }
 }