You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
No subscription cleanup: Zero uses of ngOnDestroy, DestroyRef, or takeUntilDestroyed(). All components use bare .subscribe() without teardown. Repeated navigation accumulates handlers and causes memory leaks.
No lazy loading: All feature routes eagerly import components at the top of app.routes.ts. No loadComponent / loadChildren. Increases initial bundle size unnecessarily.
Acceptance criteria
Subscription management
Add DestroyRef + takeUntilDestroyed() to all component subscriptions (Angular 16+ pattern)
Or use async pipe in templates where possible (preferred)
Problem
No subscription cleanup: Zero uses of
ngOnDestroy,DestroyRef, ortakeUntilDestroyed(). All components use bare.subscribe()without teardown. Repeated navigation accumulates handlers and causes memory leaks.No lazy loading: All feature routes eagerly import components at the top of
app.routes.ts. NoloadComponent/loadChildren. Increases initial bundle size unnecessarily.Acceptance criteria
Subscription management
DestroyRef+takeUntilDestroyed()to all component subscriptions (Angular 16+ pattern)asyncpipe in templates where possible (preferred)Lazy loading
loadComponent:ng buildoutput