feat(Grade by step): Display component summary views

angular.json

@@ -97,7 +97,7 @@
               "baseHref": "/"
             },
             "development": {
-              "optimization": false,
+              "optimization": true,
               "outputHashing": "all",
               "sourceMap": true,
               "namedChunks": true,

src/app/app-routing.module.ts

@@ -78,7 +78,7 @@ export class XhrInterceptor implements HttpInterceptor {
 }
 
 @NgModule({
-  imports: [RouterModule.forRoot(routes, {}), FormsModule],
+  imports: [RouterModule.forRoot(routes, { paramsInheritanceStrategy: 'always' }), FormsModule],
   exports: [RouterModule],
   providers: [{ provide: HTTP_INTERCEPTORS, useClass: XhrInterceptor, multi: true }]
 })

src/app/services/milestone/milestoneService.spec.ts

@@ -27,8 +27,8 @@ let teacherDataService: TeacherDataService;
 describe('MilestoneService', () => {
   beforeEach(() => {
     TestBed.configureTestingModule({
-    imports: [MatDialogModule, StudentTeacherCommonServicesModule],
-    providers: [
+      imports: [MatDialogModule, StudentTeacherCommonServicesModule],
+      providers: [
         ClassroomStatusService,
         CopyNodesService,
         MilestoneService,
@@ -38,8 +38,8 @@ describe('MilestoneService', () => {
         TeacherWebSocketService,
         provideHttpClient(withInterceptorsFromDi()),
         provideHttpClientTesting()
-    ]
-});
+      ]
+    });
     service = TestBed.inject(MilestoneService);
     achievementService = TestBed.inject(AchievementService);
     configService = TestBed.inject(ConfigService);
@@ -48,7 +48,6 @@ describe('MilestoneService', () => {
     teacherDataService = TestBed.inject(TeacherDataService);
   });
   getProjectMilestones();
-  getMilestoneReportByNodeId();
   getProjectMilestoneStatus();
   insertMilestoneItems();
   insertMilestoneCompletion();
@@ -85,63 +84,6 @@ function getProjectMilestones() {
   });
 }
 
-function getMilestoneReportByNodeId() {
-  describe('getMilestoneReportByNodeId()', () => {
-    it('should get project milestone report by node id when there is none', () => {
-      const achievements = {
-        isEnabled: true,
-        items: [
-          {
-            id: 'milestone1',
-            type: 'milestone',
-            report: {
-              templates: [
-                {
-                  satisfyCriteria: [
-                    {
-                      nodeId: 'node1',
-                      componentId: 'component1'
-                    }
-                  ]
-                }
-              ]
-            }
-          }
-        ]
-      };
-      spyOn(projectService, 'getAchievements').and.returnValue(achievements);
-      const milestoneReport = service.getMilestoneReportByNodeId('node2');
-      expect(milestoneReport).toBeNull();
-    });
-    it('should get project milestone report by node id when there is one', () => {
-      const achievements = {
-        isEnabled: true,
-        items: [
-          {
-            id: 'milestone1',
-            type: 'milestone',
-            report: {
-              templates: [
-                {
-                  satisfyCriteria: [
-                    {
-                      nodeId: 'node1',
-                      componentId: 'component1'
-                    }
-                  ]
-                }
-              ]
-            }
-          }
-        ]
-      };
-      spyOn(projectService, 'getAchievements').and.returnValue(achievements);
-      const milestoneReport = service.getMilestoneReportByNodeId('node1');
-      expect(milestoneReport).toBeDefined();
-    });
-  });
-}
-
 function getProjectMilestoneStatus() {
   describe('getProjectMilestoneStatus()', () => {
     it('should get project milestone status', () => {

src/app/services/workgroup.service.ts

@@ -5,11 +5,14 @@ import { ConfigService } from '../../assets/wise5/services/configService';
 
 @Injectable()
 export class WorkgroupService {
-  constructor(private ConfigService: ConfigService, private http: HttpClient) {}
+  constructor(
+    private ConfigService: ConfigService,
+    private http: HttpClient
+  ) {}
   getWorkgroupsInPeriod(periodId: number): Map<number, any> {
     const workgroups = new Map();
     for (const workgroup of this.getWorkgroupsSortedById()) {
-      if (workgroup.periodId === periodId && workgroup.workgroupId != null) {
+      if (periodId === -1 || (workgroup.periodId === periodId && workgroup.workgroupId != null)) {
         workgroup.displayNames = this.ConfigService.getDisplayUsernamesByWorkgroupId(
           workgroup.workgroupId
         );