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

533 Commits

Author SHA1 Message Date
saito.k
28c5704b16 redis接続確認 2023-09-09 10:20:32 +09:00
oura.a
2812bc3d20 Merged PR 383: API実装（パートナー取得API） 
## 概要
[Task2540: API実装（パートナー取得API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2540)

パートナー取得APIを実装しました。

## レビューポイント
・データ取得方法が適切かどうか。
以下の優先順位を意識して作成したが適切か？また、意識できていない実装になっていないか？
①QueryBuilderを使用せずに処理する
②RDB、adb2cへのアクセス回数を最小限にする

## UIの変更
なし
## 動作確認状況
ローカルで動作確認済み、UT実施済み

## 補足
プライマリ、セカンダリ管理者IDがない場合のテストはUTでは実装せず、ローカルでの動作確認で正常に動作することを確認しました。
（プライマリ、セカンダリ管理者IDを指定してアカウントを作成するテストユーティリティを作成する必要があるが、あまり汎用的には思えず作成する手間が惜しかったため）
 2023-09-08 09:45:10 +00:00
makabe.t
c71cab92cb Merged PR 392: API IF実装 
## 概要
[Task2591: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2591)

- オプションアイテムの取得/更新APIのIFを実装し、OpenAPIの定義を更新しました。

## レビューポイント
- パスは適切か
- パラメータの内容、バリデータは適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-09-08 01:38:44 +00:00
makabe.t
6cba76fa61 Merged PR 385: ワークタイプ更新API実装 
## 概要
[Task2569: ワークタイプ更新API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2569)

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

## レビューポイント
- リポジトリの更新処理に問題はないか
- テストケースは適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-09-07 00:53:30 +00:00
oura.a
6605658bac Merged PR 389: IF不具合修正 
## 概要
[Task2589: IF不具合修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2589)

パートナー一覧取得APIに以下の不具合があったため修正しました。
・RoleGuardが誤っている（tier3,tier4が不足）
・パラメータを文字列として受け取ってしまう
※openapi.jsonへ跳ねる修正はありません。

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

## UIの変更
なし

## 動作確認状況
ローカルで動作確認済み（パラメータ受け取り）

## 補足
なし
 2023-09-06 08:45:44 +00:00
makabe.t
bde1ff60a9 Merged PR 384: API IF実装 
## 概要
[Task2568: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2568)

- worktype更新APIのIFを実装し、OpenAPI定義を更新しました。

## レビューポイント
- パスは適切か
- パラメータの制約は適切か

## UIの変更
- なし
## 動作確認状況
- ローカルで確認
 2023-09-05 08:09:41 +00:00
saito.k
b9a2d9b6b4 redis周りの修正をもとに戻す 2023-09-05 15:28:43 +09:00
saito.k
7c946686b7 Merged PR 289: ローカルでRedisコンテナに接続する 
## 概要
[Task2312: ローカルでRedisコンテナに接続する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2312)

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

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-09-05 05:52:49 +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
oura.a
17640eefa1 Merged PR 382: API IF実装 
## 概要
[Task2542: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2542)

パートナー取得APIのIFを実装しました。

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

## UIの変更
なし

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

## 補足
なし
 2023-09-04 08:37:01 +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
oura.a
f56f95123b Merged PR 376: [Sp17完了MISO]バリデータをクラスを使用した記述に統一する 
## 概要
[Task2502: [Sp17完了MISO]バリデータをクラスを使用した記述に統一する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2502)

バリデータの記述方法をクラスで外だしする形に統一しました。
また、未使用の引数を削除しました。

## レビューポイント
期待通りの修正内容であるか。
未使用の引数を削除してしまったが、問題ないか。

## UIの変更
なし

## 動作確認状況
ローカルで該当バリデーションを使用しているAPIを実行し、動作を確認済み

## 補足
なし
 2023-09-01 09:12:52 +00:00
makabe.t
f43e0f779e Merged PR 373: DBマイグレーション 
## 概要
[Task2515: DBマイグレーション](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2515)

- オプションアイテムのテーブルを追加するマイグレーションファイルを追加しました。

## レビューポイント
- カラム項目は適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
  - migrate up/down
 2023-09-01 06:42:58 +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
e77d8d8af0 Merged PR 374: API IF実装 
## 概要
[Task2521: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2521)

- Worktype追加API IFを実装しOpenAPI定義を更新しました。

## レビューポイント
- パスは認識通りか
- パラメータは適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-08-31 07:42:38 +00:00
oura.a
cb7ba77bc3 Merged PR 362: API実装（ライセンス注文キャンセルAPI） 
## 概要
[Task2484: API実装（ライセンス注文キャンセルAPI）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2484)

ライセンス注文キャンセルAPIを実装しました。

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

## UIの変更
なし

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

## 補足
なし
 2023-08-31 00:25:13 +00:00
makabe.t
6b91745b2b Merged PR 368: DBマイグレーション 
## 概要
[Task2503: DBマイグレーション](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2503)

- Worktypeテーブルを追加するマイグレーションファイルを追加しました

## レビューポイント
- カラム名は適切か
- カラムの制約は適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
  - up/downできることを確認
 2023-08-30 07:39:19 +00:00
makabe.t
a9d5d926ba Merged PR 369: API IF実装 
## 概要
[Task2504: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2504)

- ワークタイプ一覧取得APIのIFを実装し、openapi.jsonを更新しました

## レビューポイント
- パスは適切か
- プロパティに不足はないか
  - 返却値にidを追加しています

## UIの変更
- なし
## 動作確認状況
- ローカルで確認
 2023-08-30 06:53:58 +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
masaaki
1c4026eff9 Merged PR 366: API IF実装 
## 概要
[Task2496: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2496)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
  - ライセンス発行キャンセルAPIのIFと、使用するリクエスト/レスポンスのtypeを作成しました
- このPull Requestでの対象/対象外
  - 対象外：openapi.jsonの「/licenses/orders/cancel」について、descriptionが追加されていますが本タスクでの追加ではありません。
- 影響範囲（他の機能にも影響があるか）
  - 特になし

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

## UIの変更
- 無し

## 動作確認状況
- swagger uiよりリクエスト投入できることを確認

## 補足
- 無し
 2023-08-29 02:23:34 +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
maruyama.t
c4848eb007 Merged PR 363: QueryBuilderのままにするか、service層でソートするか検討する 
## 概要
[Task2465: QueryBuilderのままにするか、service層でソートするか検討する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2465)
[タスク 2404: controller.tsに不要なログが残っている](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%2016-1?workitem=2404)
- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
割り当て可能ライセンス取得APIのQueryBuilder使用部分に対してコメント文を追加した。
ついでに過去ＭＩＳＯチームで実装したcontrollerの処理で残っていた不要なデバッグログを削除した。
- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）
なし

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-08-25 08:04:28 +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
masaaki
e1693a7323 Merged PR 356: license_historyテーブルを消し込む 
## 概要
[Task2456: license_historyテーブルを消し込む](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2456)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
  - プロダクト バックログ項目 1226: 第五として、自アカウント内ユーザーにライセンスを割り当てたい
    にてライセンス割り当て履歴テーブル（license_allocation_history）を作成したのですが、過去にlicenses_historyという名で同様のテーブルを作成していることが判明しました。
  - 影響調査の結果、過去作成したlicenses_historyについては、使用していないことが分かったので、削除するよう対応を行いました。

- このPull Requestでの対象/対象外
  - すべて対象
- 影響範囲（他の機能にも影響があるか）
  - ソースコードgrepにより、licenses_historyテーブルを使用していないことを確認済

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

## UIの変更
- 特にありません。

## 動作確認状況
- 全ユニットテスト実施し正常に通ることを確認

## 補足
- 相談、参考資料などがあれば
 2023-08-25 05:53:58 +00:00
makabe.t
50a837efb9 Merged PR 358: API IF実装 
## 概要
[Task2459: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2459)

- タイピストグループ取得、編集APIのIFを実装しました。
  - API定義をjsonファイルに出力

## レビューポイント
- パスは適切か
- パラメータの内容と制約は適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-08-25 05:47:11 +00:00
maruyama.t
7be84e9bbc Merged PR 357: [Sp16-1]カードライセンスのレコード作成がbulkinsertになっていない 
## 概要
[Task2409: [Sp16-1]カードライセンスのレコード作成がbulkinsertになっていない](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2409)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
push→saveではなく、QueryBuilderを使用してバルクインサートを行うよう修正。
- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）
既存のカードライセンス発行処理
既存のライセンス発行処理

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

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

## 動作確認状況
- ローカルで確認
既存ユニットテスト
＋ローカル環境でカードライセンステーブル、ライセンステーブルへのDB登録・更新処理が想定通りに行われることを確認

## 補足
- 相談、参考資料などがあれば
 2023-08-25 05:33:56 +00:00
makabe.t
15d7119306 Merged PR 348: 画面実装（TypistGroup追加ポップアップ＆TypistGroup設定画面） 
## 概要
[Task2424: 画面実装（TypistGroup追加ポップアップ＆TypistGroup設定画面）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2424)

- TypistGroup追加ポップアップを追加しました。
  - 言語対応を実装しています。
  - 入力エラー時のメッセージ表示を実装しています。

## レビューポイント
- エラーメッセージの出し方は問題ないか
- デザイン適用は適切か

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

## 動作確認状況
- ローカルで確認
※実際のAPIを呼んでの登録は未検証です
 2023-08-25 05:33:05 +00:00
oura.a
9f2f163141 Merged PR 359: API IF実装 
## 概要
[Task2482: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2482)

ライセンス注文キャンセルAPIのIFを実装しました。
※同ファイル内に未使用のGetAllocatableLicensesRequestのインポートが残っていたので、ついでに削除しています。本APIとは無関係です。

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

## UIの変更
なし

## 動作確認状況
Swagger UIで確認済み

## 補足
なし
 2023-08-25 05:19:34 +00:00
oura.a
0fd1ff2b6a Merged PR 349: API実装（ライセンス割り当て解除API） 
## 概要
[Task2450: API実装（ライセンス割り当て解除API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2450)

ライセンス割り当て解除APIを実装しました。

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

## UIの変更
なし

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

## 補足
なし
 2023-08-24 08:24:06 +00:00
makabe.t
ba371ac4c0 Merged PR 345: API IF実装 
## 概要
[Task2438: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2438)

- TypistGroup追加APIを実装しました。
  - openapi.jsonを更新

## レビューポイント
- パラメータの制限は適切か
- パスは適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-08-24 00:08:22 +00:00
masaaki
422622c9ef Merged PR 351: license_allocation_historyのテーブル名が誤っている 
## 概要
[Task2469: license_allocation_historyのテーブル名が誤っている](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2469)

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

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-08-23 01:40:13 +00:00
masaaki
5eabfdb1a4 Merged PR 350: 割り当て可能ライセンスを取得する検索条件が誤っている 
## 概要
[Task2468: 割り当て可能ライセンスを取得する検索条件が誤っている](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2468)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
  - 割り当て可能ライセンスを取得する際の条件について、or条件が独立していた。
WHERE
  `license`.`account_id` = ?
  AND `license`.`status` IN (?, ?)
  AND   `license`.`expiry_date` >= ?
  OR `license`.`expiry_date` IS NULL
このため、「`license`.`expiry_date` IS NULL」に合致するレコードが他の条件に関係なく取得されていた。

- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）

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

## UIの変更
- 変更なし

## 動作確認状況
- ユニットテスト

## 補足
- 相談、参考資料などがあれば
 2023-08-23 00:52:55 +00:00
maruyama.t
249689a40e Merged PR 347: 割り当て可能ライセンス取得APIをQueryBuilderでの実装に戻す 
## 概要
[Task2466: 割り当て可能ライセンス取得APIをQueryBuilderでの実装に戻す](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2466)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
  - EntityManegarで実装した部分について、レビューバック後のPOSTMANによる確認が漏れていました。
結果、MySQLにクエリを投入する際に、typeORMでNULLS FIRSTを使用できない旨のエラーが出てしまいました。
よって、暫定的にQueryBuilderを使用しての実装にしています。
EntityManagarを使用するか、QueryBuilderを使用するかは別途開発メンバで相談して、修正する場合は別タスクで追って対応いたします。
[タスク 2465: QueryBuilderのままにするか、service層でソートするか検討する]()
  - ライセンス種別の取得について、結果0件時の考慮が出来ていなかったので修正しています
- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-08-22 12:22:47 +00:00
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