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

310 Commits

Author SHA1 Message Date
saito.k
8ace80de74 Merged PR 517: API IF実装（代行操作用トークン生成） 
## 概要
[Task2904: API IF](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2904)

- 以下のAPIのIF実装
  - POST /api/auth/delegation-token
  - POST /api/auth/delegation-access-token
- OpenApiを生成
- 不要なas変換を削除

## レビューポイント
- リクエスト・レスポンスの型は認識通りか
  - 代行操作用トークン生成APIの引数のパラメータ名はよさそうか（delegatedAccountId）
- アクセストークン再生成APIについてはガードを付けず、service内で引数に受け取ったリフレッシュトークンを検証し、正しいロール、階層かチェックする方向でよさそうか（通常のアクセストークン再生成と同様に）

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

## 補足
- 相談、参考資料などがあれば
 2023-10-26 01:37:35 +00:00
makabe.t
08e5a9cd4a Merged PR 525: ファイルアップロード先取得APIの修正 
## 概要
[Task2935: ファイルアップロード先取得APIの修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2935)

- ファイルアップロード先取得APIの実装を修正しました。
  - ContorollerからServiceへ外部ユーザーIDを渡す想定の部分がアクセストークンをそのまま渡すようになっていたので引数を修正しました。
- ログを整理して引数がわかるようにしています。

## レビューポイント
- 共有
- ログ内容は適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-10-25 08:47:15 +00:00
maruyama.t
c283df9b0a Merged PR 521: API実装（ユーザ名取得API） 
## 概要
[Task2924: API実装（ユーザ名取得API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2924)

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

## レビューポイント
ユーザー存在チェックを行う必要性はあるか？

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

## 補足
- 相談、参考資料などがあれば
 2023-10-25 08:28:57 +00:00
masaaki
855cfbf347 Merged PR 527: terms初期値用のmigrateファイルが誤っていたのを修正する 
## 概要
[Task2901: terms初期値用のmigrateファイルが誤っていたのを修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2901)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- termsテーブルの初期値について、ドキュメントタイプが「EURA」で登録していたので「EULA」となるように修正しました

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

## UIの変更
- 無し

## 動作確認状況
- migrate upおよびdown実施＋ローカルで確認

## 補足
- 相談、参考資料などがあれば
 2023-10-25 06:44:59 +00:00
masaaki
4bedc9ec95 Merged PR 526: タスク2898：リフレッシュトークン処理の修正戻し 
## 概要
[Task2933: タスク2898：リフレッシュトークン処理の修正戻し](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2933)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- タスク2898で実施した比較演算子の修正について、バグの恒久対応が行われたので厳密等価演算子に戻しました。

## レビューポイント
- 特にありません

## UIの変更
- 無し

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

## 補足
- 相談、参考資料などがあれば
 2023-10-25 06:17:01 +00:00
maruyama.t
4b812fc0b3 Merged PR 520: API IF作成 
## 概要
[Task2919: API IF作成](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2919)

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

## レビューポイント
メソッド名がふさわしいか

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

## 補足
- 相談、参考資料などがあれば
 2023-10-24 08:35:31 +00:00
saito.k
48b45d2773 Merged PR 518: Entityのtransformer横展開対応 
## 概要
[Task2899: 横展開対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2899)

- Bigintがコード上でstringとして扱われるのを考慮し、number型に変換する処理を実装
- テスト実装
- 各Entityに展開

## レビューポイント
- 適用漏れはないか
- 実装内容に疑問はないか

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

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

## 補足
- 相談、参考資料などがあれば
 2023-10-24 07:54:46 +00:00
masaaki
7ea9f5b811 Merged PR 511: 直接の事象修正（ログイン時のリフレッシュトークン処理修正） 
## 概要
[Task2898: 直接の事象修正（ログイン時のリフレッシュトークン処理修正）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2898)

- リフレッシュトークン生成時の管理者判定について、stringとnumberでの比較となっているため厳密ではない等価演算子を使用するよう修正しました。

## レビューポイント
- 特にありません

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

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

## 補足
- 相談、参考資料などがあれば
 2023-10-19 10:27:35 +00:00
saito.k
83e923d15e mysqlで対応していないtypeを修正 2023-10-19 17:17:48 +09:00
saito.k
f553bfc95b Merged PR 501: strictNullChecks修正①（accounts,auth,Repositoiesのaccounts,common） 
## 概要
[Task2835: 修正①（accounts,auth,Repositoiesのaccounts,common）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2835)

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

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

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

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

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

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

## レビューポイント
-

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

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

## 補足
- 相談、参考資料などがあれば
 2023-10-19 01:04:14 +00:00
水本 祐希
434e3d9c27 Merged PR 502: users.repository.serviceのログ修正 
## 概要
[Task2859: users.repository.serviceのログ修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2859)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
idTokenとversionのログ出力が正常にできるよう修正

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

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-10-18 02:02:38 +00:00
水本 祐希
364bfb5135 Merged PR 504: 利用規約テーブルの初期値を設定 
## 概要
[Task2853: 利用規約テーブルの初期値を設定](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2853)

- termsテーブルの初期データを登録するmigrateファイルを追加しました。

## レビューポイント
- 特にありません

## UIの変更
- なし

## 動作確認状況
- migrate upでデータ登録されること、downで削除されることを確認

## 補足
- 相談、参考資料などがあれば
 2023-10-18 01:29:02 +00:00
makabe.t
a6206a624a Merged PR 500: ActiveWorktypeIDの削除処理を追加 
## 概要
[Task2861: ActiveWorktypeIDの削除処理を追加](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2861)

- 削除で指定したワークタイプIDをActiveWorktypeIDに指定していた場合にNULLをセットする処理を追加しました。

## レビューポイント
- ActiveWorktypeID設定内容は適切か
- テストケース追加に不足はないか

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-10-17 08:21:35 +00:00
saito.k
d2c2223acf Merged PR 497: strictNullChecks修正③（task,users ,Repositoriesのusers） 
## 概要
[Task2837: 修正③（task,users ,Repositoriesのusers）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2837)

- feature
  - tasks
  - users
- Repositories
  - users

## レビューポイント
- 該当箇所にコメントで記載

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

## 補足
- 相談、参考資料などがあれば
 2023-10-16 10:25:45 +00:00
makabe.t
c9bc6393c6 Merged PR 490: ワークタイプ削除API実装 
## 概要
[Task2611: ワークタイプ削除API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2611)

- ワークタイプの削除APIとテストを実装しました。
※API IF部分は対象外です。

## レビューポイント
- リポジトリのエラーチェックに問題はないか
- テストケースは適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-10-16 09:39:01 +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
makabe.t
45350d0ab8 Merged PR 485: API IF実装 
## 概要
[Task2610: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2610)

- WorkTypeID削除API IFを実装し、OpenAPI定義を更新しました。

## レビューポイント
- パスは適切か
- レスポンスは想定通りか

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-10-16 02:14:22 +00:00
saito.k
d258d569f7 Merged PR 483: strictNullCheck修正④（gateways ,notification） 
## 概要
[Task2838: 修正④（gateways ,notification）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2838)

- strictNullCheckの対応
  - gateways配下
  - feartures
    - notification

## レビューポイント

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

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

## 補足
- 相談、参考資料などがあれば
 2023-10-16 01:34:28 +00:00
oura.a
273ba588ce Merged PR 495: API作成（バージョン更新API） 
## 概要
[Task2804: API作成（バージョン更新API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2804)

同意済み利用規約バージョン更新APIを実装しました。

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

## UIの変更
なし

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

## 補足
なし
 2023-10-16 01:31:30 +00:00
oura.a
162470838d Merged PR 489: API作成（利用規約情報取得API） 
## 概要
[Task2808: API作成（利用規約情報取得API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2808)

利用規約情報取得APIを作成しました。

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

## UIの変更
なし

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

## 補足
なし
 2023-10-13 06:36:57 +00:00
makabe.t
69ff6f3432 Merged PR 493: API作成（アカウント情報取得（未認証時最小アクセス）API） 
## 概要
[Task2807: API作成（アカウント情報取得（未認証時最小アクセス）API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2807)

- 未ログインユーザーについて、IDトークンを受け取ってユーザの所属するアカウントの階層情報を返却するAPIを実装しました。

## レビューポイント
- ContorollerでIDトークンをデコードしているが問題ないか？
  - ※ログインAPIを参考にしています。
- テストケースは適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-10-13 05:33:02 +00:00
saito.k
370d143c2c Merged PR 473: strictNullCheckの対応を部分的に行う 
## 概要
[Task2795: 部分的に修正を行う](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2795)

- strictNullChecks対応
  - features
    - template
    - workflow
  - gateways
    - adb2c
  - repositories
    - template
    - workflow

## レビューポイント
- entityの修正内容
  - nullを追加する項目はあってるか
- adb2cの環境変数を取得している箇所
  - getOrThrowで値が取得できなければエラーになる関数があったので使用しています。
![image.png](https://dev.azure.com/ODMSCloud/6023ff7b-d41c-4fa7-9c6f-f576ba48c07c/_apis/git/repositories/302da463-a2d7-40f9-b2bb-6e8edf324fa9/pullRequests/473/attachments/image.png)

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

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

## 補足
- 相談、参考資料などがあれば
 2023-10-13 04:07:18 +00:00
oura.a
55b854af36 Merged PR 492: ビルドエラー修正 
## 概要
[Task2844: ビルドエラー修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2844)

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

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-10-12 09:17:35 +00:00
oura.a
274292c389 Merged PR 477: API修正（トークン生成API） 
## 概要
[Task2800: API修正（トークン生成API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2800)

既存のトークン生成APIに、利用規約バージョンのチェック処理を追加しました。
また、チェック処理で同意済みバージョンが最新でないときのエラー時にログアウトしないような処理を追加しました。

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

## UIの変更
なし

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

## 補足
なし
 2023-10-12 08:59:36 +00:00
湯本 開
49bd0e5ffe Merged PR 476: DeplymentSlotを使ってデプロイする形式でPipelineを稼働するよう構成する 
## 概要
[Task2816: DeplymentSlotを使ってデプロイする形式でPipelineを稼働するよう構成する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2816)

- 本番環境用のデプロイパイプラインを更新
- バックエンド側起動時にビルドバージョンをログに出す機能を実装

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

## 動作確認状況
- 本番環境で確認
 2023-10-12 06:30:43 +00:00
oura.a
a5b07596c1 Merged PR 486: IF不具合修正 
## 概要
[Task2840: api.ts最新化](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2840)

IFの修正を行いました。
・/api/termsがPOSTになっていたのでGETに修正
・/api/termsのtypesの記載が足りていなかったので修正

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

## UIの変更
なし

## 動作確認状況
swaggerUIで確認

## 補足
なし
 2023-10-12 05:37:25 +00:00
Kentaro Fukunaga
be475c29a6 Merged PR 481: 音声ファイルアップロードできる権限制御を外す 
## 概要
[Task2764: 音声ファイルアップロードできる権限制御を外す](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2764)

- Authorでなくとも音声ファイルのアップロードやタスク追加をするケースがあるとのことだったので対応しました。
   - アップロード要求APIと音声ファイルアップロード完了APIのRoleGuardを外しました

## レビューポイント
- ほか気になるケースあれば（この挙動はできるようにしなくて大丈夫？とか）

## 動作確認状況
- ローカルでツールを使ってアップロードでき、タスク追加できるところまで確認
 2023-10-11 23:28:41 +00:00
makabe.t
d48afdbffd Merged PR 468: API実装（ワークフロー削除API） 
## 概要
[Task2785: API実装（ワークフロー削除API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2785)

- ワークフロー削除APIとテストを実装しました。

## レビューポイント
- リポジトリの削除ロジックは適切か
- テストケースは適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-10-11 09:14:41 +00:00
saito.k
8e317f5aca Merged PR 478: POデモ修正対応 
## 概要
[Task2821: POデモ修正対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2821)

- リテラル修正
  - 追加Popupのタイトル
  - AuthorIDの初期値
- ワークフロー追加・更新の重複チェックの条件指定を修正
  - undefined,nullのままだと条件指定しないという挙動になるためIsNull（）を使用
- 該当ケースをテストに追加

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

## UIの変更
- タスクのなか

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

## 補足
- 相談、参考資料などがあれば
 2023-10-10 07:22:00 +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
oura.a
1eedc5c0be Merged PR 474: ビルドエラー修正 
## 概要
[Task2815: ビルドエラー修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2815)

ビルドエラー修正

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

## UIの変更
なし

## 動作確認状況
UT確認

## 補足
なし
 2023-10-06 05:50:30 +00:00
oura.a
34cf80d636 Merged PR 469: API-IF実装 
## 概要
[Task2806: API-IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2806)

以下APIのIFを実装しました。
・アカウント情報取得（未認証時最小アクセス）API
・利用規約情報取得API
・同意済バージョン更新API
またトークン生成APIのIFにコメントを追加しました。

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

## UIの変更
なし

## 動作確認状況
swaggerUIで動作確認

## 補足
なし
 2023-10-06 05:17:43 +00:00
makabe.t
f70e266e85 Merged PR 465: API実装（ワークフロー更新API） 
## 概要
[Task2776: API実装（ワークフロー更新API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2776)

- ワークフロー編集APIとテストを実装しました。

## レビューポイント
- リポジトリでのチェック処理は適切か
- テストケースは適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-10-06 00:10:47 +00:00
makabe.t
983726eaf3 Merged PR 467: API IF実装 
## 概要
[Task2784: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2784)

- ワークフロー削除APIのIFを実装しOpenAPI定義を更新しました。
- アプリで使用しない環境変数を削除し、チェック対象から外しました。
  - `KEY_VAULT_NAME`

## レビューポイント
- APIのパスは適切か
- 対応する環境変数は足りているか

## UIの変更
- なし
## 動作確認状況
- ローカルで確認
 2023-10-05 09:41:20 +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
makabe.t
964077a480 Merged PR 460: Author一覧取得API実装 
## 概要
[Task2748: Author一覧取得API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2748)

- Author一覧を取得するAPIとテストを実装しました。

## レビューポイント
- テストケースは適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-10-05 07:49:55 +00:00
makabe.t
c4c2038e6e Merged PR 458: ワークフロー追加API実装 
## 概要
[Task2739: ワークフロー追加API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2739)

- ワークフロー追加APIとテストを実装しました。

## レビューポイント
- リポジトリのチェックロジックは適切か
- テストケースは適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-10-05 00:20:42 +00:00
makabe.t
6baeb0b049 Merged PR 463: API IF実装 
## 概要
[Task2775: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2775)

- ワークフロー編集APIのIFを実装し、OpenAPIを更新しました。

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

## UIの変更
- なし
## 動作確認状況
- ローカルで確認
 2023-10-05 00:11:41 +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
oura.a
d942dc73f1 Merged PR 457: Accountレコード削除時に同時に削除されるよう外部キー制約を追加 
## 概要
[Task2766: Accountレコード削除時に同時に削除されるよう外部キー制約を追加](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2766)

各テーブルに外部キー制約を追加しました。
アカウント削除時にアカウントテーブルのデータを削除した際、ON DELETE CASCADEにより関連項目をすべて削除する用途になります。

## レビューポイント
設定内容は適切か。
設定箇所に過不足はないか。
マイグレーションの途中で元データの不整合などで失敗した場合、それまでに外部キー制約の追加・削除に成功していた分が巻き戻らなかったのですが、何か巻き戻す方法はあるでしょうか？

## UIの変更
なし

## 動作確認状況
ローカルでmigrate Up/Downの動作を確認

## 補足
なし
 2023-10-03 02:22:19 +00:00
makabe.t
1cc7a0141d Merged PR 453: ワークフロー一覧取得API実装 
## 概要
[Task2736: ワークフロー一覧取得API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2736)

- ワークフロー一覧取得APIとテストを実装しました

## レビューポイント
- リポジトリの取得処理は適切か（リレーションなど）
- ADB2Cからの取得処理は適切か
- サービスでのワークフローの整形処理は適切か
- テストケースは適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-10-03 01:14:18 +00:00
makabe.t
65f80b9a5b Merged PR 459: パイプラインテストエラー修正 
## 概要
[Task2773: パイプラインテストエラー修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2773)

- パイプラインでのテストエラー対応のため環境変数のチェックを外しました。

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

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-10-02 08:11:36 +00:00
makabe.t
3e0c483b57 Merged PR 456: NestJSの環境変数を整理してバリデーションを修正 
## 概要
[Task2769: NestJSの環境変数を整理してバリデーションを修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2769)

- 非必須の環境変数をチェックしないようにオプショナルを修正しました
- ローカルでのDBマイグレーションに必要なため、コンテナ起動時の`.env`ファイル読み込みを追加しました。
- 環境変数についてDB関連項目を残し、すべて`.env.local`に移動しました。

## レビューポイント
- 環境変数に対するチェックのオプショナル設定は適切か
- 環境変数をlocalに移動させたが問題ないか。

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
## 補足
- 不足している環境変数についてレビュー完了後にAppService、Wikiを整備しておきます。
 2023-10-02 07:13:34 +00:00
makabe.t
90a4d99cf5 Merged PR 446: DBマイグレーション 
## 概要
[Task2733: DBマイグレーション](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2733)

- 以下のテーブルを追加するマイグレーションファイルを追加しました。
  - ワークフローテーブル
  - ルーティング候補テーブル

## レビューポイント
- テーブル名、カラム名は適切か
- 外部キー制約は適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
  - migurate up/down
 2023-10-02 01:45:04 +00:00
oura.a
98e207f19f Merged PR 455: ユーザー退避テーブル修正 
## 概要
[Task2768: ユーザー退避テーブル修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2768)

ユーザー退避テーブルに対して以下の修正を行いました。
・accepted_terms_versionのNOT NULLを削除
　（オリジナルのユーザーテーブルで後から修正を入れていたのですが、退避テーブルへの反映が漏れていました）
・DEFAULTを削除
・作成時刻などにオリジナルテーブルのものである旨のコメントを追加
・archived_atカラムを追加

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

## UIの変更
なし

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

## 補足
なし
 2023-09-29 05:44:00 +00:00
oura.a
d016242bab Merged PR 452: DBマイグレーション（ライセンス系の退避テーブル作成） 
## 概要
[Task2765: DBマイグレーション（ライセンス系の退避テーブル作成）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2765)

以下テーブルの退避用テーブルを作成しました。
・ライセンス
・ライセンス割り当て履歴

## レビューポイント
保持するべきではない情報は持っていないか。

## UIの変更
なし

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

## 補足
両テーブルとも、もともとのテーブルをほぼそのままの形で作成しています。
（idのAUTO_INCREMENTだけ外しています）
 2023-09-29 02:51:49 +00:00
makabe.t
879169c3c7 Merged PR 450: ワークフローAPI IF実装 
## 概要
[Task2737: ワークフローAPI IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2737)

- Workflow関連で以下のAPIIFを実装し、OpenAPIを更新しました。
  - ワークフロー一覧取得API
  - ワークフロー追加API
  - Author一覧取得API

## レビューポイント
- パラメータは想定通りか
- バリデーションは適切か
- パスは適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-09-28 10:06:51 +00:00
oura.a
423e5ab1e3 Merged PR 445: DBマイグレーション（削除ユーザの情報退避テーブル） 
## 概要
[Task2682: DBマイグレーション（削除ユーザの情報退避テーブル）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2682)

削除ユーザー退避テーブルを作成しました。
license_allocation_historyテーブルにaccountIdのカラムを追加しました。
また、ライセンス割り当て・割り当て解除時にlicense_allocation_historyテーブルにaccountIdを登録するようにしました。

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

## UIの変更
なし

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

## 補足
削除ユーザー退避テーブルは現在のユーザーテーブルをほぼそのままの形で作成しています。
（idのAUTO_INCREMENTだけ外しています）
 2023-09-28 08:16:20 +00:00