nik/OMDSCloud
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity
673 Commits 1 Branch 50 Tags
Commit Graph

673 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
maruyama.t
6d56255a5a Merged PR 792: parent_account_idが正しく設定されない 
## 概要
[Task3804: parent_account_idが正しく設定されない](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3804)

Map配列からaccountidをキーにdealerAccountIdを取る処理で、検索keyが逆になっていたため修正。

## レビューポイント
- とくになし

## 動作確認状況
- ローカルで確認（階層を付け替えたアカウントの親子階層が正しいことを確認）

## 補足
- 相談、参考資料などがあれば
 2024-02-28 09:04:36 +00:00
masaaki
0be9c26f09 Merged PR 781: データ検証ツール作成＋動作確認 
## 概要
[Task3573: データ検証ツール作成＋動作確認](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3573)

- データ検証ツールを作成しました

## レビューポイント
- 特にレビューしてほしい箇所
　詳細情報の突き合わせについて、ラフスケッチと対応しているか第三者目線でも確認してほしいです
　verification.serviceのcompareCardLicenses、compareLicenses、compareAccountsになります。

## UIの変更
- 無し

## 動作確認状況
- ローカルで確認

## 補足
- 無し
 2024-02-28 05:31:13 +00:00
maruyama.t
f6d39a4c26 Merged PR 788: [2回目実行]実施後の動作確認 
## 概要
[Task3577: [2回目実行]実施後の動作確認](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3577)

accountがCountryの場合に配下のDistributorの親アカウントIDを付け替える処理について、Typeの付け替えができていなかったのを修正。
ライセンスの有効期限が"9999/12/31 23:59:59.997"でフォーマットチェックをしているが、移行元は"9999/12/31 23:59:59"なので移行元に合わせた。
dealerAccountIdが設定されているが、そのdealerが存在しない場合もデータを作ってしまっている。
→該当レコードはエラーファイルを出力する。

## レビューポイント
- エラーファイルの出力処理だが簡素すぎるか？
JSONで出力する意味はないが、これまでの動作確認で動作担保できているのでJSONで出しています。

## 動作確認状況
- ローカルで確認
正常の場合データ変換が行われることを確認。
dealerAccountIdが設定されているが、そのdealerが存在しない場合もデータでテストした場合、error.jsonが作られることを確認。
## 補足
- 相談、参考資料などがあれば
 2024-02-27 23:55:44 +00:00
maruyama.t
f0d71937e3 Merged PR 780: データ変換ツール（汚いデータ対応版）の作成＋動作確認 
## 概要
[Task3776: データ変換ツール（汚いデータ対応版）の作成](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3776)

綺麗なデータ対応版のレビュー指摘も合わせて修正。
一旦OMDS様よりいただいた1万件～のデータを処理できることは確認済みです。
実装コストとバグの入れ込みを懸念し、有効期限が"9999/12/31"のデータは最初にデータを積む段階で除外するようにしました。

## レビューポイント
- メールアドレス重複チェックについて、想定通りの重複対象を検索出来ているか。
- step3の1.アカウントとユーザが同じ場合
adminMainとuserEmailが重複していた場合に、重複していたユーザーは削除し、アカウントのみを残す（accountユーザーのroleとauthorIdは削除したuserに設定されていたものとする）処理は妥当か。
→accountのIFにroleとauthorIdを追加し、register側のcreateAccountで登録するようにしています。

## 動作確認状況
- ローカルで確認（Account_transition_2024.1.19.csvで実施）
４つのJSONファイルができていることを確認。
Countryの場合の付け替えができていることを確認。
adminMainとemailが重複している場合の重複削除ができていることを確認。

## 補足
- 登録ツールと共通のパラメータで動作するようにしました。
例）
POST:
localhost:8280/transfer
Body:
{
    "inputFilePath": "./data/"
}

　変換ツールの使い方としてはAccount_transition.jsonというファイルを見るようにしています。
 2024-02-27 06:24:41 +00:00
saito.k
0ab6488f58 Merged PR 779: タスク一覧画面のOptionItemがソート条件によって表示順がおかしくなる 
## 概要
[Task3787: タスク一覧画面のOptionItemがソート条件によって表示順がおかしくなる](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3787)

- タスク一覧取得APIレスポンスにあるOptionItemの順番を固定する（idの昇順）
- テスト修正

## レビューポイント
- 特になし

## 動作確認状況
- ローカルで確認

## 補足
- 相談、参考資料などがあれば
 2024-02-26 11:48:02 +00:00
saito.k
5a78a6668f Merged PR 769: /users/relationsレスポンスのWorkTypeIDをID名の昇順にする 
## 概要
[Task3783: /users/relationsレスポンスのWorkTypeIDをID名の昇順にする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3783)

- relationsAPIレスポンスのWorkTypeList内の順番を指定する
- テスト修正

## レビューポイント
- 修正内容に不足はないか

## UIの変更
- Before/Afterのスクショなど
- スクショ置き場

## 動作確認状況
- ローカルで確認

## 補足
- 相談、参考資料などがあれば
 2024-02-26 11:29:22 +00:00
saito.k
d0628caa05 Merged PR 767: パートナー追加成功時に一覧の更新が行われていない 
## 概要
[Task3769: パートナー追加成功時に一覧の更新が行われていない](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3769)

- パートナー追加成功時に一覧の更新を行う

## レビューポイント
- 特になし

## 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/Task3769?csf=1&web=1&e=ajJOBd

## 動作確認状況
- ローカルで確認、develop環境で確認など

## 補足
- 相談、参考資料などがあれば
 2024-02-26 10:42:26 +00:00
masaaki
68d1a1796b Merged PR 783: [1回目実行]実施後の修正実施 
## 概要
[Task3790: [1回目実行]実施後の修正実施](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3790)

- 移行データ動作確認（1回目）で発生した不具合の対応を実施しました
　・バリデータでエラーとなる（コメントアウトして実行したところ、成功）
　・アカウント管理者のユーザーのメール認証がfalseで登録されるので、
　　パスワード変更では認証をできない⇒強制敵にtrueで登録する
　・（指摘外、検証ツール実装時に内部検出）カードライセンス登録時、カードライセンス発行・ライセンステーブルも登録する

- このPull Requestでの対象/対象外
　上記以外の指摘（下記）はタスク3772にて対応するため本プルリク対象外

　変換ツール
　　・ディーラーアカウントに登録されたアカウントがいないCSVで
　　　変換するとディーラーがundefined（パラメータがない）状態でJSON出力されてしまう。
　　・重複したメールアドレスの取り込みが未実装
　　・有効期限が9999/~は移行対象外とする
　　・ワークタイプの出力がされない

　登録ツール
　　・ファイルパスの取り扱いが変換ツールと異なる
- 影響範囲（他の機能にも影響があるか）

## レビューポイント
- カードライセンス登録時のライセンスのアカウントIDについて、第一階層アカウントのため「AUTO_INCREMENT_START: 853211」を設定しているが問題ないか？
　→移行データ上第一階層アカウントは最初に登場するため問題ない認識

## UIの変更
- 無し

## 動作確認状況
- ローカルで確認済

## 補足
- 相談、参考資料などがあれば
 2024-02-26 08:59:37 +00:00
saito.k
b524fd5995 Merged PR 768: U-105メッセージのドイツ語部分が英語になっている 
## 概要
[Task3770: U-105メッセージのドイツ語部分が英語になっている](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3770)

- U-105のメール文面を修正
 - ドイツ語の文章であるべき箇所が英語になっていた

## レビューポイント
- 特になし

## UIの変更
- Before/Afterのスクショなど
- スクショ置き場

## 動作確認状況
- ローカルで確認

## 補足
- 相談、参考資料などがあれば
 2024-02-23 00:42:05 +00:00
makabe.t
fbdfeee73c Merged PR 778: 削除ツールの修正 
## 概要
[Task3788: 削除ツールの修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3788)

- ADB2Cの削除に失敗するので対応しました。
  - 最後のページの場合に削除より先に抜けていたので、削除処理を先にやるように修正しました。

## レビューポイント
- 共有

## UIの変更
- なし

## 動作確認状況
- developで確認
 2024-02-22 11:42:31 +00:00
makabe.t
f03342bc55 Merged PR 777: データ削除ツール作成＋動作確認 
## 概要
[Task3569: データ削除ツール作成＋動作確認](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3569)

- クライアントに不要なフォルダが残っていたので削除しました。

## レビューポイント
- 共有

## UIの変更
- なし
## 動作確認状況
- ローカルで確認
 2024-02-22 08:45:20 +00:00
maruyama.t
a65d6a2774 Merged PR 776: 階層の付け替えを誤っていたのを修正 
## 概要
[Task3570: データ変換ツール（きれいなデータ版）作成＋動作確認](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3570)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）

## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載

## UIの変更
- Before/Afterのスクショなど
- スクショ置き場

## 動作確認状況
- ローカルで確認、develop環境で確認など

## 補足
- 相談、参考資料などがあれば
 2024-02-22 08:40:57 +00:00
maruyama.t
cb68c16eb8 Merged PR 775: 変換ツールのバリデーションチェックを修正 
## 概要
[Task3570: データ変換ツール（きれいなデータ版）作成＋動作確認](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3570)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）

## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載

## UIの変更
- Before/Afterのスクショなど
- スクショ置き場

## 動作確認状況
- ローカルで確認、develop環境で確認など

## 補足
- 相談、参考資料などがあれば
 2024-02-22 08:23:31 +00:00
maruyama.t
c31bb47bb8 Merged PR 773: apiにtransferが存在しなかったのを修正 
## 概要
[Task3570: データ変換ツール（きれいなデータ版）作成＋動作確認](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3570)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）

## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載

## UIの変更
- Before/Afterのスクショなど
- スクショ置き場

## 動作確認状況
- ローカルで確認、develop環境で確認など

## 補足
- 相談、参考資料などがあれば
 2024-02-22 07:52:16 +00:00
makabe.t
dc52ec2022 Merged PR 765: データ削除ツール作成＋動作確認 
## 概要
[Task3569: データ削除ツール作成＋動作確認](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3569)

- ADB2Cからのユーザー削除が100件ごとにしか削除できていなかったので、修正しました。
  - 取得が100件まででそのユーザーに対して削除処理をしていたので100件までの削除になっていました。
  - 対応として、100件づつの削除をユーザーが全削除されるまで実行するようにしました。

## レビューポイント
- 対応方法として適切でしょうか？
- ループで制限を設けていますが、MAX値として適切でしょうか？

## UIの変更
- なし

## 動作確認状況
- ローカルで順に実行できることを確認
- 実際の削除は別途develop環境で実施します。
 2024-02-22 07:33:55 +00:00
maruyama.t
e3ee9412c9 Merged PR 772: フォルダリネーム 
## 概要
[Task3570: データ変換ツール（きれいなデータ版）作成＋動作確認](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3570)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）

## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載

## UIの変更
- Before/Afterのスクショなど
- スクショ置き場

## 動作確認状況
- ローカルで確認、develop環境で確認など

## 補足
- 相談、参考資料などがあれば
 2024-02-22 06:40:52 +00:00
maruyama.t
8110b9cccc Merged PR 753: データ変換ツール作成＋動作確認 
## 概要
[Task3570: データ変換ツール作成＋動作確認](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3570)

## レビューポイント
- 特にtransfer配下の処理を見ていただきたいです。
データのバリデーションは十分か。
アカウントIDとユーザーIDのインクリメント（採番）の場所は正しいか。
ユーザー名は正しく設定できているか。
worktypeの重複を除外する処理は正しいか。
COUNTRY_LIST（既存のクライアントから流用）した変換処理には問題ないか。
出力ファイル名は登録ツール側の想定通りか。

## 動作確認状況

動作確認は、以下で行う。
[タスク 3575: [1回目実行]実施後の動作確認](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_sprints/taskboard/OMDSDictation%20%E3%83%81%E3%83%BC%E3%83%A0/OMDSDictation/%E3%82%B9%E3%83%97%E3%83%AA%E3%83%B3%E3%83%88%2028-2?workitem=3575)

## 補足
- 相談、参考資料などがあれば
 2024-02-22 05:04:48 +00:00
saito.k
ebbf957419 Merged PR 770: AuthorIDを大文字とアンダースコアのみとするバリデーションを入れたことによるデグレの解消 
## 概要
[Task3784: AuthorIDを大文字とアンダースコアのみとするバリデーションを入れたことによるデグレの解消](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3784)

- タイトルの通り
  - ロールがAuthorの時のみチェックするように修正

## レビューポイント
- 特になし

## UIの変更
- Before/Afterのスクショなど
- スクショ置き場

## 動作確認状況
- ローカルで確認、develop環境で確認など

## 補足
- 相談、参考資料などがあれば
 2024-02-22 02:38:05 +00:00
masaaki
12d168d14c Merged PR 754: データ登録ツール作成＋動作確認 
## 概要
[Task3571: データ登録ツール作成＋動作確認](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3571)

- 移行データの登録ツールを作成しました
  - 入力用jsonファイルの読み込み
  - アカウント・ユーザの登録
    - 既存サービスを移植・微修正し呼び出し
    - rate_limit用のsleep実施
  - ワークタイプ・ライセンス・カードライセンスの登録
- 実行についてはpostmanでの実行を考えており、clientは作成しておりません

## レビューポイント
- 既存サービスからの流用が多いですが、メインの処理はfeatures/registerになるため、こちらをメインに見ていただければと思います。

## UIの変更
- 無し

## 動作確認状況
- ローカルで動作確認

## 補足
- 相談、参考資料などがあれば
 2024-02-21 01:41:21 +00:00
makabe.t
5adf7ed12e Merged PR 763: ツールをexeとしてビルドできるビルドコマンドを整備する 
## 概要
[Task3742: ツールをexeとしてビルドできるビルドコマンドを整備する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3742)

- データ移行ツールフォルダの中に`tool`フォルダを掘って、実行ファイル形式で配置されるようにしました。
  - コンテナ内からはサーバー側の`build:exe`コマンドでビルドできるようにしています。
  - コンテナ外からサーバー/クライアント一括でビルドできるスクリプトを配置しています。
※toolフォルダがあればツールとして実行できる想定です。

## レビューポイント
- exe化のイメージは認識通りでしょうか？

## UIの変更
- なし

## 動作確認状況
- ローカルでビルドできることを確認
 2024-02-21 00:11:19 +00:00
makabe.t
a9aca6e4ff Merged PR 750: データ削除ツール作成＋動作確認 
## 概要
[Task3569: データ削除ツール作成＋動作確認](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3569)

- データ削除ツールを実装しました。
  - Blobストレージからのコンテナ削除
  - ADB2Cからのユーザー削除
  - DBの全削除
  - Auto Incrementの設定

## レビューポイント
- Blobストレージの削除対象の取得に問題はないでしょうか？
  - 3つのリージョン内のすべてのコンテナを取得してから、取得したコンテナを全削除するようにしています。
- ADB2Cの削除対象の取得に問題はないでしょうか？
  - ローカルアカウントなユーザーのみを取得してから、取得したユーザーを全削除するようにしています。
- フォルダ構成に違和感はないでしょうか？

## UIの変更
- [Task3569](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/Task3569?csf=1&web=1&e=wU1st1)

## 動作確認状況
- ローカルで確認
  - DB操作のみ確認しています。Azureリソースの削除についてはdevelop環境で改めて実施します。
 2024-02-20 10:09:05 +00:00
saito.k
f8ff19a3fa Merged PR 758: リテラル反映・Typist表示をTranscriptioninstにする 
## 概要
[Task3692: リテラル反映・Typist表示をTranscriptioninstにする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3692)

- フッターのコピーライトを修正
  - 文字列がべた書きだったのを翻訳情報を入れるように修正
- User一覧画面でRoleがTypistの時の表記をTranscriptioninstに修正
- OptionItemの項目名を入れ替え
  - initialValue　⇔　defaultValue
- アカウント画面のSaveボタン下にある代理操作に対する注釈を削除
- 翻訳情報反映

## レビューポイント
- OptionItemの項目名を入れ替えたが、表示するプロパティ名も名前を修正したほうが良いか？(initialValueの列にdefaultValueTypeを表示しているため混乱するのではないかという懸念)
  - APIのレスポンスのプロパティ名から変えないと揃えられないため、表示する項目名の並びだけを変えた
    - なんで入れ替えたのかコメントしたし、これで良いか

## 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/Task3692?csf=1&web=1&e=yWY4ab

## 動作確認状況
- ローカルで確認

## 補足
- 相談、参考資料などがあれば
 2024-02-19 10:29:38 +00:00
saito.k
4caa23ba3c Merged PR 755: [FB対応]File Property表示について 
## 概要
[Task3735: 対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3735)

- FilePropertyの表示順を変更する
  - カテゴリ（General、Job）はそのままで、OMDS様の希望する並び順にする

## レビューポイント
- 並び順の変更にミスはないか

## 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/Task3735?csf=1&web=1&e=r3IhXL

## 動作確認状況
- ローカルで確認

## 補足
- 相談、参考資料などがあれば
 2024-02-19 02:17:37 +00:00
saito.k
b7db9c5fad Merged PR 752: [FB対応]Edgeで表示言語が切り替わらないことがある 
## 概要
[Task3728: 原因調査＆調査結果をPBIに記載する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3728)

- cookieから言語情報取得時にほかのcookieが存在していると、うまく取得できなくなってしまう
  - https://into-the-program.com/javascript-cant-get-value-cookie-name-array/
  - この方法でほかのcookieがある場合でも半角スペースを排除して正しく取得できるようにした。

## レビューポイント
- 特になし

## 動作確認状況
- ローカルで確認

## 補足
- 相談、参考資料などがあれば
 2024-02-19 01:49:23 +00:00
saito.k
b4df229de4 Merged PR 751: [FB対応]WorkTypeIDを大文字固定にしたい 
## 概要
[Task3733: 対応する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3733)

- 大文字のみの入力に強制する
  - WorkTypeID
  - OptionItem

- APIの大文字固定のパラメータにバリデーションを付与する
  - AuthorID
  - WorkTypeID
  - OptionItem

## レビューポイント
- 大文字に変換する処理をreducerに寄せたが問題ないか
- バリデータを付けるパラメータに漏れはないか

## 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/Task3733?csf=1&web=1&e=zNB1Hi

## 動作確認状況
- ローカルで確認

## 補足
- 相談、参考資料などがあれば
 2024-02-19 01:09:39 +00:00
湯本 開
cd58916e05 Merged PR 746: ツールの雛形作成 
## 概要
[Task3696: ツールの雛形作成](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3696)

- DevContainerを追加

## レビューポイント
- 特になし

## 動作確認状況
- ローカルで確認
 2024-02-15 05:49:45 +00:00
saito.k
25b7936bf4 Merged PR 747: デザイン反映 
## 概要
[Task3670: デザイン反映](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3670)

- ユーザー一覧画面のヘッダーのスタイル修正
  - ブラウザのサイズに合わせて折り返すようにした
  - ユーザー一覧のテーブルを横スクロールできるように修正
- ライセンス履歴画面にパンくずリストを表示
- OptionItem更新ポップアップのスタイル修正
  - 端が切れてしまっていたため枠に収まるようにした
- Typistを「Selected」と「Pool」で選択するUIのスタイル修正
  - typist名やグループ名が長くてもチェックアイコンに重ならないようにした

## レビューポイント
- 特になし

## 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/Task3670?csf=1&web=1&e=zIcngJ
## 動作確認状況
- ローカルで確認

## 補足
- 相談、参考資料などがあれば
 2024-02-14 01:50:38 +00:00
saito.k
73ef74770e Merged PR 745: タスク取得処理のクエリ発行箇所の修正 
## 概要
[Task3673: タスク取得処理のクエリ発行箇所の修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3673)

- タスク取得時にRelationでOptionItemも取得しているが、そのJoinカラムの指定方法に誤りがあったため正しいジョインカラムを指定するように修正した。
  - 本来はタスクレコードのaudio_file_idとOptionItemのaudio_file_idでjoinするはずだが、タスクレコードのidとOptionItemのaudio_file_idでjoinしていた
  - タスクと音声ファイルは同時にレコードが作られるため基本的にidが一致する傾向があり、テストで発見できていなかった模様
  - https://orkhan.gitbook.io/typeorm/docs/relations#joincolumn-options
- テスト修正

## レビューポイント
- 特になし

## UIの変更

## 動作確認状況
- ローカルで確認

## 補足
- 相談、参考資料などがあれば
 2024-02-13 02:36:01 +00:00
saito.k
9cc9a3bd94 Merged PR 744: タスク一覧画面修正 
## 概要
[Task3672: タスク一覧画面修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3672)

- タスク一覧の表示項目制御用のチェックボックスの操作可能範囲を修正
  - 「Comment」のチェックボックスをリテラルまで押せるようにした
- タスク一覧のPriorityがHighの時の表示を変更した
  - 行を赤く表示→文字を赤く表示

## レビューポイント
- 修正する箇所の認識は合っているか

## 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/Task3672?csf=1&web=1&e=nSX5B0

## 動作確認状況
- ローカルで確認

## 補足
- 相談、参考資料などがあれば
 2024-02-13 02:12:00 +00:00
makabe.t
a9d326c234 Merged PR 740: トライアルライセンスを有効にしても有効化メールが届かない対応 
## 概要
[Task3651: トライアルライセンスを有効にしても有効化メールが届かない対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3651)

- ライセンス割り当て時のメール（U-108）について、ディーラーが設定されていない場合にはディーラー関連の文言を表示しないように修正しました。

## レビューポイント
- メッセージ内容は適切でしょうか？
- ディーラーの取り扱いは適切でしょうか？

## UIの変更
- なし
## 動作確認状況
- ローカルで確認
 2024-02-09 09:58:06 +00:00
湯本 開
7ca4249f04 Merged PR 742: ログアウトせずに認証切れまで待った後にTOPページからログインしようとするとエラーが発生する 
## 概要
[Task3680: ログアウトせずに認証切れまで待った後にTOPページからログインしようとするとエラーが発生する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3680)

- 現象
  - ブラウザバック対策のコードで、アクセストークンの寿命を意識せずにログイン後画面に遷移していたため、リフレッシュトークンとアクセストークンの再発行がSkipされた上で寿命が切れたトークンを使うような状態でログイン後画面に遷移してしまっていた
- 対応
  - /login, /authで、アクセストークンチェック→IdTokenを使ったログイン中プロセスかチェック の順番を変更
    - 有効なアクセストークンがあったとしても、再ログインをしてるのであればリフレッシュトークン等の再発行を実施する形に変更

## レビューポイント
- Bugの現象が解消する以外の挙動の変更が発生してしまわないか
  - 特にブラウザバック等
- 実装方法で問題がありそうな部分はないか
  - 不要なTokenのクリア等はないか

## UIの変更
- なし

## 動作確認状況
- ローカルで確認

## 補足
- **可能であればローカルで挙動を確認していただきたいです**
  - .env.local の `ACCESS_TOKEN_LIFETIME_WEB` を60(=1分)とかにすれば確認はしやすいはず
 2024-02-09 08:12:26 +00:00
maruyama.t
0d0f624a3f Merged PR 733: ライセンス自動割り当てのFunctionsがエラーになっている 
## 概要
[Task3631: ライセンス自動割り当てのFunctionsがエラーになっている](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3631)

ユーザーテーブルからライセンスアラートを削除したが、Functionsの追従が漏れていた

## レビューポイント
- とくになし

## 動作確認状況
- ローカルで確認

## 補足
- 相談、参考資料などがあれば
 2024-02-07 07:34:53 +00:00
湯本 開
a1b7505035 Merged PR 732: [改善]認証用URLについて、ドメイン名の末尾に/が必要となることへの対応 
## 概要
[Task3625: [改善]認証用URLについて、ドメイン名の末尾に/が必要となることへの対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3625)

- URLクラスとpathクラスを用いてURLを構築するよう修正
- 送信するメールに関わるテストを追加

## レビューポイント
- 修正内容は妥当であるか
- 漏れていそうなURL系の処理はないか
- 工数面を考慮したコスト対効果の観点から、メール送信を行うテスト全てに送信メール内容のチェックを行うテストコードは入れなかったが許容可能か

## 動作確認状況
- npm run testは通過
  - `.env.test` の `APP_DOMAIN` の末尾 `/` を付けて通過 & 消して通過 するかを確認
  - **一応追試をお願いしたいです**
 2024-02-06 05:03:45 +00:00
saito.k
84b0da1f95 Merged PR 723: [FB対応]タイピストグループ重複時のエラーとする 
## 概要
[Task3613: 対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3613)

- タイピストグループ名が重複した際のエラーを追加
  - タイピストグループ追加API
  - タイピストグループ更新API
- タイピストグループ設定画面に表示するエラーメッセージを追加

## レビューポイント
- 行ロックするべきかどうか
  - ギリギリのタイミングで同名のタイピストグループが作成される場合は防げないのでDBでユニーク制約を設定する？
    - insertにロックはかけられないから

## 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/Task3613?csf=1&web=1&e=i8cN2g

## 動作確認状況
- ローカルで確認

## 補足
- 相談、参考資料などがあれば
 2024-02-05 11:46:29 +00:00
saito.k
48d2e625db Merged PR 730: [FB対応]タスク一覧画面のデザイン修正 
## 概要
[Task3614: 対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3614)
- デザイン修正
  - ソートが昇順の時は▲
  - ソートが降順の時は▼
- Priorityが「High」の時にその行を赤くする

## レビューポイント
- 特になし

## 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/Task3614?csf=1&web=1&e=fm3MFs

## 動作確認状況
- ローカルで確認

## 補足
- 相談、参考資料などがあれば
 2024-02-05 07:08:00 +00:00
makabe.t
878657ad4a Merged PR 713: 行ロック横展開3 
## 概要
[Task3471: 行ロック横展開3](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3471)

- 以下のリポジトリのメソッドに行ロックを追加しました。
- tasks
  - checkout
    - タスク、チェックアウト情報にロックを追加して途中で変更できないようにする
  - checkin
    - タスクのチェックにロックを追加してタスクのステータスを途中で変更できないようにする
  - cancel
    - タスクのチェックにロックを追加してタスクのステータスを途中で変更できないようにする
  - suspend
    - タスクのチェックにロックを追加してタスクのステータスを途中で変更できないようにする
  - backup
    - タスクのチェックにロックを追加してタスクのステータスを途中で変更できないようにする
  - create
    - タスクのチェックにロックを追加してJobNumberのチェックで重複しないようにする
  - changeCheckoutPermission
    - タスク、チェックアウト候補のチェックにロックを追加して途中で変更されないようにする
  - autoRouting
    - 処理中にワークフロー・ワークタイプの取得にロックを追加して意図しない対象のでチェックアウト権限が作成されないようにする
    - ワークタイプ・ワークフローの更新/削除時にもロックを追加

こちらの資料を参考に対応しています。
[行ロックに関する影響調査](https://ndstokyo.sharepoint.com/❌/r/sites/Piranha/Shared%20Documents/General/OMDS/%E8%A1%8C%E3%83%AD%E3%83%83%E3%82%AF%E3%81%AB%E9%96%A2%E3%81%99%E3%82%8B%E5%BD%B1%E9%9F%BF%E8%AA%BF%E6%9F%BB.xlsx?d=wdd6f3d97f7b04a538095c459f8eee2eb&csf=1&web=1&e=9M43di)

対応箇所について以下にまとめました。
[Task3471](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/Task3471?csf=1&web=1&e=wptJqD)

## レビューポイント
- 対応箇所は適切でしょうか？

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2024-02-05 06:19:24 +00:00
saito.k
1daeedbfdb Merged PR 722: アカウント画面に注釈を追加する＆翻訳反映 
## 概要
[Task3592: アカウント画面に注釈を追加する＆翻訳反映](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3592)

- ディーラーマネジメントの下、保存ボタンの下に注釈追加
  - タスクの中のBacklogに詳細があります
- 翻訳反映

## レビューポイント
- 特になし

## 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/Task3592?csf=1&web=1&e=1wBNB6

## 動作確認状況
- ローカルで確認

## 補足
- 相談、参考資料などがあれば
 2024-02-05 06:11:41 +00:00
makabe.t
d5178e7435 Merged PR 716: 行ロック横展開5 
## 概要
[Task3473: 行ロック横展開5](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3473)

以下のリポジトリのメソッドについてlockを追加しました。
- worktypes
  - createWorktype
    - ワークタイプ取得にロックを追加して途中で追加できないようにする
  - updateWorktype
    - ワークタイプ取得にロックを追加して途中で同一ワークタイプIDが作られないようにする
  - deleteWorktype
    - ワークフロー取得にロックを追加して途中でワークタイプが紐づけられないようにする
  - updateOptionItems
    - ワークタイプ取得にロックを追加して同一ワークタイプに対してオプションアイテムを作らないようにする

こちらの資料を参考にして対応しています。
[行ロックに関する影響調査](https://ndstokyo.sharepoint.com/❌/r/sites/Piranha/Shared%20Documents/General/OMDS/%E8%A1%8C%E3%83%AD%E3%83%83%E3%82%AF%E3%81%AB%E9%96%A2%E3%81%99%E3%82%8B%E5%BD%B1%E9%9F%BF%E8%AA%BF%E6%9F%BB.xlsx?d=wdd6f3d97f7b04a538095c459f8eee2eb&csf=1&web=1&e=Y5l3aA)

対応内容についてはこちらにまとめています。
[行ロック5の対応整理](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/Task3473?csf=1&web=1&e=wqCx0Z)
## レビューポイント
- 対応箇所は適切でしょうか？

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2024-02-05 00:21:57 +00:00
saito.k
4548b5e510 Merged PR 715: 行ロック横展開4 
## 概要
[Task3472: 行ロック横展開4](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3472)

- 対象メソッド
  - user_groups
    - createTypistGroup
      - グループに含めるユーザー情報取得箇所でロック追加
        - ユーザー削除と被ると、削除済みユーザーをユーザーグループに含めてしまう
    - updateTypistGroup
       - グループに含めるユーザー情報取得箇所でロック追加
         - ユーザー削除と被ると、削除済みユーザーをユーザーグループに含めてしまう
       - グループの存在確認を行う箇所
         - グループ削除と被ると、削除済みのグループにメンバーを割り当ててしまう
  - workflows
    - createtWorkflows
    - updatetWorkflow
- インデックス追加
  - user
     - role

## レビューポイント
- インデックスの貼り忘れはないか
- ロックの追加忘れはないか

## 共有資料
  - 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/Task3472?csf=1&web=1&e=jjb0QV

## 動作確認状況
- ローカルでロックされている箇所で待ちが発生していることを確認

## 補足
- 相談、参考資料などがあれば
 2024-02-02 02:37:39 +00:00
saito.k
06b5249e5a Merged PR 717: [FB対応]画面更新するとヘッダーのソートがJob numberに戻ってしまう 
## 概要
[Task3611: 対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3611)

- ソート条件を変更した際にローカルストレージに保持するように修正
  - 例　`direction:ASC,paramName:RECORDING_STARTED_DATE`
- ローカルストレージにソート条件が入っていれば、その条件でソートしたタスクを取得するように修正
- PlayBack時にローカルストレージにあるソート条件を削除するように修正
  - 削除することで、次回の画面初期表示時はPlayBackを押したときのソート条件を使用することができる。
- PlayBack時にユーザーがタイピストの時のみソート条件を保存していたがその制限は不要そうだったのでAuthorでもソート条件を更新するように修正。
  - AuthorがPlayBack押下時にソート条件を更新しても不都合はないため。

## レビューポイント
- ローカルストレージに保存する処理を入れる箇所に問題はないか

## 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/Task3611?csf=1&web=1&e=5uG6f4

## 動作確認状況
- ローカルで確認

## 補足
- 相談、参考資料などがあれば
 2024-02-02 02:00:47 +00:00
maruyama.t
e877942175 Merged PR 711: Repositoryロック対応 
## 概要
[Task3523: Repositoryロック対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3523)

LicensesRepository
  - allocateLicense
割当先ユーザ取得しロックする処理を追加

WorkflowRepository
 - createtWorkflows
authorの存在確認時にロックを追加
 - updatetWorkflows
authorの存在確認時にロックを追加
UserGroup&UserGroupMemberのロックを追加

AccountsRepository
- updateAccountInfo
プライマリ/セカンダリ管理者ユーザーの存在チェックのロック
（行ロック横展開1で修正されていた）

TasksRepository
- create
タスクの所有者の存在確認とロックを追加
- checkout
対象ユーザの存在確認とロックを追加
- changeCheckoutPermission
対象ユーザの存在確認とロックを追加

UserGroupsRepository
- createTypistGroup
対象ユーザ達のロックを追加
- updateTypistGroup
対象ユーザ達のロックを追加
## レビューポイント
ラフスケッチの、

```
競合ケース E-3. Typistが削除条件判定を行った直後に、チェックアウト候補に削除ユーザーが含まれるTypistGroupが割り当てられる
TypistGroupに割り当たっている時点で削除条件を満たさないので、このケースはないはず
```

ここは未対応でよい認識か。

## 動作確認状況
- ローカルで確認

## 補足
- 相談、参考資料などがあれば
 2024-02-01 06:06:10 +00:00
makabe.t
6d6eee91e0 Merged PR 712: 行ロック横展開2 
## 概要
[Task3470: 行ロック横展開2](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3470)

以下のリポジトリの各メソッドについてロックを追加し、必要なインデックスを追加するマイグレーションファイルを追加しました。
- licenses
  - order
    - 注文の取得にロックを追加して同じPO番号をチェックできるようにする
  - createCardLicenses
    - ライセンスキーチェック毎にロックを追加して同じライセンスキーをチェックできるようにする
  - activateCardLicense
    - ライセンスキーのチェックにロックを追加して同一のライセンスキーがアクティベート済みかチェックできるようにする
  - issueLicense
    - 注文のチェックにロックを追加して同一の注文に対して複数回ライセンスが発行されないようにする
  - allocateLicense
    - ユーザーのライセンス状態取得にロックを追加して複数回割り当てできないようにする
  - deallocateLicense
    - ユーザーのライセンス状態取得にロックを追加して複数回解除できないようにする
  - cancelOrder
    - 注文のチェックにロックを追加してキャンセル中にライセンスが発行されないようにする

※こちらの資料を参考に各メソッド内で影響に関連すると思われるselectにロックを追加しています。
[行ロックに関する影響調査.xlsx](https://ndstokyo.sharepoint.com/❌/r/sites/Piranha/Shared%20Documents/General/OMDS/%E8%A1%8C%E3%83%AD%E3%83%83%E3%82%AF%E3%81%AB%E9%96%A2%E3%81%99%E3%82%8B%E5%BD%B1%E9%9F%BF%E8%AA%BF%E6%9F%BB.xlsx?d=wdd6f3d97f7b04a538095c459f8eee2eb&csf=1&web=1&e=qASAOx)

上記資料を参考にタスク内で担当するメソッドについてロックの対応箇所を整理しました。
[Task3470](https://ndstokyo.sharepoint.com/:f:/r/sites/Piranha/Shared%20Documents/General/%E3%83%A9%E3%82%A4%E3%82%BB%E3%83%B3%E3%82%B9%E3%83%9D%E3%83%BC%E3%82%BF%E3%83%AB/%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/Task3470?csf=1&web=1&e=OGnOhp)

## レビューポイント
- 各メソッドの対応方針は適切でしょうか？

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2024-01-31 00:57:36 +00:00
saito.k
f9d5082f39 Merged PR 706: 翻訳情報を反映する 
## 概要
[Task3529: 翻訳情報を反映する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3529)

- 翻訳情報を反映

## レビューポイント
- 特になし

## UIの変更
- Before/Afterのスクショなど
- スクショ置き場

## 動作確認状況
- ローカルで確認、develop環境で確認など

## 補足
- 相談、参考資料などがあれば
 2024-01-29 08:38:26 +00:00
makabe.t
dbff86915e Merged PR 708: 対応 
## 概要
[Task3550: 対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3550)

- Worktypeの登録・編集時に16文字までとなるように制限を修正しました。
  - APIのバリデータ
  - 画面の最大文字数

## レビューポイント
- 対応内容の認識はあっていますでしょうか？

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2024-01-29 05:54:35 +00:00
makabe.t
4d462a883a Merged PR 688: 行ロック横展開1 
## 概要
[Task3468: 行ロック横展開1](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3468)

- 各リポジトリのメソッドについて行ロックが必要な箇所に処理を追加しました。
- **accounts**
  - **cancelIssue**
    - `canselIssue`と`allocateLicense`での対象ライセンス取得にロックを追加しました。
  - **updateAccountInfo**
    - `updateAccountInfo`でのプライマリ/セカンダリ管理者取得にロックを追加しました。
- **templates**
  - **upsertTemplateFile**
    - `upsertTemplateFile`のテンプレートファイル取得にロックを追加しました。
- **users**
  - **update**
    - ユーザー取得にロックを追加しました。
      - 影響としてはAuthorIDの重複が考えられたのでその対応のために入れています。
  - **findDelegateUser**
    - selectのみでデータの不整合はないので特に処置はしていません。
  - **isAllowDelegationPermission**
    - selectのみでデータの不整合はないので特に処置はしていません。

※こちらの資料を参考に各メソッド内で影響に関連すると思われるselectにロックを追加しています。
[行ロックに関する影響調査.xlsx](https://ndstokyo.sharepoint.com/❌/r/sites/Piranha/Shared%20Documents/General/OMDS/%E8%A1%8C%E3%83%AD%E3%83%83%E3%82%AF%E3%81%AB%E9%96%A2%E3%81%99%E3%82%8B%E5%BD%B1%E9%9F%BF%E8%AA%BF%E6%9F%BB.xlsx?d=wdd6f3d97f7b04a538095c459f8eee2eb&csf=1&web=1&e=saqcTC)

上記資料を参考にタスク内で担当するメソッドについてロックの対応箇所を整理しました。
[Task3520](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/Task3520?csf=1&web=1&e=ewuJoe)

## レビューポイント
- 競合の対応として適切でしょうか？

## UIの変更
- なし

## 動作確認状況
- ローカルでマイグレーション確認
 2024-01-29 02:52:24 +00:00
maruyama.t
986b710aaa Merged PR 707: 対応 
## 概要
[Task3507: 対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3507)

アップロードとcheckout時にライセンスチェックを行う。ダウンロード時にはチェックを行わない。
- 影響範囲（他の機能にも影響があるか）
checkoutのとき、第五階層の場合にチェックが行われる。
外部連携APIが返却するパラメータが変わるため、OMDSさんに連携する

## レビューポイント
- 不要な個所まで削除していないか（アップロードの場合はチェックを残す）

## 動作確認状況
- ローカルで確認

## 補足
- 相談、参考資料などがあれば
 2024-01-26 11:07:56 +00:00
makabe.t
794dae0c15 Merged PR 709: テスト対応もれ修正 
## 概要
[Task3589: テスト対応もれ修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3589)

- バグ対応で漏れていたテスト修正を実施しました。

## レビューポイント
- 共有

## UIの変更
-なし

## 動作確認状況
- ローカルで確認
 2024-01-26 08:58:08 +00:00
makabe.t
271d85482d Merged PR 705: 対応 
## 概要
[Task3549: 対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3549)

- 第五階層のユーザーがライセンス注文する際にディーラーが設定されていない場合に出るエラーを専用のメッセージとなるように修正しました。
※文言はメッセージレビュー後に必要があれば修正します。

## レビューポイント
- エラーの仕分け方は適切でしょうか？

## UIの変更
- [Task3549](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/Task3549?csf=1&web=1&e=uwyFMW)

## 動作確認状況
- ローカルで確認
 2024-01-26 08:03:18 +00:00
saito.k
7c83a9ccae Merged PR 704: ユーザー認証メールのURLの期限について 
## 概要
[Task3515: ユーザー認証メールのURLの期限について](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3515)

- 環境変数の有効期限に使用されている値を見直し
  - アクセストークンの有効期限
    - ミリ秒→秒単位に修正
  - リフレッシュトークンの有効期限
    - ミリ秒→秒単位に修正
  - 認証メールリンクの有効期限
    - ミリ秒→秒単位に修正
  - SASトークンの有効期限
    - 有効期限が間違っていたので修正
      - 2時間に修正（奥澤さんと調整済み）
  - キャッシュの有効期限
    - 24時間であっていたのでそのまま
## レビューポイント
- ほかに確認する値はあるか

## UIの変更
- Before/Afterのスクショなど
- スクショ置き場

## 動作確認状況
- トークン生成時に渡す有効期限情報が秒単位であっているのか値を変更して確認した

## 補足
- 相談、参考資料などがあれば
 2024-01-25 06:53:02 +00:00
masaaki
52b71a267b Merged PR 695: テスト対応 
## 概要
[Task3505: テスト対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3505)

- アカウントIDの初期値を変更するためのSQLを作成
- 運用開始時に忘れないよう、登録を行うためのタスクを追加
  タスク 3530: アカウントIDをユーザ指定の通番から開始するようにAUTO_INCRIMENTの値を変更する

## レビューポイント
- 特にレビューしてほしい箇所
  - SQLファイルの登録先。
    既存のフォルダで意味として矛盾の無い「DB/init」配下としたが妥当か。
    今後移行用のスクリプトの登録などが予想されるので、そこを見据えた方がよいか？

## UIの変更
- 無し

## 動作確認状況
- ローカルで確認

## 補足
- レビューOKが出たタイミングで、dev/stg環境に一度適用を行います。
    prod環境については、現在のデータを残す可能性があるので、適用は移行時に実施する想定です。
　→（2024/1/24追記）データ移行のタイミングでデータは一度きれいにするので、prod環境にも適用する
 2024-01-24 09:25:47 +00:00