Skip to content

Commit a4d5745

Browse files
committed
perf: TT-316 removal of ui-list-technologies toggle, technologies-report.guard and their dependants
1 parent ebb65c3 commit a4d5745

File tree

5 files changed

+1
-110
lines changed

5 files changed

+1
-110
lines changed

src/app/app-routing.module.ts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,6 @@ import { HomeComponent } from './modules/home/home.component';
1111
import { LoginComponent } from './modules/login/login.component';
1212
import { CustomerComponent } from './modules/customer-management/pages/customer.component';
1313
import { UsersComponent } from './modules/users/pages/users.component';
14-
import { TechnologyReportComponent } from './modules/technology-report/pages/technology-report.component';
15-
import { TechnologiesReportGuard } from './guards/technologies-report-guard/technologies-report.guard';
1614

1715
const routes: Routes = [
1816
{
@@ -26,7 +24,6 @@ const routes: Routes = [
2624
{ path: 'activities-management', component: ActivitiesManagementComponent },
2725
{ path: 'customers-management', canActivate: [AdminGuard], component: CustomerComponent },
2826
{ path: 'users', canActivate: [AdminGuard], component: UsersComponent },
29-
{ path: 'technology-report', canActivate: [AdminGuard, TechnologiesReportGuard], component: TechnologyReportComponent },
3027
{ path: '', pathMatch: 'full', redirectTo: 'time-clock' },
3128
],
3229
},

src/app/guards/technologies-report-guard/technologies-report.guard.spec.ts

Lines changed: 0 additions & 45 deletions
This file was deleted.

src/app/guards/technologies-report-guard/technologies-report.guard.ts

Lines changed: 0 additions & 28 deletions
This file was deleted.

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

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -86,21 +86,4 @@ describe('SidebarComponent', () => {
8686
});
8787
});
8888

89-
it('List Technologies item is added when feature flag "ui-list-technologies" is enabled for user', () => {
90-
spyOn(featureManagerServiceStubInjected, 'isToggleEnabledForUser').and.returnValue(of(true));
91-
const itemsSidebar = [];
92-
93-
component.toggleListTechnologies(itemsSidebar);
94-
95-
expect(itemsSidebar.length).toBe(1);
96-
});
97-
98-
it('List Technologies item is not added when feature flag "ui-list-technologies" is disabled for user', () => {
99-
spyOn(featureManagerServiceStubInjected, 'isToggleEnabledForUser').and.returnValue(of(false));
100-
const itemsSidebar = [];
101-
102-
component.toggleListTechnologies(itemsSidebar);
103-
104-
expect(itemsSidebar.length).toBe(0);
105-
});
10689
});

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

Lines changed: 1 addition & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ export class SidebarComponent implements OnInit, OnDestroy {
2929
ngOnInit(): void {
3030
this.toggleSideBar();
3131
this.sidebarItems$ = this.getSidebarItems().subscribe();
32-
this.toggleListTechnologies(this.itemsSidebar);
32+
// this.toggleListTechnologies(this.itemsSidebar);
3333
this.highlightMenuOption(this.router.routerState.snapshot.url);
3434
this.navStart.subscribe((evt) => {
3535
this.highlightMenuOption(evt.url);
@@ -68,22 +68,6 @@ export class SidebarComponent implements OnInit, OnDestroy {
6868
);
6969
}
7070

71-
toggleListTechnologies(itemsSidebar: ItemSidebar[]) {
72-
this.featureManagerService
73-
.isToggleEnabledForUser('ui-list-technologies')
74-
.subscribe((enabled) => {
75-
if (enabled === true) {
76-
const listTechnologiesItem = {
77-
route: '/technology-report',
78-
icon: 'fas fa-user',
79-
text: 'Technology Report',
80-
active: false,
81-
};
82-
itemsSidebar.push(listTechnologiesItem);
83-
}
84-
});
85-
}
86-
8771
highlightMenuOption(route) {
8872
this.itemsSidebar.map((item) => (item.active = false));
8973
this.itemsSidebar.filter((item) => item.route === route).map((item) => (item.active = true));

0 commit comments

Comments
 (0)