Merge pull request #3125 from codecrafters-io/arpan/cc-1941-deprecated-courses-should-not-be-visible-on-tracks-page

Arpan-206 · web-flow · commit c081dd231ec9 · 2025-09-25T18:03:42.000+05:30
Filter out deprecated courses for staff users in TrackController.
diff --git a/app/controllers/track.ts b/app/controllers/track.ts
@@ -10,10 +10,6 @@ export default class TrackController extends Controller {
   @service declare authenticator: AuthenticatorService;
 
   get courses(): CourseModel[] {
-    if (this.authenticator.currentUser && this.authenticator.currentUser.isStaff) {
-      return this.model.courses;
-    }
-
     return this.model.courses.rejectBy('releaseStatusIsAlpha').rejectBy('releaseStatusIsDeprecated').rejectBy('visibilityIsPrivate');
   }
 

Original file line number	Diff line number	Diff line change
`@@ -10,10 +10,6 @@ export default class TrackController extends Controller {`
`10`	`10`	`@service declare authenticator: AuthenticatorService;`
`11`	`11`
`12`	`12`	`get courses(): CourseModel[] {`
`13`		`- if (this.authenticator.currentUser && this.authenticator.currentUser.isStaff) {`
`14`		`- return this.model.courses;`
`15`		`- }`
`16`		`-`
`17`	`13`	`return this.model.courses.rejectBy('releaseStatusIsAlpha').rejectBy('releaseStatusIsDeprecated').rejectBy('visibilityIsPrivate');`
`18`	`14`	`}`
`19`	`15`