Skip to content

Commit 19a8734

Browse files
committed
Revert "feat: TT-337 "Add new activity button" implementation (#754)"
This reverts commit 2461f61.
1 parent 7ca1f55 commit 19a8734

File tree

6 files changed

+13
-62
lines changed

6 files changed

+13
-62
lines changed

src/app/modules/activities-management/components/activity-list/activity-list.component.ts

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { Component, EventEmitter, Input, OnInit, Output } from '@angular/core';
1+
import { Component, OnInit } from '@angular/core';
22
import { select, Store } from '@ngrx/store';
33
import { Observable } from 'rxjs';
44
import { delay, map } from 'rxjs/operators';
@@ -14,9 +14,6 @@ import { ActivityState } from './../../store/activity-management.reducers';
1414
styleUrls: ['./activity-list.component.scss'],
1515
})
1616
export class ActivityListComponent implements OnInit {
17-
@Input() showActivityForm: boolean;
18-
@Output() changeValueShowActivityForm = new EventEmitter<boolean>();
19-
2017
constructor(private store: Store<ActivityState>) {
2118
this.isLoading$ = store.pipe(delay(0), select(getIsLoading));
2219
}
@@ -69,8 +66,6 @@ export class ActivityListComponent implements OnInit {
6966

7067
updateActivity(activityId: string): void {
7168
this.store.dispatch(new SetActivityToEdit(activityId));
72-
this.showActivityForm = true;
73-
this.changeValueShowActivityForm.emit(this.showActivityForm);
7469
}
7570

7671
unarchiveActivity(): void {

src/app/modules/activities-management/components/create-activity/create-activity.component.html

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,7 @@
1+
<h4>Activities</h4>
2+
<hr/>
13
<div>
4+
25
<form [formGroup]="activityForm" (ngSubmit)="onSubmit(activityForm.value)">
36
<div class="form-group">
47
<label>Name:</label>
@@ -18,7 +21,7 @@
1821

1922
<div class="form-group" style="text-align: right;">
2023
<button class="btn btn-primary" type="submit" [disabled]="!activityForm.valid">Save</button>
21-
<button class="btn btn-secondary mb-2 ml-2 mt-2" type="button" (click)="cancelButton()">
24+
<button class="btn btn-secondary mb-2 ml-2 mt-2" type="reset" [hidden]="!activityToEdit" (click)="cancelButton()">
2225
Cancel
2326
</button>
2427
</div>

src/app/modules/activities-management/components/create-activity/create-activity.component.ts

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
import { FormBuilder, Validators, FormGroup } from '@angular/forms';
2-
import { Component, EventEmitter, OnInit, Output } from '@angular/core';
2+
import { Component, OnInit } from '@angular/core';
33
import { Store, select } from '@ngrx/store';
4+
45
import { Activity } from '../../../shared/models';
56
import { ActivityState } from './../../store/activity-management.reducers';
67
import { CreateActivity, UpdateActivity, getActivityById, ResetActivityToEdit } from '../../store';
@@ -11,9 +12,9 @@ import { CreateActivity, UpdateActivity, getActivityById, ResetActivityToEdit }
1112
styleUrls: ['./create-activity.component.scss'],
1213
})
1314
export class CreateActivityComponent implements OnInit {
14-
@Output() closeActivityForm = new EventEmitter<boolean>();
1515
activityForm: FormGroup;
1616
activityToEdit: Activity;
17+
1718
constructor(private formBuilder: FormBuilder, private store: Store<ActivityState>) {
1819
this.activityForm = this.formBuilder.group({
1920
name: ['', Validators.required],
@@ -59,12 +60,9 @@ export class CreateActivityComponent implements OnInit {
5960
this.store.dispatch(new CreateActivity(activityData));
6061
this.activityForm.get('description').setValue('');
6162
}
62-
this.closeActivityForm.emit(false);
6363
}
6464

6565
cancelButton() {
66-
this.activityForm.reset();
6766
this.store.dispatch(new ResetActivityToEdit());
68-
this.closeActivityForm.emit(false);
6967
}
7068
}
Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,4 @@
11
<div class="col-12 col-md-9 px-0">
2-
<div class="row">
3-
<div style="padding: 15px;">
4-
<button (click)="activateActivityForm()" class="btn btn-primary">Add new Activity</button>
5-
</div>
6-
</div>
7-
<div *ngIf="showActivityForm">
8-
<app-create-activity
9-
(closeActivityForm)="closeFormActivity($event)"></app-create-activity>
10-
</div>
11-
<app-activity-list
12-
(changeValueShowActivityForm)="showActivityForm = $event"></app-activity-list>
2+
<app-create-activity></app-create-activity>
3+
<app-activity-list></app-activity-list>
134
</div>
Lines changed: 1 addition & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -1,46 +1,24 @@
11
import { waitForAsync, TestBed, ComponentFixture } from '@angular/core/testing';
2-
import { MockStore, provideMockStore } from '@ngrx/store/testing';
32
import { ActivitiesManagementComponent } from './activities-management.component';
4-
import { SetActivityToEdit } from '../store';
5-
import { Activity } from '../../shared/models';
63

74
describe('ActivitiesManagementComponent', () => {
85
let component: ActivitiesManagementComponent;
96
let fixture: ComponentFixture<ActivitiesManagementComponent>;
10-
let store: MockStore<Activity>;
117

128
beforeEach(waitForAsync(() => {
139
TestBed.configureTestingModule({
1410
imports: [],
15-
declarations: [ActivitiesManagementComponent],
16-
providers: [provideMockStore({ initialState: {} })],
11+
declarations: [ActivitiesManagementComponent]
1712
}).compileComponents();
1813
}));
1914

2015
beforeEach(() => {
2116
fixture = TestBed.createComponent(ActivitiesManagementComponent);
2217
component = fixture.componentInstance;
2318
fixture.detectChanges();
24-
store = TestBed.inject(MockStore);
2519
});
2620

2721
it('should create the component', () => {
2822
expect(component).toBeTruthy();
2923
});
30-
31-
it('should dispatch an action on activateActivityForm', () => {
32-
spyOn(store, 'dispatch');
33-
34-
component.activateActivityForm();
35-
36-
expect(store.dispatch).toHaveBeenCalledWith(new SetActivityToEdit(null));
37-
});
38-
39-
it('should call closeActivityForm function', () => {
40-
component.closeFormActivity(false);
41-
42-
expect(component.showActivityForm).toBe(false);
43-
});
44-
45-
4624
});
Lines changed: 2 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -1,24 +1,10 @@
1-
import { Component, EventEmitter, Output } from '@angular/core';
2-
import { Store } from '@ngrx/store';
3-
import { Activity } from '../../shared/models';
4-
import { SetActivityToEdit } from '../store';
1+
import { Component } from '@angular/core';
52

63
@Component({
74
selector: 'app-activities-management',
85
templateUrl: './activities-management.component.html',
96
styleUrls: ['./activities-management.component.scss'],
107
})
118
export class ActivitiesManagementComponent {
12-
@Output() closeActivityForm = new EventEmitter<boolean>();
13-
showActivityForm = false;
14-
constructor(private store: Store<Activity>) {}
15-
16-
activateActivityForm() {
17-
this.store.dispatch(new SetActivityToEdit(null));
18-
this.showActivityForm = true;
19-
}
20-
21-
closeFormActivity(event) {
22-
this.showActivityForm = event;
23-
}
9+
constructor() {}
2410
}

0 commit comments

Comments
 (0)