devsecopsmaturitymodel / DevSecOps-MaturityModel

GNU General Public License v3.0
459 stars 267 forks source link

Implement ESLint Check and Update Codebase #285

Open nishant2253 opened 3 months ago

nishant2253 commented 3 months ago

This pull request, tagged as #283, focuses on augmenting the DSOMM (DevSecOps Maturity Model) application's functionality by incorporating Title Tabs to the window and implementing necessary code changes with respect to this issue #268.

Changes made in app.component.ts:

Implemented functionality to dynamically update the document title based on the current route.
Utilized the Title service from @angular/platform-browser to set the document title.
Introduced a method updateTitle() to facilitate updating the document title with the corresponding page title.
Subscribed to router events to detect navigation end events and subsequently update the title accordingly.
Established a constant, DEFAULT_TITLE, to define the default title and improved maintainability by refactoring the code to use this constant instead of a hard-coded string.

Changes made in app.component.spec.ts:

Updated tests to accommodate the modifications made in the app.component.ts file.
Added tests to ensure the accurate updating of the document title when navigating to different routes.

Changes made in app-routing.module.ts:

Adjusted route configurations to incorporate a data property for each route, specifying the title of the respective page.

Changes made in circular-heatmap.component.ts: & teams.component.ts

Added specific comments to disable ESLint rules, such as @angular-eslint/use-lifecycle-interface and @angular-eslint/ component-class-suffix. 
Comments strategically placed to suppress errors flagged by ESLint for improved code development.