chore: remove dead code and tune frontend chunks
Remove unused production symbols, starter assets, and stale browser tooling; reduce frontend chunk fragmentation.
This commit is contained in:
@@ -63,7 +63,7 @@ describe('AttendanceController — DingTalk import scope', () => {
|
||||
attendanceService.getTeacherClassDingUserIds.mockResolvedValue(['ding-1', 'ding-2']);
|
||||
|
||||
await controller.importFromDingTalk(
|
||||
{ start: '2026-07-01', end: '2026-07-02', classId: 8, autoMatch: true },
|
||||
{ start: '2026-07-01', end: '2026-07-02', classId: 8 },
|
||||
{ user: { id: 21, username: 'teacher', permissions: [], isSuperAdmin: false } } as never,
|
||||
);
|
||||
|
||||
@@ -79,7 +79,7 @@ describe('AttendanceController — DingTalk import scope', () => {
|
||||
it('does not allow a teacher to supply arbitrary DingTalk user IDs', async () => {
|
||||
await expect(
|
||||
controller.importFromDingTalk(
|
||||
{ start: '2026-07-01', end: '2026-07-02', users: 'someone-else', autoMatch: true },
|
||||
{ start: '2026-07-01', end: '2026-07-02', users: 'someone-else' },
|
||||
{ user: { id: 21, username: 'teacher', permissions: [], isSuperAdmin: false } } as never,
|
||||
),
|
||||
).rejects.toBeInstanceOf(ForbiddenException);
|
||||
@@ -89,7 +89,7 @@ describe('AttendanceController — DingTalk import scope', () => {
|
||||
|
||||
it('requires teachers to select one of their classes', async () => {
|
||||
await expect(
|
||||
controller.importFromDingTalk({ start: '2026-07-01', end: '2026-07-02', autoMatch: true }, {
|
||||
controller.importFromDingTalk({ start: '2026-07-01', end: '2026-07-02' }, {
|
||||
user: { id: 21, username: 'teacher', permissions: [], isSuperAdmin: false },
|
||||
} as never),
|
||||
).rejects.toBeInstanceOf(BadRequestException);
|
||||
@@ -106,11 +106,11 @@ describe('AttendanceController — DingTalk import scope', () => {
|
||||
expect(attendanceService.getImportableClasses).toHaveBeenCalledWith(21, false);
|
||||
});
|
||||
|
||||
it('always auto-matches class-scoped imports even if an old client sends autoMatch=false', async () => {
|
||||
it('always auto-matches class-scoped imports', async () => {
|
||||
attendanceService.getTeacherClassDingUserIds.mockResolvedValue(['ding-1']);
|
||||
|
||||
await controller.importFromDingTalk(
|
||||
{ start: '2026-07-01', end: '2026-07-02', classId: 8, autoMatch: false },
|
||||
{ start: '2026-07-01', end: '2026-07-02', classId: 8 },
|
||||
{ user: { id: 21, username: 'teacher', permissions: [], isSuperAdmin: false } } as never,
|
||||
);
|
||||
|
||||
@@ -137,7 +137,7 @@ describe('AttendanceController — DingTalk import scope', () => {
|
||||
expect(attendanceService.getImportableClasses).toHaveBeenCalledWith(7, true);
|
||||
|
||||
await controller.importFromDingTalk(
|
||||
{ start: '2026-07-01', end: '2026-07-02', users: 'ding-1,ding-2', autoMatch: true },
|
||||
{ start: '2026-07-01', end: '2026-07-02', users: 'ding-1,ding-2' },
|
||||
{
|
||||
user: {
|
||||
id: 7,
|
||||
|
||||
@@ -27,11 +27,6 @@ export class DingTalkImportDto {
|
||||
@IsOptional()
|
||||
@IsString()
|
||||
users?: string;
|
||||
|
||||
/** @deprecated Imports are always matched through DingTalk user mappings. */
|
||||
@IsOptional()
|
||||
@Type(() => Boolean)
|
||||
autoMatch?: boolean;
|
||||
}
|
||||
|
||||
/**
|
||||
|
||||
@@ -1,4 +1,4 @@
|
||||
import { ArrayNotEmpty, IsArray, IsIn, IsInt, IsNotEmpty, IsOptional, IsString, Matches, MaxLength } from 'class-validator';
|
||||
import { IsIn, IsNotEmpty, IsOptional, IsString, Matches, MaxLength } from 'class-validator';
|
||||
|
||||
export class GenerateBillsDto {
|
||||
@IsOptional()
|
||||
@@ -36,10 +36,3 @@ export class CancelBillDto {
|
||||
@MaxLength(300)
|
||||
reason: string;
|
||||
}
|
||||
|
||||
export class BatchUpdateBillStatusDto extends UpdateBillStatusDto {
|
||||
@IsArray()
|
||||
@ArrayNotEmpty()
|
||||
@IsInt({ each: true })
|
||||
ids: number[];
|
||||
}
|
||||
|
||||
@@ -1,7 +1,13 @@
|
||||
import { CLASSROOM_TEMPLATE_HEADERS } from './classroom-template';
|
||||
import { CLASSROOM_TEMPLATE_COLUMNS } from './classroom-template';
|
||||
|
||||
describe('classroom import template', () => {
|
||||
it('contains only classroom fields used by the product', () => {
|
||||
expect(CLASSROOM_TEMPLATE_HEADERS).toEqual(['教室名', '楼栋', '楼层', '类型', '容量']);
|
||||
expect(CLASSROOM_TEMPLATE_COLUMNS.map(({ header }) => header)).toEqual([
|
||||
'教室名',
|
||||
'楼栋',
|
||||
'楼层',
|
||||
'类型',
|
||||
'容量',
|
||||
]);
|
||||
});
|
||||
});
|
||||
|
||||
@@ -5,5 +5,3 @@ export const CLASSROOM_TEMPLATE_COLUMNS = [
|
||||
{ header: '类型', key: 'roomType', width: 10 },
|
||||
{ header: '容量', key: 'capacity', width: 10 },
|
||||
];
|
||||
|
||||
export const CLASSROOM_TEMPLATE_HEADERS = CLASSROOM_TEMPLATE_COLUMNS.map(({ header }) => header);
|
||||
|
||||
@@ -16,11 +16,6 @@ export interface JinshujuFieldMapping {
|
||||
studentNo?: string;
|
||||
}
|
||||
|
||||
export const DEFAULT_MAPPING: JinshujuFieldMapping = {
|
||||
name: 'field_1',
|
||||
phone: 'field_2',
|
||||
};
|
||||
|
||||
const mappingTransformer = {
|
||||
to(value: JinshujuFieldMapping): string {
|
||||
return JSON.stringify(value);
|
||||
|
||||
@@ -915,11 +915,6 @@ export class DingTalkService {
|
||||
}));
|
||||
}
|
||||
|
||||
private delay(requestIndex: number): Promise<void> {
|
||||
const ms = requestIndex % 10 === 0 ? 200 : DingTalkService.MIN_INTERVAL;
|
||||
return this.sleep(ms);
|
||||
}
|
||||
|
||||
private sleep(ms: number): Promise<void> {
|
||||
return new Promise((resolve) => setTimeout(resolve, ms));
|
||||
}
|
||||
|
||||
@@ -1,5 +1,4 @@
|
||||
import { NestFactory } from '@nestjs/core';
|
||||
import { ValidationPipe } from '@nestjs/common';
|
||||
import { AppModule } from './app.module';
|
||||
import { runMigrationsOnStartup } from './migration-runner';
|
||||
|
||||
|
||||
Reference in New Issue
Block a user