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

42 Commits

Author SHA1 Message Date
saito.k
dfdc6a33ad Merged PR 894: API修正(アカウント削除系) 
## 概要
[Task4034: API修正(アカウント削除系)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/4034)

- アカウント削除時にJobNumberテーブルのレコードも削除するように修正
- パートナー削除時にJobNumberテーブルのレコードも削除するように修正
- テスト修正

## レビューポイント
- テストケースに不足はないか
- ジョブナンバーテーブルの削除順に問題はないか

## クエリの変更
- Repositoryを変更し、クエリが変更された場合は変更内容を確認する
- JobNumberテーブルのレコードを削除する処理を追加した
  - 既存のクエリに影響はなし

## 動作確認状況
- ローカルで確認
- 行った修正がデグレを発生させていないことを確認できるか
  - 既存のテストが通ることを確認
  - テストしていなかった観点（ソート条件も削除されているか等）も確認するように修正

## 補足
- 相談、参考資料などがあれば
 2024-05-14 02:12:41 +00:00
Kentaro Fukunaga
75f0a49fc1 Merged PR 831: 親アカウント変更API実装 
## 概要
[Task3853: 親アカウント変更API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3853)

- 親アカウント切り替えAPIを実装しました。

## レビューポイント
- Service層の関数の分け方に改善点ないか？
- テストケースで他にあったほうがいいものや観点などあるか？

## UIの変更
- なし

## クエリの変更
- なし

## 動作確認状況
- ローカルで全テスト通ることを確認
- 行った修正がデグレを発生させていないことを確認できるか
    - 新規APIの実装のため既存実装に変更なし
 2024-03-18 05:47:24 +00:00
Kentaro Fukunaga
aef30c8cbe Merged PR 748: 第五階層ライセンス情報取得API実装 
## 概要
[Task3655: 第五階層ライセンス情報取得API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3655)

- 第五階層ライセンス情報取得APIに、ストレージ上限とストレージ使用量を取得する処理を追加しました。
- 既存の「割り当て済みライセンス取得処理」と「再利用可能ライセンス取得処理」に不要な条件があったため削除しました

## レビューポイント
- 上限計算方法、使用量取得条件に仕様との認識齟齬はないか？
    - もしくはテストケースで「これもあったほうがいいのでは？」などないか
- その他気になる点あれば

## 動作確認状況
- ローカルでテストが全部通ることを確認
 2024-02-16 02:11:34 +00:00
makabe.t
19b544540e Merged PR 724: API実装 
## 概要
[Task3535: API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3535)

- タイピストグループ削除APIとテストを実装しました。

accountのテストがうまくいっていないようなので別途見直します。
※タイピストグループ削除のテストはうまくいっています

## レビューポイント
- エラーケースと出力されるコードは適切でしょうか？
- テストケースは適切でしょうか？

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2024-02-06 07:46:57 +00:00
makabe.t
81c299dd99 Merged PR 680: タスク削除API IF実装 
## 概要
[Task3456: タスク削除API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3456)

- タスク削除APIのIFを実装しopenapi.jsonを更新しました。

## レビューポイント
- パス、バリデータは想定通りでしょうか？

## UIの変更
- なし
## 動作確認状況
- ローカルで確認
 2024-01-11 08:47:28 +00:00
makabe.t
87ede1ad3b Merged PR 672: パートナー追加時のメール文面の適用 
## 概要
[Task3430: パートナー追加時のメール文面の適用](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3430)

- パートナー追加メールの文面をU-114メールとなるようにしました。
  - 併せてメール送信処理で`await`になっていなかった箇所を修正しました。

## レビューポイント
- パートナー追加メールの適用は適切でしょうか？
- 各メール送信処理の修正内容は適切でしょうか？

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2024-01-09 08:00:50 +00:00
makabe.t
6f62a016d4 Merged PR 663: develop不具合対応 
## 概要
[Task3404: develop不具合対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3404)

- ライセンス割り当て通知メールのTOにAdminが入るように修正しました。
- 不要なメソッド`createMailContentFromEmailConfirm`を削除しました。

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

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-12-26 07:24:10 +00:00
水本 祐希
5b97b61966 Merged PR 619: API修正（License Inventoryの計算変更） 
## 概要
[Task3270: API修正（License Inventoryの計算変更）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3270)

- 何をどう変更したか、追加したライブラリなど
License Inventoryの計算方法を変更
→割り当て済みライセンスは件数に含まれないようにする（未使用と再利用可能ライセンスのみ）

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

## 動作確認状況
- ローカルで確認、ユニットテスト

## 補足
LicenseSummaryのユニットテストが旧式であったため、新しい方式でテストを追加しました。
 2023-12-13 02:04:16 +00:00
saito.k
36716dc408 Merged PR 547: 音声ファイルアップロード完了API修正（repository実装含む） 
## 概要
[Task2971: 音声ファイルアップロード完了API修正（repository実装含む）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2971)

- 音声ファイルアップロード完了API修正
  - 自動ルーティング処理を追加
    - authorIDとworktypeの組み合わせでワークフロー（ルーティングルール）を取得し、そのワークフローに従って、タスクのチェックアウト候補を設定する。
  - チェックアウト候補に設定したユーザーに対して通知を行う処理を追加

## レビューポイント
- 自動ルーティング処理を実装しているメソッドのメソッド名はこれでよいか
  - ほかに思いつかなかったので
- AudioOptionItemのentityの定義はあっている？
  - がタスクにあるaudio_file_idに紐づいている感じになっている
- 自動ルーティング処理で失敗したときの挙動は認識あっているか
  - エラーログだけ出してAPIとしては成功とする
- テストケースは足りているか
- 古い形式で記述されていたタスク作成のテストを新しい形で作り替えたが、反映漏れている部分はあるか

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

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

## 補足
- 相談、参考資料などがあれば
 2023-11-07 08:48:53 +00:00
makabe.t
b314fe4b46 Merged PR 513: 次タスク取得API実装 
## 概要
[Task2874: 次タスク取得API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2874)

- 次タスク取得APIとテストを実装しました。

## レビューポイント
- リポジトリからのタスク取得ロジックは適切か
- テストケースは適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-10-26 10:46:15 +00:00
saito.k
f553bfc95b Merged PR 501: strictNullChecks修正①（accounts,auth,Repositoiesのaccounts,common） 
## 概要
[Task2835: 修正①（accounts,auth,Repositoiesのaccounts,common）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2835)

- features
  - accounts
  - auth
- common
- repositories
  - accounts
- 各entity
  - Nullableの項目の`@Column`デコレータに`type`を追加しないとTypeORMがエラーになりテストが通らないので追加
    - https://qiita.com/maruware/items/08c9ad594e14e4ea1497#%E5%95%8F%E9%A1%8C

## レビューポイント
- コメントとして記載

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

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

## 補足
- レビュー完了後、TODOコメント(strictNullChecks対応)は削除します
 2023-10-19 07:13:56 +00:00
saito.k
96848f5e54 Merged PR 499: 修正②（files,licenses , Repositoiesのlicenses） 
## 概要
[Task2836: 修正②（files,licenses , Repositoiesのlicenses）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2836)

- feature
  - files
  - licenses
- repositories
  - licenses
  - users
  - worktypes
- entity
  - licenses
  - audio_files
  - audio_option_item
  - checkout_permission
- アクセストークンをそのままService層に渡している箇所を修正し、必要なパラメータのみ渡すように修正
- クライアントの型生成
  - 割り当て可能ライセンス取得APIのIFが変わったため

## レビューポイント
-

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

## 動作確認状況
- ローカルでテストが通ることを確認

## 補足
- 相談、参考資料などがあれば
 2023-10-19 01:04:14 +00:00
oura.a
0e57d0cb81 Merged PR 471: DBマイグレーション 
## 概要
[Task2805: DBマイグレーション](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2805)

利用規約情報管理テーブルを作成しました。
ユーザーテーブル・ユーザー退避テーブルの同意済み利用規約バージョンカラムを、EULA、DPA用に分割しました。
また、ユーザーテーブルの変更に伴うソースコード修正を実施しました。

## レビューポイント
DB定義の認識が合っているか。

## UIの変更
なし

## 動作確認状況
migration up/downでの動作を確認
既存UTがすべて正常に動作することを確認

## 補足
なし
 2023-10-10 04:59:57 +00:00
makabe.t
81d17fb57e Merged PR 408: API修正（WorktypeID一覧API） 
## 概要
[Task2621: API修正（WorktypeID一覧API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2621)

- Worktype一覧APIでActiveWorkTypeも取得するよう修正しました。
- API IFでプロパティがnullableになっているところをrequiredに修正しました。

## レビューポイント
- ActiveWorktypeの取得に問題はないか
- テストケースは適切か
- IFの修正は問題ないか

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-09-14 09:20:04 +00:00
makabe.t
2dcb1c1f84 Merged PR 396: API実装（オプションアイテム取得） 
## 概要
[Task2592: API実装（オプションアイテム取得）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2592)

- オプションアイテム取得APIとテストを実装しました。

## レビューポイント
- リポジトリの取得ロジックは想定通りか
- テストケースは適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-09-11 08:31:03 +00:00
maruyama.t
7524abbae6 Merged PR 378: API実装（ライセンス発行キャンセルAPI） 
## 概要
[Task2498: API実装（ライセンス発行キャンセルAPI）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2498)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
ライセンス発行をキャンセルするAPIを実装
下位のアカウント情報と、上位のアカウント情報をセットすると、パートナー関係であるかを返す関数を追加
既存のユニットテストのライセンス作成箇所で、注文ID、削除日時、削除注文IDを指定できるように修正
- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）
既存ユニットテストのライセンス作成部分

## レビューポイント
- 特にレビューしてほしい箇所
パートナー関係かどうかを返す箇所、共通的に使いやすいかどうか
14日より経過していた場合の箇所、ライセンスの有効期限の定数を使っているが分けたほうが良いか
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場

## 動作確認状況
- ローカルで確認
■正常系
ライセンス発行のキャンセルが完了できる（第一階層で実行）
ライセンス発行のキャンセルが完了できる（第二階層で実行）
キャンセルした発行の注文状態が発行待ちに戻る
発行されたライセンスは物理削除される
論理削除されていたライセンスは未割当で、削除前の状態に戻る
■異常系
第一、第二階層以外で実行した場合はエラー
キャンセル対象の発行が存在しない場合エラー
キャンセル対象の発行が14日より経過していた場合はエラー
キャンセル対象の発行のライセンスが使われていた場合はエラー
自身のパートナー以外の発行をキャンセルしようとした場合、エラー

## 補足
- 相談、参考資料などがあれば
 2023-09-05 05:17:47 +00:00
makabe.t
0b7d979fae Merged PR 375: API実装（ワークタイプID追加API） 
## 概要
[Task2516: API実装（ワークタイプID追加API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2516)

- ワークタイプ追加APIとテストを実装しました。
  - オプションアイテムも一緒に追加されるように実装しています。
  - ワークタイプの制限のためにカスタムバリデータを実装しています。

## レビューポイント
- 追加時のエラー処理は適切か
- バリデータは適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-09-04 07:08:19 +00:00
makabe.t
2da40e278b Merged PR 371: API実装 
## 概要
[Task2505: API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2505)

- ワークタイプ一覧取得APIとテストを実装しました。
  - ワークタイプテーブルのリポジトリも実装しています。

## レビューポイント
- リポジトリの実装に問題はないか
- テスト実装に不足はないか

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-09-01 00:24:01 +00:00
makabe.t
bba69651fe Merged PR 364: API実装（TypistGroup更新API） 
## 概要
[Task2460: API実装（TypistGroup更新API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2460)

- TypistGroup更新APIとテストを実装しました。

## レビューポイント
- DBの更新ロジックに問題はないか
- テストケースは適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-08-29 08:10:16 +00:00
makabe.t
5b1c3a0e99 Merged PR 361: API実装（TypistGroup取得API） 
## 概要
[Task2461: API実装（TypistGroup取得API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2461)

- TypistGroup取得APIとテストを実装しました。

## レビューポイント
- DBからの取得ロジックに問題はないか
- テストケースは十分か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-08-29 06:25:15 +00:00
湯本 開
89751396bf Merged PR 354: テスト用関数を作成する(Sprint16) 
## 概要
[Task2383: テスト用関数を作成する(Sprint16)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2383)

- WIPレビューで指摘された関数名等を修正
- 汎用関数で問題があった部分を修正
  - データ構築ミスしていた箇所を修正
  - DB的にnullableな所をテスト用関数でも省略できるよう修正
- 各種Account、User作成関数を汎用関数に置き換え

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

## 動作確認状況
- npm run test 成功
 2023-08-28 10:40:55 +00:00
saito.k
d5c756184b Merged PR 352: API実装（TypistGroup追加API） 
## 概要
[Task2428: API実装（TypistGroup追加API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2428)

- IFを修正
  - typistIdsの要素が数値であることをOpenAPIに明記する
  - typistIdsのチェックバリデータを追加・修正
- entity修正
- typistGuroup作成処理を実装
- テスト作成

## レビューポイント
- テストケースは足りているか
- entityの修正に問題はないか
- typistIdsのチェック処理で漏れているものはないか

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

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

## 補足
- 相談、参考資料などがあれば
 2023-08-25 07:11:09 +00:00
saito.k
b8c2640719 Merged PR 335: API修正（アカウント登録）&テスト実装 
## 概要
[Task2369: API修正（アカウント登録）&テスト実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2369)

- リカバリ処理を実装
- テスト追加

## レビューポイント
- テストケースは足りているか
- リカバリ処理を追加したが、漏れはないか

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

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

## 補足
- 相談、参考資料などがあれば
 2023-08-21 09:21:04 +00:00
湯本 開
9803ba4e46 Merged PR 326: テストを最新化（パートナー追加） 
## 概要
[Task2401: テストを最新化（パートナー追加）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2401)

- DBテストに修正
- Utilityを追加
- 実装側の変数名やコメント等を修正
- テストに使用するメールアドレスを一般的に使用するべきドメインに修正
  - 参考:
    - https://zenn.dev/progfay/articles/email-example-com
    - https://qiita.com/suzutsuki0220/items/4ad83ed2e2adbb6507a4
- `Promise<void>` となっていた部分をテスト用に `Promise<{accountId: number}>` に修正
  - 返り値を使用しているのはテスト側のみ

## レビューポイント
- 将来的にBlobStorageやSendMailで失敗したケース等も必要だが、それは異常系実装タスク内でテストが追加される想定なので今回追加していないが認識は合っているか
- 各種修正に対して、疑問点や問題点はないか

## 動作確認状況
- npm run testで成功
 2023-08-18 02:11:09 +00:00
makabe.t
ad969bd2cf Merged PR 323: アカウント登録APIを修正 
## 概要
[Task2353: アカウント登録APIを修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2353)

- アカウント登録時にコンテナを作成するように修正
- ログ追加
- リクエストのバリデータを追加

## レビューポイント
- 処理の流れに問題はないか
- テストケースに不足はないか
- バリデータに問題はないか

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-08-16 09:12:56 +00:00
湯本 開
415a76b6bf Merged PR 325: テストを最新化（アカウント登録） 
## 概要
[Task2397: テストを最新化（アカウント登録）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2397)

- アカウント登録のテストを最新化（元が無かったので実質新規追加）
- テスト時、外部サービスへの依存をMockに差し替える際のモデルケースを追加
  - `Object.defineProperty` を使って強引にメンバ変数をMock用オブジェクトへ上書きする方式
- migrationとEntityの定義内容が食い違っていた部分をmigration側に合わせるよう修正
  - 実質的にEntityの属性指定は通常実行時には使われないと思われたため、部分的にはテスト用に修正（※意図はコメント済）

## レビューポイント
- テスト時、AdB2cServiceやSendGridService等のDB以外のサービスをMockに差し替えて動かしているが、この方式に問題や懸念点はなさそうか
  - 例えば、`/app/dictation_server/src/features/users/test/utility.ts` に定義されている `makeTestingModuleWithAdb2c` のように構築時にoverrideする形式が既にあるが、そちらの方式でやるべきでは？ 等
  - 上記方式ではなく `Object.defineProperty` で良いのでは？という提案を行っているのは、以下の理由
     - モジュール構築時に指定する方法だと依存してるサービス数によっては構築時の指定パラメータが膨大になってしまう懸念がある（3~4個のサービスと連携している場合、`makeTestingModuleWithXXX(...)` の引数指定がえらいことになりそう）
     - `Object.defineProperty` 形式だとテストケース内で必要なサービスを1個1個上書きしていく事が可能なので、汎用関数として切り出して各Serviceで利用できる
- 現状のテストケースとして妥当か
  - 例: SendGridの送信に失敗した場合は最終的にはリカバリを行う実装が入るが、現段階ではリカバリ処理がないためテスト記述なし
- Entityの属性を修正したが問題なさそうか、特にテスト用に指定した箇所があるが問題ないか

## 動作確認状況
- npm run testで成功
 2023-08-14 06:52:13 +00:00
oura.a
356f5fe346 Merged PR 320: API実装（ライセンス割り当てAPI）_履歴以外 
## 概要
[Task2362: API実装（ライセンス割り当てAPI）_履歴以外](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2362)

ライセンス割り当てのAPIを作成しました。
※ライセンス割り当て履歴テーブルが絡む処理は別タスクでの対応となるので、ここでは未実装です。

## レビューポイント
なし

## UIの変更
なし

## 動作確認状況
ローカルでUT、動作確認実施済み

## 補足
なし
 2023-08-10 08:26:44 +00:00
maruyama.t
01d20df628 Merged PR 310: API実装（ライセンス発行API） 
## 概要
[Task2341: API実装（ライセンス発行API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2341)

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

## レビューポイント
- 特にレビューしてほしい箇所
第一階層の場合とそれ以外の処理の違い

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

## 動作確認状況
- ローカルで確認
①第一階層→第二階層から100件のライセンス注文を行い、ライセンス発行APIを呼び出し、
ライセンス注文テーブルの注文状態が発行済になることを確認。
ライセンステーブルが登録されることを確認。
DBの状態は、
第二階層のStock Licensesが100になることを確認。
登録・更新されたデータは処理単位で現在時刻が同じものであること
②第二階層→第三階層から90件のライセンス注文を行い、ライセンス発行APIを呼び出し、
ライセンス注文テーブルの注文状態が発行済になることを確認。
ライセンステーブルが登録されること（第三階層に対してUnAllocated）を確認。
ライセンステーブルが更新されること（第二階層に対してDeleted）を確認。
DBの状態は、
第二階層のStock Licensesが10になることを確認。
Deletedに更新されたライセンスについて、更新順がライセンスIDの順になっていることを確認。
第三階層のStock Licensesが90になることを確認。
登録・更新されたすべてのデータは、処理単位で現在時刻が同じものであることを確認。
③第五階層で呼び出した場合、エラーになることを確認。
④第４階層→第５階層に100件のライセンス注文を行い、ライセンス発行APIを呼び出し、
ライセンス数不足エラーとなることを確認。
⑤②で行ったライセンス発行APIを呼び出し、
ライセンス発行済みエラーとなることを確認。
⑥DBを起動していない状態で、ライセンス発行APIを呼び出し、
Internal Server Error 500が返却されることを確認。
## 補足
- 相談、参考資料などがあれば
 2023-08-08 10:01:02 +00:00
oura.a
98e9937d9d Merged PR 313: 第五階層のShortageについて数値をlicensesummaryのものと同じにする 
## 概要
[Task2283: 第五階層のShortageについて数値をlicensesummaryのものと同じにする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2283)

以下のプルリクで受けた指摘点を修正しました。
https://dev.azure.com/ODMSCloud/ODMS%20Cloud/_git/ODMS%20Cloud/pullrequest/273
具体的な修正内容は以下になります。
・子アカウントの数分だけDBクエリしにいく問題を修正
・有効期限の閾値となる日時を算出するclassを追加

## レビューポイント
ご指摘いただいた問題がこの対応で解消されているか。

## UIの変更
なし

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

## 補足
なし
 2023-08-08 08:11:56 +00:00
masaaki
2a54db7c8d Merged PR 294: issueDateがnullの場合画面上初期値が入ってしまう 
## 概要
[Task2324: issueDateがnullの場合画面上初期値が入ってしまう](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2324)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- issued_atがnullの場合にnew Dateして初期値が返却される問題を修正しました。値がnullの場合はそのままnullを返却しています。
- 併せて、整形されていないソースがあったので修正しています（accounts.service.mock.ts）
- 影響範囲（他の機能にも影響があるか）
  - 同様に日付を扱う項目としてordered_atがありますが、こちらはnot null項目のため対応は入れていません。

## レビューポイント
- 特筆する点は無し

## UIの変更
- 無し

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

## 補足
- 相談、参考資料などがあれば
 2023-08-01 11:37:20 +00:00
水本 祐希
3ffa45e179 Merged PR 277: API実装（注文履歴取得API） 
## 概要
[Task2261: API実装（注文履歴取得API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2261)

- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）

## レビューポイント
なし

## UIの変更

## 動作確認状況
-ローカルとポストマンにて確認
モック動作確認は１つのAPIから値を返す単純なもののため行わず。

## 補足
 2023-08-01 10:10:58 +00:00
masaaki
70cb66e517 Merged PR 254: API実装（パートナーライセンス情報取得API） 
## 概要
[Task2213: API実装（パートナーライセンス情報取得API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2213)

- パートナーライセンス情報取得APIについて実装を行いました
- このPull Request機能としては対象外。気づいたので直した部分
　・accounts.service.mock.ts
　　→整形されていなかったのを修正、不要な参照を削除
　・accounts.service.spec.ts
　　→整形されていなかったのを修正、不要な参照（userInfo）を削除
　・license.entity.ts
　　→テスト実施のため必須項目の定義を追加
　・dev-database-rg.json
　　→developからのマージだが、差分ファイルとして出てしまっている。
　　　（誤ってコミットのstagingを解除してしまい、後からpull操作したことが影響していると思われる）
- 影響範囲（他の機能にも影響があるか）
　→特になし

## レビューポイント
- MISOチームメンバについては、「このPull Request機能としては対象外。気づいたので直した部分」の確認をお願いします。整形や不要な参照はMISOで作りこんだものです。
- accounts.repository.service.tsについて、DRYの観点で一部処理をprivateなサブルーチンとしています（getAccountLicenseOrderStatus）。そもそもそういうことをやってよいか、分ける際のやりかた（引数など）について妥当か見ていただきたいです。
- repositoryからの戻り値型の名称についてわかりやすいかどうか。「PartnerLicenseInfoのうちリポジトリから取得した情報」として「PartnerLicenseInfoFromRepository」としていますが、Fromとかつけない方がよいのでは、とか呼び元を意識しすぎた名称になってる、とか気にしています。

## UIの変更
- 無し

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

## 補足
- 相談、参考資料などがあれば
 2023-07-25 02:08:20 +00:00
Kentaro Fukunaga
c6f63c962c Merged PR 258: getRelationsを外部連携テスト用に内部実装 
## 概要
[Task2257: 実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2257)

- AuthorIDは本物の値を返却し、それ以外はダミーの固定値を返却する実装を入れました。
- npm run formatをかけました

## レビューポイント
- 気になる点ないか：To真壁くん
- npm run formatの結果共有：Toガンさん（パートナー追加PBIのところで出ていたので）

## 動作確認状況
- ローカルでPostmanを使用し、Authorのときとそうでないとき両方確認
 2023-07-23 23:41:09 +00:00
maruyama.t
9739942bdf Merged PR 238: Revert 'Revert 'API実装（パートナーアカウント追加API）'' 
## 概要
[Task2157: API実装（パートナーアカウント追加API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2157)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
パートナーアカウント追加APIを実装しました。

- このPull Requestでの対象/対象外
認証メール送信後のフローは既存機能のため対象外

- 影響範囲（他の機能にも影響があるか）
既存のaccounts.service.spec.tsのテスト

## レビューポイント
- 特にレビューしてほしい箇所
エラー判定に過不足ないか

## UIの変更
なし

## 動作確認状況
- ローカルで確認
Azureに管理者ユーザが追加されたこと、認証メールが送信されてくることを確認。
（対象外だが、認証後に追加されたアカウントでログインできることを確認）

## 補足
- 相談、参考資料などがあれば
一度間違えてCompleteにしてしまったので、
Reverts !225
差分を戻すプルリクをCompleteにして出し直させていただいております。
Reverts !237
 2023-07-18 05:01:17 +00:00
maruyama.t
f8a3b1798c Merged PR 237: Revert 'API実装（パートナーアカウント追加API）' 
## 概要
[Task2157: API実装（パートナーアカウント追加API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2157)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
パートナーアカウント追加APIを実装しました。

- このPull Requestでの対象/対象外
認証メール送信後のフローは既存機能のため対象外

- 影響範囲（他の機能にも影響があるか）
既存のaccounts.service.spec.tsのテスト

## レビューポイント
- 特にレビューしてほしい箇所
エラー判定に過不足ないか

## UIの変更
なし

## 動作確認状況
- ローカルで確認
Azureに管理者ユーザが追加されたこと、認証メールが送信されてくることを確認。
（対象外だが、認証後に追加されたアカウントでログインできることを確認）

## 補足
- 相談、参考資料などがあれば

Reverts !225
 2023-07-14 04:19:24 +00:00
maruyama.t
3ddb010675 Merged PR 225: API実装（パートナーアカウント追加API） 
## 概要
[Task2157: API実装（パートナーアカウント追加API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2157)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
パートナーアカウント追加APIを実装しました。

- このPull Requestでの対象/対象外
認証メール送信後のフローは既存機能のため対象外

- 影響範囲（他の機能にも影響があるか）
既存のaccounts.service.spec.tsのテスト

## レビューポイント
- 特にレビューしてほしい箇所
エラー判定に過不足ないか

## UIの変更
なし

## 動作確認状況
- ローカルで確認
Azureに管理者ユーザが追加されたこと、認証メールが送信されてくることを確認。
（対象外だが、認証後に追加されたアカウントでログインできることを確認）

## 補足
- 相談、参考資料などがあれば
 2023-07-14 04:15:21 +00:00
maruyama.t
4bbd9b371d Merged PR 214: 第五階層の画面レイアウトを最新の状況にする 
## 概要
[Task2034: 第五階層の画面レイアウトを最新の状況にする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2034)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
クライアント側の画面表示内容の修正。
AccountLock→StorageAvailable
サーバ側の処理でAccountLockと定義している変数名を適切な名称に修正。
- 影響範囲（他の機能にも影響があるか）
openapi.json変更に伴い、api.tsがdev環境の最新版になっています。

- 本PBIの対象外
第五階層用ライセンス情報画面以外の機能
scssファイルの修正箇所は、次回デザイナさんの修正時に正式に展開される想定です。

## レビューポイント
- 特にレビューしてほしい箇所

## 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/Task2034?csf=1&web=1&e=gfSwl3
## 動作確認状況
ローカル環境で画面の表示内容が変更されていることを確認

## 補足
- 相談、参考資料などがあれば
 2023-07-10 04:03:03 +00:00
makabe.t
4d52da8d18 Merged PR 163: タイピストグループ取得API実装 
## 概要
[Task1946: タイピストグループ取得API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1946)

- ユーザーグループを取得するAPIを実装しました。
  - 別で対応予定のテストをコメントアウトしています

## レビューポイント
- 取得内容に問題ないか
- ライセンス取得のテストをコメントアウトしているが問題ないか

## UIの変更
なし

## 動作確認状況
- ローカルで確認
 2023-06-22 00:36:37 +00:00
makabe.t
ef17e4ff0b Merged PR 167: タイピストユーザー取得API実装 
## 概要
[Task1931: タイピストユーザー取得API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1931)

- アカウント内のタイピストユーザーを取得する処理を実装しました。

## レビューポイント
- ADB2Cからのユーザー取得、マージは適切か

## UIの変更
なし

## 動作確認状況
- ローカルで確認
 2023-06-20 06:16:19 +00:00
maruyama.t
f0e0d5e6b1 Merged PR 162: API IF修正（LicenseSummaryInfo） 
## 概要
[Task1961: API IF修正（LicenseSummaryInfo）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1961)

- 何をどう変更したか、追加したライブラリなど
API実装時に一時的にLicenseSummaryInfo2と命名していた定義を、LicenseSummaryInfoに修正。
それに伴い、既存のLicenseSummaryInfoはレスポンスに直接記載するよう修正。
openapi.jsonを再生成し、画面側のパラメータの取得の記載を修正。

- このPull Requestでの対象/対象外
storageSizeとusedSizeの取得はPBI対象外
- 影響範囲（他の機能にも影響があるか）
なし

## レビューポイント
- クライアント側に対する修正漏れがないか。

## UIの変更
なし

## 動作確認状況
- ローカルでAPIの戻り値の確認と、画面にその値が表示されていることを確認。
![image.png](https://dev.azure.com/ODMSCloud/6023ff7b-d41c-4fa7-9c6f-f576ba48c07c/_apis/git/repositories/302da463-a2d7-40f9-b2bb-6e8edf324fa9/pullRequests/162/attachments/image.png)
![image (2).png](https://dev.azure.com/ODMSCloud/6023ff7b-d41c-4fa7-9c6f-f576ba48c07c/_apis/git/repositories/302da463-a2d7-40f9-b2bb-6e8edf324fa9/pullRequests/162/attachments/image%20%282%29.png)
## 補足
- 相談、参考資料などがあれば
 2023-06-19 04:06:44 +00:00
maruyama.t
fd69541e1a Merged PR 142: API実装（第五階層用ライセンス情報取得API） 
## 概要
[Task1846: API実装（第五階層用ライセンス情報取得API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1846)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
accounts.controller.tsからaccountService.getLicenseSummaryを呼び出す。
一度のトランザクションで処理を行うよう、serviceとrepositoryをリファクタリング。
license.entity.tsにはライセンス系のテーブルで不足していたエンティティを追加。

- このPull Requestでの対象/対象外
Storage Sizeの値はPBI1203では対象外のため0固定
Used Sizeの値はPBI1203では対象外のため0固定
LicenseSummaryInfo2と定義している個所は、別タスクで修正します。
[タスク 1961: API IF修正（LicenseSummaryInfo）](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%2011-1?workitem=1961)

- 影響範囲（他の機能にも影響があるか）
新規追加のため、なし

## レビューポイント
- 特にレビューしてほしい箇所

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
PostmanでAPI実行。
各ライセンス数値が期待通りの結果であることを確認。
![image.png](https://dev.azure.com/ODMSCloud/6023ff7b-d41c-4fa7-9c6f-f576ba48c07c/_apis/git/repositories/302da463-a2d7-40f9-b2bb-6e8edf324fa9/pullRequests/142/attachments/image.png)

## 補足
テスト内容は、添付のテストデータを参照ください。
[テストデータ.xlsx](https://dev.azure.com/ODMSCloud/6023ff7b-d41c-4fa7-9c6f-f576ba48c07c/_apis/git/repositories/302da463-a2d7-40f9-b2bb-6e8edf324fa9/pullRequests/142/attachments/%E3%83%86%E3%82%B9%E3%83%88%E3%83%87%E3%83%BC%E3%82%BF.xlsx)
 2023-06-15 08:56:03 +00:00
oura.a
201b6be260 Merged PR 150: API実装（自アカウント情報取得API） 
## 概要
[Task1954: API実装（自アカウント情報取得API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1954)

・アクセストークンからユーザ情報を取得する処理を追加。
（本タスクで返しているのはアカウントIDのみ）
・本タスクのレビュー対象は以下となります。
　・accounts.servicce
　　　getMyAccountメソッド
　・accounts.controller
　　　@Get('me')
　・accounts.service.spec
　　　'アクセストークンからユーザ情報を取得する'
　　　'ユーザ情報が取得できない場合エラーとなる'
・別タスクで作成したテストコードを取り込むためマージしたところ、controllerの別タスク箇所でエラーが出ていたので、一時的にコメントアウトしています。本タスクの挙動には影響ありません。

## レビューポイント
処理に過不足がないか。
エラーハンドリングが適切か。

## UIの変更
なし

## 動作確認状況
ローカルでユニットテスト実施済み。
ローカルで外部からAPIを実行し、動作確認済み。

## 補足
なし
 2023-06-13 07:39:10 +00:00