From d70b471d550ad34fd80bb45819bbe16cce1b6405 Mon Sep 17 00:00:00 2001 From: "saito.k" Date: Mon, 7 Aug 2023 08:59:38 +0000 Subject: [PATCH] =?UTF-8?q?Merged=20PR=20311:=20Task=E4=B8=80=E8=A6=A7?= =?UTF-8?q?=E5=8F=96=E5=BE=97=E3=81=A7status=E6=9C=AA=E8=A8=AD=E5=AE=9A?= =?UTF-8?q?=E3=81=AE=E5=A0=B4=E5=90=88=E3=81=AB=EF=BC=90=E4=BB=B6=E3=81=A8?= =?UTF-8?q?=E3=81=AA=E3=81=A3=E3=81=A6=E3=81=97=E3=81=BE=E3=81=86=E3=81=AE?= =?UTF-8?q?=E3=81=A7=E5=AF=BE=E5=BF=9C?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ## 概要 [Task2367: Task一覧取得でstatus未設定の場合に0件となってしまうので対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2367) - タイトルの通り ## レビューポイント - 修正箇所はこれでよいか ## UIの変更 - https://ndstokyo.sharepoint.com/:f:/r/sites/Piranha/Shared%20Documents/General/OMDS/%E3%82%B9%E3%82%AF%E3%83%AA%E3%83%BC%E3%83%B3%E3%82%B7%E3%83%A7%E3%83%83%E3%83%88/Task2367?csf=1&web=1&e=u7lReD ## 動作確認状況 - ローカルで確認、develop環境で確認など ## 補足 - 相談、参考資料などがあれば --- .../src/features/tasks/tasks.controller.ts | 3 ++- dictation_server/src/features/tasks/tasks.service.ts | 10 ++++++---- 2 files changed, 8 insertions(+), 5 deletions(-) diff --git a/dictation_server/src/features/tasks/tasks.controller.ts b/dictation_server/src/features/tasks/tasks.controller.ts index f09a622..057dece 100644 --- a/dictation_server/src/features/tasks/tasks.controller.ts +++ b/dictation_server/src/features/tasks/tasks.controller.ts @@ -98,7 +98,8 @@ export class TasksController { decodedToken, offset, limit, - status?.split(',') ?? [], + // statusが指定されていない場合は全てのステータスを取得する + status?.split(','), paramName, direction, ); diff --git a/dictation_server/src/features/tasks/tasks.service.ts b/dictation_server/src/features/tasks/tasks.service.ts index 7326ac0..23659b7 100644 --- a/dictation_server/src/features/tasks/tasks.service.ts +++ b/dictation_server/src/features/tasks/tasks.service.ts @@ -51,7 +51,7 @@ export class TasksService { accessToken: AccessToken, offset: number, limit: number, - status: string[], + status?: string[], paramName?: TaskListSortableAttribute, direction?: SortDirection, ): Promise<{ tasks: Task[]; total: number }> { @@ -66,6 +66,8 @@ export class TasksService { // パラメータが省略された場合のデフォルト値: 保存するソート条件の値の初期値と揃える const defaultParamName: TaskListSortableAttribute = 'JOB_NUMBER'; const defaultDirection: SortDirection = 'ASC'; + // statusが省略された場合のデフォルト値: 全てのステータス + const defaultStatus = Object.values(TASK_STATUS); try { const { account_id, author_id } = @@ -78,7 +80,7 @@ export class TasksService { limit, paramName ?? defaultParamName, direction ?? defaultDirection, - status, + status ?? defaultStatus, ); // B2Cからユーザー名を取得する @@ -101,7 +103,7 @@ export class TasksService { limit, paramName ?? defaultParamName, direction ?? defaultDirection, - status, + status ?? defaultStatus, ); // B2Cからユーザー名を取得する @@ -122,7 +124,7 @@ export class TasksService { limit, paramName ?? defaultParamName, direction ?? defaultDirection, - status, + status ?? defaultStatus, ); // B2Cからユーザー名を取得する const b2cUsers = await this.getB2cUsers(