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

181 Commits

Author SHA1 Message Date
saito.k
f1425bc509 エラーをもとに戻す 2023-08-22 19:30:06 +09:00
saito.k
6fed7738a7 エラーをもとに戻す 
漏れてた確認のためにエラーを追加
 2023-08-22 19:10:21 +09:00
saito.k
86f9bb76e6 無理やりエラーを発生させる 2023-08-22 18:45:28 +09:00
maruyama.t
1843844c48 Merged PR 321: API実装（割り当て可能ライセンス取得API） 
## 概要
[Task2361: API実装（割り当て可能ライセンス取得API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2361)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）
メモリDB上にライセンスを作成するメソッドの有効期限を指定可能にしたため、
既存テストで特に指定していなかった箇所（デフォルトでnull）はnullを設定。

## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
取得レコードのソートの関係上、EntityManagerではなく、QueryBuilderでの実装としているが、問題ないか。

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

## 動作確認状況
- ローカルで確認
・正常系
①メモリDB上に5件の有効なライセンスを作成。無効なライセンスを４件作成。
　※内2件が有効期限null、2件が同一の有効期限、1件が最も遠い有効期限
　有効なライセンスのみ取得できることを確認。
　serviceの戻り値として、ソートされて取得できることを確認。
　（nullが最優先、有効期限の降順、同一の有効期限のものはidで昇順）
②結果が0件（別のアカウントにはライセンスが存在する）の場合（POSTMAN)
　正常終了し、空の配列を返却すること。
　別のアカウントのライセンスを取得しないこと。
・異常
①tier5以外のアカウントで実行（POSTMAN）し、"Token authority failed Error."エラーになることを確認。
②コンテナを停止して実行（POSTMAN）し、"Internal Server Error."エラーとなることを確認。
・他
①ログがポリシに従って出ていることの確認
## 補足
- 相談、参考資料などがあれば
 2023-08-22 08:55:56 +00:00
masaaki
417ba17d13 Merged PR 340: API-IF実装 
## 概要
[Task2445: API-IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2445)

ライセンス割り当て解除API-IFを作成しました。

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

## UIの変更
なし

## 動作確認状況
Swagger UIにより確認済み
 2023-08-22 08:30:01 +00:00
maruyama.t
bdd10aabf6 Merged PR 324: API修正（ライセンス割り当てAPI）_履歴部分 
## 概要
[Task2366: API修正（ライセンス割り当てAPI）_履歴部分](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2366)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
allocateLicenseの割り当て履歴テーブルのレコードを作成するロジックを追加しました。
createLicenseの引数を追加して、種別を指定して作成できるようにしました。
既存テストで項目の値などでべた書きしていた箇所を、定数を使うよう修正しました。
- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）
createLicenseの引数を追加して、種別を指定して作成できるようにしました。
ユニットテストでcreateLicenseを使っている既存テストは再テスト済み

## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
lisence_allocation_historyテーブルのスキーマを変更しています。
プライマリとしてのidを追加（ユーザIDになっていた）、
allocate_typeをallocatedに変更して、boolの値で管理するように変更（割り当て解除or割り当てしかもたないので）。

## UIの変更
なし

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

## 補足
- 相談、参考資料などがあれば
 2023-08-22 06:06:22 +00:00
saito.k
f16a5414dc Merged PR 342: API修正（パートナー追加）&テスト実装 
## 概要
[Task2399: API修正（パートナー追加）&テスト実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2399)

- パートナー追加にリカバリ処理を追加
- テスト修正

## レビューポイント
- テストケースは足りているか
- リカバリ処理に漏れはないか

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

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

## 補足
- 相談、参考資料などがあれば
 2023-08-22 02:06:53 +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
saito.k
76ed87d82a Merged PR 337: API修正（ユーザー追加）&テスト実装 
## 概要
[Task2398: API修正（ユーザー追加）&テスト実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2398)

- ユーザー追加のリカバリ処理を実装
  - ADB2Cに追加したユーザー削除
  - DBに追加したユーザー削除

## レビューポイント
- リカバリ処理に不足はないか
- ログはこれでよいか
- テストケースはこれで足りているか
  - ADB2Cのユーザー削除:OK , DB上のユーザーは削除:NO のケースはいるか等

## UIの変更

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

## 補足
- 相談、参考資料などがあれば
 2023-08-21 08:25:57 +00:00
makabe.t
5690b66e41 Merged PR 338: トライアルライセンスの期限が30日後になっていないので対応 
## 概要
[Task2442: トライアルライセンスの期限が30日後になっていないので対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2442)

- トライアルライセンスの有効期限をライセンス割り当てと同様にDateのラップクラス内で設定するように修正しました。

## レビューポイント
- 修正内容は適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-08-21 06:24:53 +00:00
湯本 開
493dbadb8e Merged PR 339: パートナー追加のテストが増えている問題を修正 
## 概要
[Task2443: パートナー追加のテストが増えている問題を修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2443)

- 重複しているテストを削除
- 重複していないテスト(blob storageで失敗したケース)を移動

## レビューポイント
- 対処として問題ないか

## 動作確認状況
- ローカル環境でnpm run test で成功
 2023-08-21 01:51:04 +00:00
makabe.t
deb753b40d Merged PR 329: API実装（BlobStorageのコンテナを削除する） 
## 概要
[Task2395: API実装（BlobStorageのコンテナを削除する）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2395)

- BlobStorageコンテナを削除するgatewayメソッドを実装しました。

## レビューポイント
- 処理内容に問題はないか
  - コンテナの有無を見て結果を返すように実装したが作りとして問題はないか

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-08-18 08:38:10 +00:00
makabe.t
f79e45be66 Merged PR 336: pipelineテスト失敗対応 
## 概要
[Task2439: pipelineテスト失敗対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2439)

- パイプライン上でパートナー追加のテストが失敗する問題を修正しました。
  - 上書きするべきメソッドが違っていたため、本物メソッドが呼ばれてCICD環境の環境変数にない秘密鍵を使おうとして失敗していました。
  - テストで上書きするメソッドを正しいものに差し替えました。

## レビューポイント
- 対応内容に問題はないか

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
  - local.envの秘密鍵をコメントアウトしてテスト実行でエラー再現、修正適用で解消されることを確認

## 補足
- 相談、参考資料などがあれば
 2023-08-18 02:48:10 +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
502c31bcac Merged PR 334: パートナー追加APIにBlobストレージ作成処理を追加 
## 概要
[Task2421: パートナー追加APIにBlobストレージ作成処理を追加](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2421)

- パートナー追加APIにblobストレージ作成処理を追加しました。

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

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-08-18 01:01:26 +00:00
makabe.t
adbd57bca6 Merged PR 333: 動作確認不具合修正 
## 概要
[Task2419: 動作確認不具合修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2419)

- ディーラーをURLで指定して見つからなかった場合にスナックバーでメッセージを表示するように修正しました。

## レビューポイント
- UIの挙動として問題ないか
- メッセージは適切か

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

## 動作確認状況
- ローカルで確認
 2023-08-17 08:08:28 +00:00
saito.k
073e28d67d Merged PR 332: API実装（Azure AD B2Cの管理者ユーザーを削除する） 
## 概要
[Task2393: API実装（Azure AD B2Cの管理者ユーザーを削除する）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2393)

- B2Cのユーザーを削除する処理を実装

## レビューポイント
- 実装で考慮漏れている部分はないか
- ログの内容はこれでよいか

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

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

## 補足
- 相談、参考資料などがあれば
 2023-08-17 05:50:21 +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
saito.k
ba81bc5cb8 Merged PR 328: API実装（DBからアカウント・管理者ユーザーを削除する） 
## 概要
[Task2394: API実装（DBからアカウント・管理者ユーザーを削除する）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2394)

- アカウント削除と紐づくユーザーを削除する処理を実装

## レビューポイント
- アカウントに紐づくユーザーを全削除しているが問題ないか
  - ユーザーIDとかを指定できた方が良い？
- このタスクで漏れている作業はあるか

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

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

## 補足
- 相談、参考資料などがあれば
 2023-08-16 08:45:41 +00:00
湯本 開
ae88da1728 Merged PR 331: UnitTestユーティリティの内部的な仕組みを改善する 
## 概要
[Task2418: UnitTestユーティリティの内部的な仕組みを改善する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2418)

- 上書きするメソッド名を型から取得するよう修正

## レビューポイント
- 修正内容として妥当か
- 懸念点がないか

## 動作確認状況
- npm run testで成功
 2023-08-16 07:51:14 +00:00
湯本 開
924106d608 Merged PR 330: pipelineでテストが失敗する現象を修正する 
## 概要
[Task2416: pipelineでテストが失敗する現象を修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2416)

- テストの仕組み改善によって、SendGridの上書き部分の挙動が変わっていたことに対する対処が漏れていたものに対処

## レビューポイント
- 情報共有
- 追加したSendMailの特別扱いは妥当であるか

## 動作確認状況
- npm run testで成功
 2023-08-16 05:28:58 +00:00
湯本 開
704a5aafc2 Merged PR 327: テストを最新化（ユーザー追加） 
## 概要
[Task2400: テストを最新化（ユーザー追加）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2400)

- 新規ユーザー追加のユニットテストをDBを利用するテストへ置き換え
- API呼び出しやRepository呼び出しをMockする仕組みを改善
  - オブジェクト全てを上書きするのではなく、特定のメソッドのみを上書きする形に修正
    - これにより、他のDBアクセスは正常にSQLiteへのアクセスが成功しつつ、特定のDBアクセスのみMockにできる

## レビューポイント
- テストケースは足りているか
- Mockする仕組みを改善したが、修正内容は問題ないか

## 動作確認状況
- npm run test で成功
 2023-08-16 02:54:32 +00:00
saito.k
86d11e7447 Merged PR 314: ユーザー確認API修正 
## 概要
[Task2351: ユーザー確認API修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2351)

- アカウント登録後のユーザー検証時に、トライアルライセンスを作成する処理を追加
- テスト修正

## レビューポイント
- トライアルライセンスの有効期限の設定はこれでよいか
- トライアルライセンスの登録内容は正しいか
- テストケースは足りているか、確認するパラメータに不足はないか

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

## 補足
- `npm run format`で複数ファイルが更新されていますが、修正を加えたのは以下のファイルです。
  - /dictation_server/src/constants/index.ts
  - /dictation_server/src/features/users配下のファイル
  - /dictation_server/src/repositories/users/users.repository.service.ts
 2023-08-16 02:13:10 +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
oura.a
3b785e97aa Merged PR 317: API-IF実装 
## 概要
[Task2360: API-IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2360)

ライセンス割り当てのAPI-IFを作成しました。

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

## UIの変更
なし

## 動作確認状況
Swagger UIで反映されていることを確認

## 補足
なし
 2023-08-09 08:46:01 +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
saito.k
712efb8bb7 Merged PR 309: Dealer取得API実装 
## 概要
[Task2348: Dealer取得API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2348)

- アカウント登録画面で呼ぶDealer取得APIを実装

## レビューポイント
- 取得条件は正しいか
- テストケースは足りているか
  - DBエラーによるエラーケースは作成しても効果が薄いと思ったため作成していません

## UIの変更

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

## 補足
- 相談、参考資料などがあれば
 2023-08-08 00:28:27 +00:00
saito.k
d70b471d55 Merged PR 311: Task一覧取得でstatus未設定の場合に０件となってしまうので対応 
## 概要
[Task2367: Task一覧取得でstatus未設定の場合に０件となってしまうので対応](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環境で確認など

## 補足
- 相談、参考資料などがあれば
 2023-08-07 08:59:38 +00:00
oura.a
b59a5df44c Merged PR 303: PBI1224_dev動作確認指摘対応 
## 概要
[Task2325: PBI1224_dev動作確認指摘対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2325)

前PBIで指摘のあった箇所を修正しました。
・IssueDateが空白の場合「-」とするように修正
・ページ上部の「EFGI Legal」に会社名を入れるように修正
・上記に伴い、アカウント情報取得API（accounts/me）で会社名を取得できるように修正
・ページネーションの位置を修正
・ページネーション部分に表示されている「xxx licenses」を「xxx histories」に修正
・ページネーション周りの挙動を修正

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

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

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

## 補足
なし
 2023-08-07 01:52:03 +00:00
saito.k
a1a91207e4 Merged PR 308: IF実装（Dealer取得API） 
## 概要
[Task2352: IF実装（Dealer取得API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2352)

- ディーラー取得APIのIFを実装

## レビューポイント
- レスポンスとして返却するデータの認識はあっているか

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

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

## 補足
- 相談、参考資料などがあれば
 2023-08-07 01:25:26 +00:00
Kentaro Fukunaga
e279957392 Merged PR 307: 音声ファイルZIPファイルをコンテナ配下に平置きするよう修正する 
## 概要
[Task2346: 音声ファイルZIPファイルをコンテナ配下に平置きするよう修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2346)

- 音声ファイルZIPのアップロード/ダウンロードを、コンテナ直下で操作するよう修正しました
   - Typistによる中間生成物アップロード対応のため
   - ツール側も修正しました。（詳細は補足参照）
- タスク一覧に表示するファイル名から.zipを抜きました（顧客要望）

## レビューポイント
- 対応に抜け漏れ無いか

## 動作確認状況
- ローカルでアップロード/ダウンロードできることを確認しました
- タスク一覧も確認しました

## 補足
- ツール側の修正もあります。
   - DL時にファイル名を抜き出しているところがありますが、そこを本修正に合わせて修正してます。
https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/_git/OMDSDictation/pullrequest/25?_a=overview
 2023-08-07 00:51:07 +00:00
saito.k
4565d59a51 Merged PR 301: ユーザー追加修正（API/画面） 
## 概要
[Task2327: ユーザー追加修正（API/画面）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2327)

- ユーザー追加API
  - リクエストにencryption,encryptionPassword,promptを追加
  - リクエストからtypistGroupIdを削除
  - ロールに応じてDBに保存するデータを作成する処理を追加
  - リクエストパラメータのバリデーションチェックを追加
- ユーザー追加画面
  - TypistGroupの選択欄を削除
  - RoleがAuthorの場合、encryption,encryptionPassword,promptを追加

## レビューポイント
- 修正に不足はないか
- 画面のユーザー追加処理を引数を渡さずにstoreから取得するようにしたが問題ないか
- 画面に必要な値をまとめて取るようにしたが問題ないか
- デザインに差異はないか

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

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

## 補足
- 相談、参考資料などがあれば
 2023-08-07 00:11:54 +00:00
makabe.t
338d6b88a9 Merged PR 299: ユーザー編集API実装 
## 概要
[Task2317: ユーザー編集API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2317)

- ユーザー編集APIとテストを実装しました。

## レビューポイント
- リポジトリでのチェックは適切か
- バリデータの適用は適切か
- テストケースは十分か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-08-04 05:39:33 +00:00
maruyama.t
34ad2e489d Merged PR 304: API-IF実装 
## 概要
[Task2338: API-IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2338)

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

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

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

## 動作確認状況
- ローカル環境で確認
（POSTMANによる正常時の戻り値nullと各Error）

## 補足
- 相談、参考資料などがあれば
 2023-08-04 02:08:42 +00:00
oura.a
eb8c31cf05 Merged PR 300: view detailsで第5階層の情報を参照した際、ログインアカウントではなく第5のアカウント情報を表示できるようにする 
## 概要
[Task2335: view detailsで第5階層の情報を参照した際、ログインアカウントではなく第5のアカウント情報を表示できるようにする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2335)

パートナーライセンス一覧画面から「viewDetails」ボタンを押下した際の挙動を実装しました。
具体的な対応は以下になります。
・第五以外でも第五の情報を取得できるよう、ライセンス情報APIを呼び出す際の処理を修正
・ライセンス情報APIのRoleGuardからtierの制限を削除
・第五以外から第五の情報を表示する際に、「Order License」「Activate License Key」ボタンを表示しないよう修正
・第五以外から第五の情報を表示する際に、「Return」ボタンを追加

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

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

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

## 補足
なし
 2023-08-03 08:56:39 +00:00
saito.k
6a2d2eabde Merged PR 298: ユーザー一覧修正（API・画面） 
## 概要
[Task2318: ユーザー追加・一覧修正（API・画面）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2318)

- API
  - ユーザー一覧取得APIのレスポンスにUserIDを追加
  - OpenAPI生成

- 画面
  - コード自動生成
  - storeにidを含めたユーザー一覧情報を格納する

## レビューポイント
- 修正内容に認識違いはないか

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

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

## 補足
- ユーザー追加については別タスクで対応
 2023-08-03 00:51:03 +00:00
makabe.t
f1684a5616 Merged PR 297: API IF実装 
## 概要
[Task2319: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2319)

- ユーザー編集APIのI/Fを実装しました。
  - openapi.json生成

## レビューポイント
- プロパティは適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認　Swagger
 2023-08-02 07:47:31 +00:00
makabe.t
794099f37d Merged PR 292: 外部連携APIにログを入れ込む（強化） 
## 概要
[Task2294: 外部連携APIにログを入れ込む（強化）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2294)

- 外部連携APIのログを強化しました。
  - contextオブジェクトで操作者情報を渡すようにしています。
- ログポリシーに従って追加しています。
  - [ログポリシー](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_wiki/wikis/OMDSDictation_wiki/223/%E3%83%AD%E3%82%B0%E3%83%9D%E3%83%AA%E3%82%B7%E3%83%BC)

## レビューポイント
- 出力内容に過不足はないか
- ログ追加対象に過不足はないか。
- contextで操作者情報を渡しているが想定通りか

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-08-02 01:07:02 +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
maruyama.t
b5183994e1 Merged PR 291: 画面実装（注文履歴画面） 
## 概要
[Task2260: 画面実装（注文履歴画面）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2260)

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

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-08-01 01:55:17 +00:00
maruyama.t
8be20b7ca8 Merged PR 269: 画面実装（注文履歴画面） 
## 概要
[Task2260: 画面実装（注文履歴画面）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2260)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
Order Cancel/Issue/Issue Cancel機能について、はこちらで実装予定[注文履歴画面_画面遷移対応](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%2014-1?workitem=2273)
Return機能の挙動
userSlice.tsの修正部分（openapi.jsonからapi.ts生成時に画面側のDomainが原因でエラーになるため、一時的に修正）
- 影響範囲（他の機能にも影響があるか）

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

## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
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/Task2260?csf=1&web=1&e=PBcZUw

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

## 補足
- 相談、参考資料などがあれば
 2023-08-01 01:35:19 +00:00
masaaki
3121e1e3ba Merged PR 273: 第五階層のShortageについて数値をlicensesummaryのものと同じにする 
## 概要
[Task2283: 第五階層のShortageについて数値をlicensesummaryのものと同じにする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2283)

- shortageについて、第五階層の場合は、「（{Stock license} - {Issue Requested}）」ではなく、第五階層のライセンス画面と同様に「{有効期限が15日以上または未設定の、未割当または割り当て解除済みライセンス数} - {有効期限が14日以内のライセンス数}」を表示するように修正
- 影響範囲（他の機能にも影響があるか）
第五階層のShortageを取得する処理について再利用可能な形に切り出しています。そのためgetLicenseSummaryInfoにも修正を行っています。

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

## UIの変更
- 無し

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

## 補足
- 相談、参考資料などがあれば
 2023-07-31 10:03:11 +00:00
makabe.t
7ac578d4cc Merged PR 280: ユーザー一覧API修正 
## 概要
[Task2231: ユーザー一覧API修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2231)

- ユーザー一覧取得APIを修正しまし、テスト実装を修正しました。
  - グループ情報、ライセンス情報を一緒に取得するよう修正
  - ADB2Cからのユーザー取得を一括取得に修正
※ユーザー以外のテスト変更はユーザーテーブルのスキーム変更に伴うものです。

## レビューポイント
- 取得内容に問題はないか
  - グループとライセンスはDBからリレーションで一緒に取得しているが処理的に問題はないか
  - ADB2Cからのメールアドレス取得のためにidentitiesパラメータを追加したが問題はないか
- テスト項目に問題はないか

## UIの変更
なし

## 動作確認状況
- ローカルで確認
 2023-07-31 08:24:51 +00:00
makabe.t
dc32b5ac67 Merged PR 286: 不要なFCM向け処理を削除 
## 概要
[Task2304: 不要なFCM向け処理を削除](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2304)

- 通知処理についてFCM向けの処理は仮検証コードなので削除しました。

## レビューポイント
- 削除内容に問題はないか

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-07-31 08:22:33 +00:00
x.yumoto.k
5fccc37dee Revert "Revert "Merged PR 282: エラーの挙動について確認する"" 
This reverts commit 6ad3a81e0823923dd084e8becc59ad2398fca19c.
 2023-07-31 16:07:34 +09:00
makabe.t
c8ecc32b80 Merged PR 268: Typist割り当てAPIに通知処理を追加 
## 概要
[Task2220: Typist割り当てAPIに通知処理を追加](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2220)

- タスクの割り当て候補変更APIに通知処理を追加しました。
  - 割り当てられたTypist全員を対象に通知を送るように実装しています。

## レビューポイント
- 対象となるユーザーは認識通りか
- 通知送信部分に問題はないか
  - tagは20個までのようなので分割して送信するようにしています。

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
  - テストが通ること
  - 割り当て変更が問題ないこと
 2023-07-31 01:50:49 +00:00
x.yumoto.k
dbb61274a1 Revert "Merged PR 284: エラーの挙動について確認する" 
This reverts commit 21ecc1db6b139882848fe573489ba83c442dec81.
 2023-07-28 17:17:00 +09:00