Skip to content

Commit 532d79e

Browse files
committed
chore: refactor per review
1 parent 8ae81b2 commit 532d79e

3 files changed

Lines changed: 18 additions & 23 deletions

File tree

src/issue/domain/errors/issue.errors.ts

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,10 @@
11
export const IssueErrorCodes = {
2-
// 404
32
NOT_FOUND: 'ISSUE.NOT_FOUND',
43

5-
// 409 — Conflict
64
TITLE_DUPLICATE: 'ISSUE.TITLE_DUPLICATE',
75
ALREADY_DELETED: 'ISSUE.ALREADY_DELETED',
86
NOT_DELETED: 'ISSUE.NOT_DELETED',
97

10-
// 400 — Bad Request
118
TITLE_REQUIRED: 'ISSUE.TITLE_REQUIRED',
129
TITLE_TOO_LONG: 'ISSUE.TITLE_TOO_LONG',
1310
AREA_REQUIRED: 'ISSUE.AREA_REQUIRED',
@@ -17,14 +14,11 @@ export const IssueErrorCodes = {
1714
PARENT_NOT_FOUND: 'ISSUE.PARENT_NOT_FOUND',
1815
SELF_PARENT: 'ISSUE.SELF_PARENT',
1916

20-
// 403 — Forbidden
2117
ACCESS_DENIED: 'ISSUE.ACCESS_DENIED',
2218

23-
// 422 — Unprocessable
2419
HAS_SUBTASKS: 'ISSUE.HAS_SUBTASKS',
2520
CANNOT_MOVE_TO_DIFFERENT_AREA: 'ISSUE.CANNOT_MOVE_TO_DIFFERENT_AREA',
2621

27-
// 500 — Internal
2822
CREATE_FAILED: 'ISSUE.CREATE_FAILED',
2923
UPDATE_FAILED: 'ISSUE.UPDATE_FAILED',
3024
DELETE_FAILED: 'ISSUE.DELETE_FAILED',

src/project/application/use-cases/index.ts

Lines changed: 15 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,11 @@
1-
import { MemberQueries, MemberUseCases } from './member';
2-
import { ProjectQueries, ProjectUseCases } from './project';
1+
import { FindProjectMemberQuery, MemberQueries, MemberUseCases } from './member';
2+
import {
3+
CheckVisibilityOrThrowQuery,
4+
CreateProjectUseCase,
5+
FindProjectQuery,
6+
ProjectQueries,
7+
ProjectUseCases,
8+
} from './project';
39

410
export * from './project';
511
export * from './member';
@@ -10,3 +16,10 @@ export const USE_CASES = [
1016
...MemberUseCases,
1117
...ProjectUseCases,
1218
];
19+
20+
export const EXPORT_USE_CASES = [
21+
FindProjectQuery,
22+
CreateProjectUseCase,
23+
FindProjectMemberQuery,
24+
CheckVisibilityOrThrowQuery,
25+
];

src/project/project.module.ts

Lines changed: 3 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -4,26 +4,14 @@ import { forwardRef, Module } from '@nestjs/common';
44

55
import { CONTROLLERS } from './application/controllers';
66
import { ProjectFacade } from './application/project.facade';
7-
import {
8-
CreateProjectUseCase,
9-
FindProjectQuery,
10-
FindProjectMemberQuery,
11-
USE_CASES,
12-
CheckVisibilityOrThrowQuery,
13-
} from './application/use-cases';
14-
import { POLICIES, ProjectAccessPolicy } from './domain/policy';
7+
import { USE_CASES, EXPORT_USE_CASES } from './application/use-cases';
8+
import { POLICIES } from './domain/policy';
159
import { REPOSITORIES } from './infrastructure/persistence/repositories';
1610

1711
@Module({
1812
imports: [UserModule, forwardRef(() => TeamsModule)],
1913
controllers: CONTROLLERS,
2014
providers: [...REPOSITORIES, ...POLICIES, ...USE_CASES, ProjectFacade],
21-
exports: [
22-
FindProjectQuery,
23-
ProjectAccessPolicy,
24-
CreateProjectUseCase,
25-
FindProjectMemberQuery,
26-
CheckVisibilityOrThrowQuery,
27-
],
15+
exports: [...EXPORT_USE_CASES, ...POLICIES],
2816
})
2917
export class ProjectModule {}

0 commit comments

Comments
 (0)