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

45 Commits

Author SHA1 Message Date
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
湯本 開
6deaa37df7 Merged PR 652: 外部連携以外のAPIバリデータ見直し(/accounts/*) 
## 概要
[Task3285: 外部連携以外のAPIバリデータ見直し](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3285)

- accounts/* 配下のRequest型に不足しているバリデータを追加

## レビュー対象外
- 他features配下のRequest型のバリデータ
  - 1個のレビューで出すとレビュー負荷が高すぎそうであったため、レビューを分割します

## レビューポイント
- 追加したバリデーションが適切か（主な対応は以下）
  - 数値に型変換していなかったものに型変換を追加
  - IsInt()を追加
  - マイナス値がセットされるとDBリクエストでコケる類のものに制限を追加
  - 文字列はDBやAzure AD B2Cで受付不能な文字数を弾く

## 動作確認状況
- ビルドとテストが成功することは確認
 2023-12-25 05:21:35 +00:00
湯本 開
1bc6618a6d Merged PR 643: ライセンス引き戻し完了通知 [U-109] の実装 
## 概要
[Task3306: ライセンス引き戻し完了通知 [U-109] の実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3306)

- ライセンス引き戻し完了のメール送信機能を追加しました。
- テストでメール送信しないようSendGridのメソッドを上書きする処理を追加しました。

## レビューポイント
- テンプレートの適用内容に不自然な点はないか
- アカウントのFromとToとCCに関わる部分で認識違いはないか
  - `orderedAccountId` という引数には注文したアカウント=下位階層のアカウントが入るという理解であっているか等

## UIの変更
- なし
## 動作確認状況
- npm run testは通過
 2023-12-20 07:54:47 +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
45b2cad30c Merged PR 624: クエリログに追跡用IDと実行日時を追加する（各作業まとめブランチ） 
## 概要
[Task3309: 修正をまとめる用のブランチ](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3309)

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

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-12-13 01:07:36 +00:00
makabe.t
b8b3416795 Merged PR 625: セレクトのクエリに追跡用のIDと実行日時の情報を追加する 
## 概要
[Task3288: セレクトのクエリに追跡用のIDと実行日時の情報を追加する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3288)

- リポジトリ内でのDB操作でSelect文となる部分にコメント（追跡ID_日時）を追加しました。
  - `find`, `fineOne`, `count`を対象にしています。
- コメントを追加するにあたってContextをリポジトリメソッドの引数に追加しています。

## レビューポイント
- 対応箇所の漏れはないでしょうか？
- コメントのつけ方は適切でしょうか？

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-12-13 00:00:15 +00:00
maruyama.t
934ee7f44d Merged PR 626: Revert 'deleteでコメントを追加できるようにする' 
## 概要
[Task3289: deleteでコメントを追加できるようにする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3289)

delete処理にSQLコメントを挿入する形にリファクタ
レビューポイント
repository/* に存在するdelete文は全て置き換えたはずだが、漏れはなさそうか

Reverts !622
 2023-12-12 09:45:30 +00:00
maruyama.t
6c6970c70a Merged PR 622: deleteでコメントを追加できるようにする 
## 概要
[Task3289: deleteでコメントを追加できるようにする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3289)

delete処理にSQLコメントを挿入する形にリファクタ
レビューポイント
repository/* に存在するdelete文は全て置き換えたはずだが、漏れはなさそうか
 2023-12-12 09:28:40 +00:00
maruyama.t
94f34a0fde Merged PR 606: 画面修正（Terms画面） 
## 概要
[Task3210: 画面修正（Terms画面）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3210)
[Task3211:API修正（バージョン取得API）](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%2023-1?workitem=3211)
[Task3212:API修正（バージョン更新API））](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%2023-1?workitem=3212)

- このPull Requestでの対象/対象外
Click here to read the terms of use.の文言は多言語対応の対象のため、現在一律同じ文言がでます。
第一～第四階層は
上からEULA,PrivacyNotice,DPAが表示されています
第五階層は、
上から、PrivacyNotice,DPAが表示されています
- 影響範囲（他の機能にも影響があるか）
ユーザアーカイブテーブルにPrivacyNoticeのバージョンを追加

## レビューポイント
同意済みプライバシーポリシーはユーザーアーカイブの対象だと認識しているが正しいか。
termsテーブルのdocument_typeの値をPrivacyNoticeにしているが、PRIVACY_NOTICEにしたほうがよいか。
ユニットテストに不足はないか。

## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
https://ndstokyo.sharepoint.com/sites/Piranha/Shared%20Documents/Forms/AllItems.aspx?csf=1&web=1&e=hzPw9b&cid=7737ed1b%2D0eb4%2D4331%2Da238%2D14dd35b27e18&FolderCTID=0x012000C0DCEE65AC2177479C3C761CD137C9C9&id=%2Fsites%2FPiranha%2FShared%20Documents%2FGeneral%2FOMDS%2F%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%2FTask3210&viewid=786a81cf%2Dd15f%2D4dc2%2D9e55%2Dc7a729fbc72f
## 動作確認状況
- ローカルで確認

## 補足
- 相談、参考資料などがあれば
 2023-12-05 09:10:49 +00:00
水本 祐希
446b9365f2 Merged PR 584: エラーログが意図した通りに出ていないところがありそうな問題を解消する 
## 概要
[Task2934: エラーログが意図した通りに出ていないところがありそうな問題を解消する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2934)

- 何をどう変更したか、追加したライブラリなど
error=Errorのログ表示をerror=××Error（エラーの原因）となるように実装

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

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

## 相談（必須レビュアーの方に）
今回の実装のみではまだ解消できていない箇所がいくつかみられました。
以下にその解消できなかったログをいくつかピックアップします。

ERROR [TasksService] error=Error: There is no AuthorId for the API executor.
ERROR [FilesService] error=Error: blob failed
ERROR [FilesService] error=Error: container not found.
ERROR [UsersService] error=Error: ADB2C error
ERROR [UsersService] error=Error: user not found
ERROR [UsersService] error=Error: sort criteria not found
ERROR [UsersService] error=Error: sort criteria not found
ERROR [UsersService] error=Error: The value stored in the DB is invalid.

原因としては、throw new ××Errorではなく、throw new Errorで実装されていました。
interface Error {
    name: string;
    message: string;
    stack?: string;
}

interface ErrorConstructor {
    new(message?: string): Error;
    (message?: string): Error;
    readonly prototype: Error;
}
上記の実装により返却されるメッセージがErrorになっているため解消されていないと考えています。

以上の事象について対応するかしないかということと、対応する場合は、どのように対応していくかをご意見いただきたいです。
本タスクで対応というよりも別タスクとして対応とMISOは想定しています。
 2023-11-20 00:49:00 +00:00
makabe.t
e228d06cc7 Merged PR 571: アカウント画面で管理者設定している箇所の対応 
## 概要
[Task3054: アカウント画面で管理者設定している箇所の対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3054)

- アカウント画面での管理者設定でEmail認証していないユーザーを設定しないように対応しました。
  - 管理者選択のドロップダウンに表示されるユーザーをクライアント側で認証済みでフィルタする対応
  - アカウント情報更新APIで管理者に未認証ユーザーを設定しようとするとエラーとなるように修正

## レビューポイント
- ユーザー取得は画面側でフィルタしているが対応としては適切でしょうか？
  - 同じAPIをフィルタせずに使うところもあるため

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-11-10 08:09:09 +00:00
水本 祐希
f4347ff5c0 Merged PR 528: 第五階層ライセンス画面の修正 
## 概要
[Task2937: 第五階層ライセンス画面の修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2937)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
allocatedLicense（Number of licenses acclocated）
reusableLicense（number of licenses available for reuse）
freeLicense（Number of unused licenses）
の3つに、「有効な総ライセンス数のうち」という条件を追加する

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

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-10-26 08:52:52 +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
maruyama.t
00f4966aa9 Merged PR 470: [Sp-19]アカウント削除時に削除するテーブルについて、on delete cascadeではなくコード上で削除を行うよう修正する 
## 概要
[Task2783: [Sp-19]アカウント削除時に削除するテーブルについて、on delete cascadeではなくコード上で削除を行うよう修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2783)

アカウント削除時に削除するテーブルについて、on delete cascadeではなくコード上で削除を行うよう修正
一部ユニットテスト用にutilityのcreateForeignKeyConstraints: falseに指定
LGTM後、すべてのテーブルのレコード削除の動作確認をDEV環境で実施します。

## レビューポイント
DBマイグレーションファイルが正しく修正されているか

## UIの変更
なし

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

## 補足
- 相談、参考資料などがあれば
 2023-10-16 09:02:38 +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
oura.a
aef6a35a7d Merged PR 466: [PBI1220残]退避テーブル対応（月の途中で退会したアカウントの集計対応） 
## 概要
[Task2767: [PBI1220残]退避テーブル対応（月の途中で退会したアカウントの集計対応）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2767)

アカウント削除時に以下のテーブルの削除対象データを退避する処理を追加しました。
・ライセンステーブル
・ライセンス割り当て履歴テーブル

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

## UIの変更
なし

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

## 補足
なし
 2023-10-06 06:04:11 +00:00
maruyama.t
a8bacefc5f Merged PR 461: APIテスト実施 
## 概要
[Task2672: APIテスト実施](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2672)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
詳細なレコード（ライセンス、タスク、ユーザーグループなど）は別途dev動作確認にてデータを用意して行います。
現時点では、各レコードの削除はMySQL用にmigrationファイルにて記述したON DELETE CASCADEの機能にて削除を行う為、SQLiteを用いた本ユニットテストでは動作確認対象外としています。
- 影響範囲（他の機能にも影響があるか）
entityの定義（accounts - users）のON DELETE CASCADEを明記

## レビューポイント
- 本ユニットテストは正常系の動作確認と、それぞれのservice内部で異常発生時もAPI自体は正常終了し、[MANUAL_RECOVERY_REQUIRED]ログが表示されることの確認を主な目的として実装しています。

## UIの変更
なし

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

## 補足
- 相談、参考資料などがあれば
 2023-10-05 08:16:00 +00:00
maruyama.t
664e815ef9 Merged PR 429: API実装（アカウント削除API：メイン処理） 
## 概要
[Task2670: API実装（アカウント削除API：メイン処理）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2670)

アカウント削除APIを実装しました。
APIとしてはこれで実装完了ですが、DBに外部キー制約をつけていないので、現時点で削除できるものは以下のみです。
・アカウントテーブル
・ADB2Cのユーザー
・BLOBストレージ

## レビューポイント
内容が重めの処理なので全体的に見ていただけると嬉しいです。

## UIの変更
なし

## 動作確認状況
ローカルで以下の動作を確認
・RDBのアカウントが削除される
・ADB2Cのユーザーが削除される
・RDBのユーザーが退避テーブルに登録される
・BLOBストレージが削除される

## 補足
UTは別タスクに切り出しているので、本タスクでは実装していません。
 2023-10-03 06:20:36 +00:00
maruyama.t
3f5f75a48f Merged PR 403: API実装（アカウント設定API） 
## 概要
[Task2603: API実装（アカウント設定API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2603)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
getDealerAccountという一階層上のアカウントを取得する共通的なAPIも実装しています。

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

## レビューポイント
- 実行中にdealerアカウントに対して変更が走った場合でも対応できるよう、トランザクションをネストした実装にしています。
トランザクションをネストした場合は、内部のトランザクションが正常に完了し、その後外部のトランザクションも正常に完了すると、変更がコミットされます。
- 画面の仕様上、第五階層でないとdealerの変更は行わないが、API側でdelegationPermissionに対する階層（不整合チェック）をやっていないが、問題ないか。
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
なし

## 動作確認状況
- ローカルで確認
第五階層以外がアカウント情報を設定できる
アカウント情報を更新する（第五階層が実行/セカンダリ管理者ユーザがnull）
・プライマリ管理者ユーザを存在する値にして、更新される
・セカンダリ管理者ユーザをundefinedで入力し、nullで更新される
アカウント情報を更新する（第五階層以外が実行）
アカウント情報を更新する（ディーラーアカウントが未入力）
・parentAccountIdがnullで更新される
アカウント情報の更新に失敗する（ディーラー未存在）
アカウント情報の更新に失敗する（プライマリ管理者ユーザ未存在）
アカウント情報の更新に失敗する（プライマリ管理者ユーザがnull）
アカウント情報の更新に失敗する（セカンダリ管理者ユーザ未存在）
アカウント情報の更新に失敗する(プライマリ管理者ユーザ、セカンダリ管理者ユーザ両方が未入力)
以下POSTMANで確認
プライマリ管理者ユーザIDをundefinedで入力した場合はエラー
管理者権限のないアカウントで実行した場合、権限エラー
500エラー
## 補足
- 相談、参考資料などがあれば
 2023-09-19 07:12:58 +00:00
makabe.t
d1a8b887e5 Merged PR 417: ActiveWorktypeIDを未設定に戻せるように修正 
## 概要
[Task2677: ActiveWorktypeIDを未設定に戻せるように修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2677)

- ActiveWorktypeIDを設定後、未選択に戻す操作ができるように修正

## レビューポイント
- 修正箇所の共有

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-09-19 02:56:19 +00:00
makabe.t
d843affe88 Merged PR 413: API実装（ActiveWorktypeID更新API） 
## 概要
[Task2622: API実装（ActiveWorktypeID更新API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2622)

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

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

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-09-15 06:26:37 +00: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
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
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
saito.k
6fed7738a7 エラーをもとに戻す 
漏れてた確認のためにエラーを追加
 2023-08-22 19:10:21 +09:00
saito.k
86f9bb76e6 無理やりエラーを発生させる 2023-08-22 18:45:28 +09: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
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
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
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
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
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
masaaki
c3845d7e9c Merged PR 272: 第五のライセンス画面について、Issue RequestingとNumber of Requestingの意味合いが逆だったので修正する 
## 概要
[Task2282: 第五のライセンス画面について、Issue RequestingとNumber of Requestingの意味合いが逆だったので修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2282)

- 第五のライセンス画面について、Issue RequestingとNumber of Requestingの意味合いが逆だったのを修正
- 検討課題OMDS_IS-129における三浦さんの回答にて判明。
　> 【Issue Requestingについて】
　　Issue Requestingはリクエストされた総ライセンス数を意味していて、注文数はその下の[Number of Requesting]で表示する想定です。（パートナーから見たときは総ライセンス数の情報があれば良く、注文数は[Issue]ボタンの数から判断できるかと）

　→PO確認の結果、基本設計誤りと判明。

- 影響範囲（他の機能にも影響があるか）
　・第一～第四ライセンス画面については、Issue Requestingはリクエストされた「総ライセンス数」となっていることを確認済。
　・基本設計修正済

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

## UIの変更
- 無し

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

## 補足
- 相談、参考資料などがあれば
 2023-07-26 01:50:17 +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
saito.k
e4ba5229df Merged PR 224: タスク中断API実装 
## 概要
[Task2119: タスク中断API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2119)

- タスク中断APIの処理を実装
- テスト実装
- チェックアウト処理のエラーチェックを修正
  - エラーが発生したときに何が原因なのかログに出力するように修正

## レビューポイント
- チェックインと同様の処理部分を切り出さずにそのまま実装したが、スマートに切り出せる方法はありそうか。
  - チェックインとサスペンドの取得処理だけ切り出してもあまりうれしくない（この２つ以外のところで使えなさそう）
    - キャンセルでは使えるかもだけど
  - チェックアウトやその他のメソッドのタスク取得とまとめようとすると、チェック内容や検索条件に差異がありきれいに切り出すことができなさそう。
    - まとめようとすると、引数が膨大でチェック項目も複雑になる

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

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

## 補足
- コンフリクトが発生しているが、現在のdevelopを取り込むとエラーになるので解消し次第、取り込んで競合解決します
- チェックインAPIの実装も入っていますが、そこは別のPRでレビューを行っているので対象外となります
 2023-07-12 08:27:46 +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
saito.k
6a1226c62e Merged PR 165: タイピスト割り当て変更API実装 
## 概要
[Task1932: タイピスト割り当て変更API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1932)

- タイピスト割り当て変更APIを実装
- テスト実装

## レビューポイント
- ＩＦのバリデーションを実装したがチェック内容はこれでよさそうか
- ＤＢのデータ取得・更新処理は問題ないか
  - ＤＢへアクセスする回数は問題ない程度か
- パスパラメータのバリデーションは問題ないか

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

## 動作確認状況
- ローカルで確認（swaggerUI,Postman）

## 補足
- 別途sqliteを用いたテストを実装する予定
 2023-06-23 04:04:01 +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
masaaki
3220ae0552 Merged PR 157: accounts.repository.service.tsの参照があやまっている 
## 概要
[Task1960: accounts.repository.service.tsの参照があやまっている](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1960)

- constantsの参照についてsrcフォルダを参照しないように修正しました

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

## UIの変更
- 特になし

## 動作確認状況
- ビルドが成功することを確認

## 補足
- 相談、参考資料などがあれば
 2023-06-13 09:45:30 +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
saito.k
d5e5e59f8c Merged PR 128: API実装（ソート条件変更） 
## 概要
[Task1835: API実装（ソート条件変更）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1835)

- ソート条件変更APIを実装
  - トークンからB2CのIDを取得→userテーブルからユーザー情報を取得
  - ユーザーIDでソート条件テーブルを検索→レコードを更新
- ソート条件テーブルにレコードを作成する
  - アカウント作成時の処理にソート条件レコードを作成する処理を追加
  - ユーザー追加時にも処理を追加
- テスト修正

## レビューポイント
- APIの引数をチェックする関数をControllerに配置してもよいか
- ソート条件のレコードを作成するタイミングに漏れはないか
- 実装漏れはないか

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

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

## 補足
- 相談、参考資料などがあれば
 2023-06-06 08:20:21 +00:00
maruyama.t
2c935c8b52 Merged PR 101: API実装（ライセンス注文登録） 
## 概要
[Task1685: API実装（ライセンス注文登録）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1685)

タスク 1685: API実装（ライセンス注文登録）
ライセンス注文APIを追加

## レビューポイント
登録時のDB処理方法に問題がないか。
処理、エラーハンドリングに過不足がないか。

## UIの変更
なし

## 動作確認状況
ローカルでユニットテストを実施。
ローカルでAPIを実行し、DBに登録できること・poNumberの重複チェックが想定通りに動作していることを確認。

## 補足
なし
 2023-05-30 07:17:43 +00:00
湯本 開
9f5252baf8 Merged PR 56: API実装（アカウント登録） 
## 概要
[Task1496: API実装（アカウント登録）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1496)

- アカウント登録のAzureAD B2Cへのユーザー作成部分以外について実装
- migration SQLに足りない要素があった箇所を修正(default追加)
- 各種環境変数を追加
- 秘密鍵/公開鍵を取得する方法を環境変数に変更（KeyVaultからWebAppsの機能で環境変数へ流し込む想定）

## レビューポイント
- 実装方法として問題がありそうな箇所が存在しないか
- 可読性の低い箇所が存在しないか
- Moduleの分け方、つなげ方などは問題ないか
- ラフスケッチと違い、Account作成と管理者ユーザー作成を同一トランザクションで行うよう修正したが問題ないか

## 動作確認状況
- メール送信以外はローカルで確認、メール送信部分は未確認
 2023-03-29 03:55:44 +00:00