From 51690928927cd1015bdb3463b805abdc3805e38a Mon Sep 17 00:00:00 2001 From: "makabe.t" Date: Thu, 14 Mar 2024 01:46:57 +0000 Subject: [PATCH] =?UTF-8?q?Merged=20PR=20833:=20=E6=9C=AC=E7=95=AA?= =?UTF-8?q?=E7=92=B0=E5=A2=83=E3=81=AB=E5=AF=BE=E3=81=99=E3=82=8B=E7=A7=BB?= =?UTF-8?q?=E8=A1=8C=E3=83=87=E3=83=BC=E3=82=BF=E3=81=AE=E6=8A=95=E5=85=A5?= =?UTF-8?q?=E5=BE=8C=E3=81=AE=E4=BF=AE=E6=AD=A3?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ## 概要 [Task3580: 本番環境に対する移行データの投入後の修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3580) - デモライセンスの判定を日時から日付だけ見るように修正しました。 - 9999/12/31で始まるかを見ています。 - メールアドレス重複のチェックは大文字小文字を区別せずに実行するようにしています。 ## レビューポイント - 対応箇所は適切でしょうか? - 対応として先頭文字列を見ていますが適切でしょうか? ## UIの変更 - なし ## クエリの変更 - なし ## 動作確認状況 - 本番踏み台で変換できることを確認 - 行った修正がデグレを発生させていないことを確認できるか - ツールの変更だけなので問題なし --- .../server/src/features/transfer/transfer.service.ts | 4 ++-- .../src/features/verification/verification.service.ts | 7 ++++--- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/data_migration_tools/server/src/features/transfer/transfer.service.ts b/data_migration_tools/server/src/features/transfer/transfer.service.ts index b229a51..7ebb4de 100644 --- a/data_migration_tools/server/src/features/transfer/transfer.service.ts +++ b/data_migration_tools/server/src/features/transfer/transfer.service.ts @@ -144,8 +144,8 @@ export class TransferService { } // ライセンスのデータの作成を行う - // line.expired_dateが"9999/12/31 23:59:59"のデータの場合はデモライセンスなので登録しない - if (line.expired_date !== "9999/12/31 23:59:59") { + // line.expired_dateが"9999/12/31"で始まるデータの場合はデモライセンスなので登録しない + if (!line.expired_date.startsWith("9999/12/31")) { // authorIdが設定されてる場合、statusは"allocated"、allocated_user_idは対象のユーザID // されていない場合、statusは"reusable"、allocated_user_idはnull let status: string; diff --git a/data_migration_tools/server/src/features/verification/verification.service.ts b/data_migration_tools/server/src/features/verification/verification.service.ts index 4fda7d7..10733f5 100644 --- a/data_migration_tools/server/src/features/verification/verification.service.ts +++ b/data_migration_tools/server/src/features/verification/verification.service.ts @@ -63,7 +63,7 @@ export class VerificationService { const licensesCountFromFile = csvInputFiles.filter( (item) => - item.type === "USER" && item.expired_date !== "9999/12/31 23:59:59" + item.type === "USER" && !item.expired_date.startsWith("9999/12/31") ).length + cardLicensesCountFromFile; // 管理ユーザ数のカウント @@ -105,7 +105,8 @@ export class VerificationService { // 重複する要素を抽出 const duplicates: { [key: string]: number } = {}; mailAdresses.forEach((str) => { - duplicates[str] = (duplicates[str] || 0) + 1; + duplicates[str.toLowerCase()] = + (duplicates[str.toLowerCase()] || 0) + 1; }); // 重複する要素と件数を表示 @@ -166,7 +167,7 @@ export class VerificationService { VerificationResultDetails, csvInputFiles.filter( (item) => - item.type === "USER" && item.expired_date !== "9999/12/31 23:59:59" + item.type === "USER" && !item.expired_date.startsWith("9999/12/31") ), licenses.filter((item) => item.expiry_date !== null), accountsMappingInputFiles