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

163 Commits

Author SHA1 Message Date
makabe.t
e6d6e477d9 Merged PR 862: パートナー一覧画面＆パートナー編集ポップアップ実装 
## 概要
[Task3935: パートナー一覧画面＆パートナー編集ポップアップ実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3935)

- パートナー一覧画面からパートナー編集ポップアップを表示して情報を変更できる画面実装をしています。

## レビューポイント
- エラーの表示は適切でしょうか？
- 画面イメージは認識通りでしょうか？

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

## クエリの変更
- なし
## 動作確認状況
- ローカルで確認
- 行った修正がデグレを発生させていないことを確認できるか
  - 新規機能なので問題なし
 2024-04-08 07:41:05 +00:00
maruyama.t
915483c109 Merged PR 860: パートナー情報更新API実装 
## 概要
[Task3937: パートナー情報更新API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3937)

- パートナーアカウント情報更新APIとUTを実装しました。

## レビューポイント
- エラーケースの出し分けは適切でしょうか？
- テストケースは過不足ないでしょうか？

## UIの変更
- なし

## クエリの変更
- 新規追加のため変更はなし

## 動作確認状況
- ローカルで確認
- 行った修正がデグレを発生させていないことを確認できるか
  - 新規追加なので問題なし。
 2024-04-05 02:37:58 +00:00
maruyama.t
0288292058 Merged PR 859: パートナーユーザー取得API実装 
## 概要
[Task3936: パートナーユーザー取得API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3936)
- このPull Requestでの対象/対象外
パートナー変更APIの修正は別タスクで対応
- 影響範囲（他の機能にも影響があるか）
新規APIのため他の機能に影響はない

## レビューポイント
- パートナーのアカウントIDからユーザー一覧を取得する際に、Repository層ではEmai認証状態を意識した取得は行わない
　→service層でフィルタリングする実装にしたが
　（アカウントIDからユーザー一覧を取得する処理がいままでなかったので、あったほうがいいかなと思い）

## クエリの変更
新規APIのためクエリの変更はない

## 動作確認状況
- ローカルで確認
UT+POSTMAN
- 行った修正がデグレを発生させていないことを確認できるか
  - 具体的にどのような確認をしたか
    - どのケースに対してどのような手段でデグレがないことを担保しているか
既存機能には手を入れていない
## 補足
- 相談、参考資料などがあれば
 2024-04-03 00:50:53 +00:00
maruyama.t
114ded790e Merged PR 855: API IF実装（パートナーを編集したい） 
## 概要
[Task3930: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3930)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 新規追加API2本のIFを作成、controllerの返却値は仮実装（別タスクで実装）
- 影響範囲（他の機能にも影響があるか）
　新規追加のみなので影響はなし

## レビューポイント
- 特筆する点はありません

## UIの変更
なし

## クエリの変更
なし

## 動作確認状況
- ローカルで確認
　バリデーションテストとPOSTMANからの起動の確認
- 行った修正がデグレを発生させていないことを確認できるか
  - 具体的にどのような確認をしたか
    - どのケースに対してどのような手段でデグレがないことを担保しているか
完全新規のIFの実装のみなのでデグレはない想定

## 補足
- 相談、参考資料などがあれば
 2024-03-26 06:22:07 +00:00
makabe.t
6e93a5be79 Merged PR 846: パートナーアカウント削除API実装 
## 概要
[Task3834: パートナーアカウント削除API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3834)

- パートナーアカウント削除APIとUTを実装しました。

## レビューポイント
- 削除対象データは適切でしょうか？
- テストケースに不足はないでしょうか？

## UIの変更
- なし

## クエリの変更
- なし

## 動作確認状況
- ローカルで確認
  - テストとローカルで実行確認
- 行った修正がデグレを発生させていないことを確認できるか
  - 既存処理への変更なし
 2024-03-22 06:12:47 +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
masaaki
f80912c617 Merged PR 834: API IF実装 
## 概要
[Task3904: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3904)

- 「プロダクト バックログ項目 1242: パートナーを削除したい」のAPI IFを作成しました
- 影響範囲（他の機能にも影響があるか）
  - 新規IFのため影響はなし

## レビューポイント
- controllerの試験実装が初なので、テストケース過不足ないか確認いただきたいです。

## UIの変更
- 無し

## クエリの変更
- 無し

## 動作確認状況
- ユニットテストが通ることを確認、ローカル環境でpostmanで呼び出せることを確認、SWAGGER UI上で追加されていることを確認
- 行った修正がデグレを発生させていないことを確認できるか
  - 具体的にどのような確認をしたか
    - ユニットテストが通ることを確認

## 補足
- 相談、参考資料などがあれば
 2024-03-15 07:41:56 +00:00
湯本 開
43561f237e Merged PR 822: クエリ比較用ログ出力の仕組みを改良 
## 概要
[Task3889: クエリ比較用ログ出力の仕組みを改良](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3889)

- SQLクエリを比較可能とするための仕組みを導入
  - UUIDや日付等の実行の度に変更される要素を出力段階で置き換えてしまうロガーを追加
  - テストで上記ロガーを使うよう修正
- テストで使用する環境変数が必要がないのにlocalを指定するようになっていたため、production想定でテストが実施されるようテスト用環境変数ファイルを変更

## レビューポイント
- この仕組みを使ってデグレを防ぐための「クエリの変更内容を確認する」事が問題なく出来そうか

## UIの変更
- なし

## クエリの変更
- なし

## 動作確認状況
- npm run test が正常に実施されることを確認
- 行った修正がデグレを発生させていないことを確認できるか
  - testファイルにしか参照されない変更なので、デグレは原理的に発生しないはず
  - testも正常に成功したことを確認したので、デグレは発生していないはず
 2024-03-12 03:57:29 +00:00
湯本 開
ff4cd35ed3 Merged PR 819: [3848]アカウント削除処理修正 
## 概要
[Task3847: [3848]アカウント削除処理修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3847)

- AccountArchiveエンティティを追加
- アカウント削除時、Accountをアーカイブする処理を追加
- アーカイブ関連テストを追加

## レビューポイント
- 実装の修正内容は問題なさそうか
- テストケースの修正内容は問題なさそうか
- クエリの変更内容の確認方法は問題なさそうか to 斎藤さん

## レビュー対象外
- テスト用ロガーに以下の比較用前処理を追加するべきだが、別タスクを作って対応予定
- 下記クエリの変更点にて、CommentOut判定に環境変数STAGEを使用している部分にRequestIdが表示されていない部分が存在するが、テスト用環境変数の変更は上記と同じく別タスクを作って対応予定
[タスク 3889: クエリ比較用ログ出力の仕組みを改良](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_workitems/edit/3889)

## クエリの変更
- ロガーを有効にした状態でテストを実行し、ログのUUIDと日付を処理して比較できるよう加工した
- https://ndstokyo.sharepoint.com/:f:/r/sites/Piranha/Shared%20Documents/General/OMDS/%E3%82%AF%E3%82%A8%E3%83%AA/3847?csf=1&web=1&e=xlK011

## 動作確認状況
- npm run testで確認
- 行った修正がデグレを発生させていないことを確認できるか
  - アカウント削除テストで発行されるクエリを比較し、AccountArchiveする対象を特定するためのAccountのSELECT、AccountArchiveのINSERTとSELECTのみが追加されている事が確認できたので、デグレはないと判断
 2024-03-11 02:08:29 +00:00
Kentaro Fukunaga
f386a8f7e0 Merged PR 817: API IF実装（親アカウント変更API） 
## 概要
[Task3852: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3852)

- 親アカウント変更APIのIFを実装し、OpenAPIの生成もしました。
- 影響範囲（他の機能にも影響があるか）
    - なし

## レビューポイント
- controllerのメソッド名にほか良い案ないか？
- validationに過不足や間違いないか？
- ~~controllerのテストは正常系ひとつだけ追加しているが、他にあったほうがいいものあるか？~~
    - ~~個人的には、テスト追加してもnpmライブラリのvalidatorのテストになるだけな気がするため不要では？と思っています。~~
    - 「npmライブラリのvalidatorを正しいパラメータで正しく利用しているか」が目的であるとの認識を得たため異常系も追加しました。

## 動作確認状況
- apigenを実行してOpenAPI生成できることを確認、controllerテスト通ることを確認。
- 行った修正がデグレを発生させていないことを確認できるか
  - 新規APIのため無し
 2024-03-11 01:29:55 +00:00
Kentaro Fukunaga
f8183399e2 Merged PR 762: アカウント利用制限更新API実装 
## 概要
[Task3710: アカウント利用制限更新API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3710)

- タイトルの通りです。
- 既存実装でログが不足している箇所あったのでちょろ修正もしました。

## レビューポイント
- これと言ってみて欲しいポイントはないので、何か気になる点あれば

## 動作確認状況
- ローカルで全テストが通ることを確認済み
 2024-02-20 11:23:04 +00:00
Kentaro Fukunaga
ecb28b9328 Merged PR 759: API IF実装 
## 概要
[Task3711: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3711)

- アカウント利用制限更新APIのIFを実装し、openapi生成しました。

## レビューポイント
- メソッド名で他に案あれば。

## 動作確認状況
- 特になし
 2024-02-19 04:32:59 +00:00
SAITO-PC-3\saito.k
cdef84e269 Merge branch 'develop' into ccb 
# Conflicts:
#	dictation_server/src/features/accounts/accounts.service.spec.ts
 2024-02-19 12:00:01 +09:00
saito.k
b4df229de4 Merged PR 751: [FB対応]WorkTypeIDを大文字固定にしたい 
## 概要
[Task3733: 対応する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3733)

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

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

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

## UIの変更
- https://ndstokyo.sharepoint.com/:f:/r/sites/Piranha/Shared%20Documents/General/OMDS/%E3%82%B9%E3%82%AF%E3%83%AA%E3%83%BC%E3%83%B3%E3%82%B7%E3%83%A7%E3%83%83%E3%83%88/Task3733?csf=1&web=1&e=zNB1Hi

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

## 補足
- 相談、参考資料などがあれば
 2024-02-19 01:09:39 +00:00
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
masaaki
32d8c6b896 Merged PR 737: ファイル削除設定API作成 
## 概要
[Task3546: ファイル削除設定API作成](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3546)

- プロダクト バックログ項目 1199: アカウント設定を変更したい（ファイル削除設定）
- ファイル削除設定API作成を実装しました
- 変数定義時に型指定が不要で、自動的に明示的な型宣言が削除されています。非テストコードについてはeslint-disable-lineによって無視設定を行っています。テストコードについては削除したままとしています。

## レビューポイント
- 特筆するものはありません

## UIの変更
- 無し

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

## 補足
- 無し
 2024-02-07 02:47:41 +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
x.yumoto.k
4442c7cbbe Merge branch 'develop' into ccb 2024-02-06 14:56:39 +09:00
湯本 開
a1b7505035 Merged PR 732: [改善]認証用URLについて、ドメイン名の末尾に/が必要となることへの対応 
## 概要
[Task3625: [改善]認証用URLについて、ドメイン名の末尾に/が必要となることへの対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3625)

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

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

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

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

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

## UIの変更
- https://ndstokyo.sharepoint.com/:f:/r/sites/Piranha/Shared%20Documents/General/OMDS/%E3%82%B9%E3%82%AF%E3%83%AA%E3%83%BC%E3%83%B3%E3%82%B7%E3%83%A7%E3%83%83%E3%83%88/Task3613?csf=1&web=1&e=i8cN2g

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

## 補足
- 相談、参考資料などがあれば
 2024-02-05 11:46:29 +00:00
maruyama.t
df74dc358c Merged PR 731: 既存API修正（アカウント情報取得API、アカウント作成API） 
## 概要
[Task3555: 既存API修正（アカウント情報取得API、アカウント作成API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3555)

Accountにauto_file_delete、file_retention_daysを追加
既存のテストでアカウントを作成している部分に項目の値を追加。

## レビューポイント
-entityの

```
@Column({ default: 30 })
```
が必要かどうか。

## UIの変更
なし

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

## 補足
- 相談、参考資料などがあれば
 2024-02-05 02:27:43 +00:00
Kentaro Fukunaga
f595cae1b8 Merged PR 728: ファイル削除設定更新API IF実装 
## 概要
[Task3557: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3557)

- ファイル削除設定更新APIのIFを実装してOpenAPI生成しました。

## 補足
- 別PRでOKもらってるのでそのままマージします
 2024-02-03 06:22:53 +00:00
makabe.t
9f7c8c99c0 Merged PR 720: API IF実装 
## 概要
[Task3534: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3534)

- タイピストグループ削除APIのIFを実装し、OpenAPIを更新しました。

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

## UIの変更
- なし
## 動作確認状況
- ローカルで確認
 2024-02-02 00:27:08 +00:00
makabe.t
dbff86915e Merged PR 708: 対応 
## 概要
[Task3550: 対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3550)

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

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

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2024-01-29 05:54:35 +00:00
湯本 開
cf56239da2 Merged PR 669: Pipeline上でdocker-composeを用いてMySQLを起動する方法を調査する 
## 概要
[Task3427: Pipeline上でdocker-composeを用いてMySQLを起動する方法を調査する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3427)

- MySQLでUnitTestを実施する仕組みを作成
  - DevContainerにUnitTestからのみアクセスされるMySQLが動くコンテナ( `test_mysql_db` )を追加
  - テストでMySQLを使用し、上記 `test_mysql_db` に対してアクセスするよう構成
  - テストの前処理で `test_mysql_db` の全てのテーブルをTRUNCATEする処理を実行し、データをクリアする
    - 常にテスト用MySQLは起動しているが、テスト実行前にクリアされるので空っぽ前提の状態でテストを実施できる
  - 実際にMySQLを使用して実行されるテストを1つ追加
    - `users.service.spec.ts | UsersService.createUser` が対象
- Pipeline上でDevContainerを起動し、DevContainer内で `npm ci` `(UnitTest用DBへの) migration` `npm run test` を行う仕組みを作成
  - Pipelineを追加
    - **StagingPipelineでテスト・ステップを切り出し**
  - Pipelineテスト用DevContainer（のdocker-compose.yml）を作成
    - 構成は通常の物と同一だが、ネットワーク設定だけ外部のMySQLやRedisと通信しない前提に変更している
  - テストの実行環境を自己ホストではなく、一般で提供されているマシンに変更
    - 自己ホストのマシンにdocker-composeが入ってない/ビルド後のクリア処理が大変等の理由
- テストで使用する環境変数を `.env.test` という別環境変数に切り出し
  - そうすることで、PipelineでKeyVaultへのアクセスをする必要がなくなる
- **migrationの漏れを修正**
  - テストが通らないことで発覚したmigration漏れを修正
- **テストコードを修正**
  - **Date型のミリ秒単位の誤差を setMillseconds(0) を用いることで0にセットするコードを追加（ライセンス付近）**

## レビューポイント
- **WIPレビュー以降の追加の修正で違和感のある箇所はないか**
  - コミット履歴等で追っていただいた方が楽かと思います
  - 直近の`Merge branch 'develop' into feature/3427/poc-mysql-test` 以降がWIP以降に変更されたコードです
- **レビュー通過後、developのPipelineについては手動で同等のコードに変更→PRをdevelopにマージ予定だが進め方として問題はないか**
- [※WIPでレビュー済み] テスト用DevContainerを別途作成したが、現行のDevContainerを使用するよう頑張った方がいいか？
  - [※WIPでレビュー済み] CI/CDの実行速度面を考慮し、使用されないMySQLとRedisのサービスとネットワーク作成が同居しているdocker-compose.ymlの実行を避けたが、管理対象は増えているので議論の余地はある
- [※WIPでレビュー済み] MySQLでUnitTestを実行する際に懸念事項はないか
- [※WIPでレビュー済み] Dockerを起動する関係でCI/CDのスピードが数分遅くなると思われるが、許容可能か
- [※WIPでレビュー済み] `.env.test` に置き換えて問題ないか

## 動作確認状況
- ローカル＆Pipelineで実行して確認済
 2024-01-23 07:58:42 +00:00
saito.k
c4efaf1a1a Merged PR 671: 「LicenseAlert」を消す対応 
## 概要
[Task3381: 対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3381)

- 以下の画面から「LicenseAlert」を削除
  - ユーザー追加Popup
  - ユーザー更新Popup
  - ユーザー一覧画面
- 以下のAPIの引数から「LicenseAlert」の設定をするパラメータを削除
  - ユーザー作成API（users/signup）
  - ユーザー更新API（users/update）
- ユーザー一覧取得APIのレスポンスから「LicenseAlert」を削除
- usersEntityから「LicenseAlert」を削除
- クライアントのAPI呼び出し部分を自動生成するopenapi-generator-cliのバージョンを7.1.0で固定
  - 最新バージョン（7.2.0）はaxiosのバージョン1.x用になるため
  - https://github.com/OpenAPITools/openapi-generator/releases/tag/v7.2.0

## レビューポイント
- openapi-generator-cliのバージョンを固定するのではなく、axiosのバージョンを上げたほうがよいか
  - axiosのバージョン1.xが出たのがおととしになるのでそろそろ上げてもいい？
  - このタイミングでメジャーバージョンを上げるのはどうか
    - 基本的な仕様について大きくは変わってない模様
    - 実際に上げてみて軽く動作確認してみたが問題なかった

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

## 補足
- 相談、参考資料などがあれば
 2024-01-11 06:58:01 +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
masaaki
7652e5ef08 Merged PR 662: [OMDS様要望] タスク一覧画面の日時表示を変えたい（修正タスク） 
## 概要
[Task3362: [OMDS様要望] タスク一覧画面の日時表示を変えたい（修正タスク）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3362)

- 以下の画面について、日時の表示形式をブラウザのロケールに合わせて変換し、時差を考慮した内容になるよう修正しました。
  - タスク一覧画面
  - 音声メタ情報ポップアップ
  - 注文履歴画面（日付のみ表示）
    - サーバからUTCでのYYYY/MM/DDの文字列として返却していたので、ISOStringとして返却して画面で表示内容を変換するよう処理を修正しました
- タスク一覧画面のFile Lengthについて、hh:mm:ss形式となるよう修正しました

## レビューポイント
- formatMillisecondsToHHMMSSについて、他で使うシーンがなかったのでdictation内の関数としたが、全体の関数とした方がよいか
- convertUtcToLocalについて不正な日付の文字列が渡された場合、変換せずに元の値を返却しているが、エラー対応として問題ないか

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

## 動作確認状況
- ローカルで確認、サーバ側はnpm run test実施済

## 補足
- 相談、参考資料などがあれば
 2023-12-27 02:24:39 +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
makabe.t
f6c3f69801 Merged PR 657: ユーザー認証完了通知の実装 [U-113] 
## 概要
[Task3322: ユーザー認証完了通知の実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3322)

- アカウント認証完了後のパスワード通知メール送信処理を追加しました。
- テストでSendGridのメソッドを`overrideSendgridService`で上書きしている箇所について、個別の送信メソッドは不要なので削除しました。

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

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-12-25 05:59:10 +00:00
makabe.t
9f8ccc436f Merged PR 658: アカウント認証のお願い通知の実装 [U-102] 
## 概要
[Task3321: アカウント認証のお願い通知の実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3321)

- アカウント登録時の認証メールを送信する機能を追加しました。

## レビューポイント
- 既存のエラー処理はそのままにしていますが不自然な点はないでしょうか？

## UIの変更
- なし
## 動作確認状況
- ローカルで確認
 2023-12-25 05:38: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
makabe.t
3fc184b4af Merged PR 646: アカウント情報変更完了通知 [U-112] の実装 
## 概要
[Task3308: アカウント情報変更完了通知 [U-112] の実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3308)

- アカウント情報変更時の通知メール送信機能を追加しました。

## レビューポイント
- メールに渡す情報の取得内容に不自然な点はないでしょうか？

## UIの変更
- なし
## 動作確認状況
- ローカルで確認
 2023-12-22 00:18:11 +00:00
makabe.t
a6f56d71ee Merged PR 645: アカウント情報消去完了通知 [U-111] の実装 
## 概要
[Task3307: アカウント情報消去完了通知 [U-111] の実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3307)

-

## レビューポイント
- アカウント削除時の削除通知メール送信機能を追加しました。

## UIの変更
- メールに必要な情報を取得するために削除前に取得処理を入れていますが不自然な点はないでしょうか？

## 動作確認状況
- ローカルで確認
 2023-12-20 08:50:16 +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
makabe.t
9e1bc8944f Merged PR 641: ライセンス割当完了通知 [U-108] の実装 
## 概要
[Task3305: ライセンス割当完了通知 [U-108] の実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3305)

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

## レビューポイント
- テンプレートの適用内容に不自然な点はないでしょうか？
- テストでのメソッドの上書きは適切でしょうか？

## UIの変更
- なし
## 動作確認状況
- ローカルで確認
 2023-12-19 09:57:07 +00:00
makabe.t
11aa73f190 Merged PR 639: ライセンス発行完了通知 [U-107] の実装 
## 概要
[Task3304: ライセンス発行完了通知 [U-107] の実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3304)

- ライセンス発行完了時のメール送信機能を追加しました。

## レビューポイント
- テンプレートの内容は適切でしょうか？
- メールに渡す値の取得で不自然な点はないでしょうか？

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-12-19 01:25:57 +00:00
makabe.t
a676d65f0a Merged PR 633: HTMLテンプレートファイルの形式を決定 
## 概要
[Task3320: HTMLテンプレートファイルの形式を決定](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3320)

- HTMLテンプレートの形式を見ていただくにあたって、ライセンス注文キャンセルでテンプレートからメールを生成する処理を実装しました。
  - ライセンス注文キャンセルメールのテンプレートを追加しています。
- メール送信時にTOとCCに複数人を設定できるように修正しました。

## レビューポイント
- テンプレート中で置き換える文字列を定数として定義していますが違和感はないでしょうか？
- テンプレートの文言置き換え処理ですべてのパターンに引っかかるように正規表現で検索していますが問題ないでしょうか？
- HTMLテンプレート、メール表示内容に違和感はないでしょうか？

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

## 動作確認状況
- ローカルで確認
 2023-12-18 04:54:53 +00:00
湯本 開
1c39555bfc Merged PR 632: ライセンスオーダーのキャンセル受付通知 [U-106] の実装 
## 概要
[Task3303: ライセンスオーダーのキャンセル受付通知 [U-106] の実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3303)

- オーダーキャンセルが完了時にメールを送信する処理を追加
- SendGridServiceにテンプレートメールを送信するメソッドを追加
- Adb2cUserからメールアドレスを取得する方法が分散して実装してあったので、取り出す用のメソッドを定義してそれを使用するよう修正

## レビュー対象外
- 実際のメール送信処理部分は未実装なので対象外

## レビューポイント
- SendGridServiceに`sendTtemplateMailWithU{番号}`というメソッドを用意してメール送信を行う形式で問題ないか
  - from/toやCC等もメールの種別毎に異なるので、SendGridService側に具体的にどんなメールを送るかの責任を持たせる方針でいいか
- `sendMailWithU106` の引数に不足や認識間違いはないか
  - キャンセルを行った本人へのメールだけで本当によいか？（他の管理者には知らせないでいいか）等
  - to/cc等も考慮してチェックお願いします

- **特にMISOチーム向け** 依存関係の追加で壊れたテストを削除したが、別途DBテストを追加しないで問題なさそうか？
  - 問題ありそうでれば、別途テスト実装タスクを作る想定

## 動作確認状況
- npm run testが通るところまで確認
 2023-12-15 05:56:09 +00:00
makabe.t
ad715285c6 Merged PR 613: 外部連携APIのバリデータ見直し 
## 概要
[Task3279: files配下APIの対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3279)

- 外部連携API（以下のAPIが対象）のバリデータを修正しました。
  - ~~音声ファイルアップロード要求API~~
    - ~~GET /files/audio/upload-location~~
  - 音声ファイルアップロード完了API（タスク追加API）
    - POST /files/audio/upload-finished
  - タスクチェックアウトAPI
    - POST /tasks/{audioFileId}/checkout
  - タスクペンディングAPI
    - POST /tasks/{audioFileId}/suspend
  - タスクキャンセルAPI
    - POST /tasks/{audioFileId}/cancel
  - タスクチェックインAPI
    - POST /tasks/{audioFileId}/checkin
  - 音声ファイルダウンロード先取得API
    - GET /files/audio/download-location
  - テンプレートファイルダウンロード先要求API
    - GET /files/template/download-location
  - 次ファイル情報取得要求API
    - GET /tasks/next
  - 認証情報作成API
    - POST /auth/token
  - 通知登録API
    - POST /notification/register

## レビューポイント
- 対象APIに漏れはないでしょうか。
- バリデータの制約は適切でしょうか？

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-12-13 08:27:00 +00:00
saito.k
0f35789b91 IPアドレスログ出力に伴う変更の漏れ対応 2023-12-13 12:56:55 +09: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
saito.k
5ef222134e IPアドレス追加に伴う修正漏れ対応 2023-12-12 15:46:37 +09:00
saito.k
f1583cf783 Merged PR 611: 操作を特定できる文字列を追跡用のIDに追加する（IPアドレスもログに出力する） 
## 概要
[Task3265: IPアドレスを追跡用のIDに追加する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3265)

- MiddlewareでUUIDを発行しリクエストのヘッダに追加する
- 各コントローラーではヘッダからUUIDとIPアドレスを取得する
  - 取得したUUIDとADB2Cの外部IDでトラッキングIDを作成する
  - 作成したトラッキングIDとIPアドレスの繋がりをログに出力する。

## レビューポイント
- ADB2Cの外部IDがない場合にUnauthorized Userという文字列を入れているがほかの表現のほうが良いか
  - 外部IDもオプショナルにして入れなくてもよくする？
-

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

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

## 補足
- 相談、参考資料などがあれば
 2023-12-12 04:11:36 +00:00
makabe.t
a0da277c05 Merged PR 607: メールアドレス入力チェックの修正 
## 概要
[Task3232: メールアドレス入力チェックの修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3232)

- メールアドレス入力に対して規約に則ったチェックを実施するように修正しました。
  - 画面での入力チェックとAPIのバリデーションを修正しています。

## レビューポイント
- クライアントのチェック内容は規約に則った内容になっているでしょうか？
  - 正規表現の構文として不自然な点はないでしょうか？
- サーバー側のチェックをIsEmail＋禁止文字の追加という形で対応していますが適切でしょうか？

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-12-08 02:14:25 +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
水本 祐希
302f302473 Merged PR 597: ライセンス確認画面（第五階層）に会社名を表示する 
## 概要
[Task2762: ライセンス確認画面（第五階層）に会社名を表示する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2762)

- 何をどう変更したか、追加したライブラリなど
ライセンス確認画面（第5階層）の会社名を取得するAPIを実装
- このPull Requestでの対象/対象外
画面側の実装は別ブランチで対応するためここでは対象外

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

## 動作確認状況
- ポストマン、ユニットテスト
 2023-12-05 07:39:13 +00:00