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

412 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
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
saito.k
088e6afc85 Merged PR 454: ワークフロー一覧画面 
## 概要
[Task2735: ワークフロー一覧画面](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2735)

- ワークフロー一覧画面のデザイン反映
- 多言語対応
- 一覧取得API呼び出し

## レビューポイント
- デザイン反映に問題はないか
- フォルダ構成はこれでよいか
  - workflow配下に直置き

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

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

## 補足
- 相談、参考資料などがあれば
 2023-10-03 02:15:57 +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
水本 祐希
5cfe069b58 Merged PR 444: 画面実装（アカウント削除成功ページ） 
## 概要
[Task2715: 画面実装（アカウント削除成功ページ）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2715)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
・アカウント削除成功後に、アカウント削除成功ページに遷移するよう実装
・ログオフ状態にする

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

## レビューポイント
- 特にレビューしてほしい箇所
・レイアウトやメッセージの文言
・ログオフ状態にするタイミング
→Back to TOP Pageを押下時にログオフ状態になるようにしています
・Delete Accountボタン（ポップアップ上の）押下時のページ遷移のコード
・その他、漏れがないか

- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載

## 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/Task2715?csf=1&web=1&e=0M85t6

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

## 補足
- 相談、参考資料などがあれば
 2023-10-02 01:19:18 +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
saito.k
3e90261ecc エラーラベルのデザインを修正 2023-09-29 14:41:05 +09: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
makabe.t
ded673ec74 Merged PR 448: NestJSの環境変数をバリデータでチェックする 
## 概要
[Task2729: NestJSの環境変数をバリデータでチェックする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2729)

- 環境変数のバリデータを実装しました。
  - 起動時にバリデータのチェックに失敗したらエラーとなるようにしています。

## レビューポイント
- チェック対象の環境変数は適切か
  - チェック内容は適切か

## UIの変更
- なし

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

## 補足
- 参考：[環境変数をバリデーションしたい](https://zenn.dev/waddy/articles/nestjs-configuration-service#5.-%E7%92%B0%E5%A2%83%E5%A4%89%E6%95%B0%E3%82%92%E3%83%90%E3%83%AA%E3%83%87%E3%83%BC%E3%82%B7%E3%83%A7%E3%83%B3%E3%81%97%E3%81%9F%E3%81%84)
 2023-09-28 07:18:09 +00:00
saito.k
deb08d81d2 コメントを修正 2023-09-28 15:22:30 +09:00
saito.k
f289bb3d19 エラーラベルの文言修正 2023-09-28 14:44:49 +09:00
saito.k
9a01eff837 Merged PR 449: [Sp18-2 or 19-1]翻訳情報を翻訳Excelに適用し、Webアプリにも適用する 
## 概要
[Task2166: [Sp18-2 or 19-1]翻訳情報を翻訳Excelに適用し、Webアプリにも適用する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2166)

- 翻訳情報を取り込む

## レビューポイント
- 一通り画面は確認しましたが、各画面をざっと見ていただいて問題なさそうか確認していただきたいです。
- 文字数が変わったことにより、項目名が改行されているところがあるが、一行で表示できた方が良いでしょうか。
  - 個人的には問題ないと思っています。

## UIの変更
- 変更点が多いため、手元で動かして確認していただきたいです。

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

## 補足
- 相談、参考資料などがあれば
 2023-09-28 01:23:36 +00:00
oura.a
ba7ef1b4c6 Merged PR 439: [PBI1197残]ドロップダウンの挙動について統一する 
## 概要
[Task2713: [PBI1197残]ドロップダウンの挙動について統一する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2713)

ドロップダウンの選択肢表示を修正しました。

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

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

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

## 補足
なし
 2023-09-27 08:40:26 +00:00
saito.k
de13753ef6 Merged PR 447: [Sp18-2 or 19-1]翻訳情報を翻訳Excelに適用し、Webアプリにも適用する 
## 概要
[Task2166: [Sp18-2 or 19-1]翻訳情報を翻訳Excelに適用し、Webアプリにも適用する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2166)

- 不要な文言を削除
- 削除した文言が使用されている箇所を修正

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-09-27 08:12:25 +00:00
saito.k
8265ca38c8 Merged PR 440: 画面実装（テンプレートファイルアップロードPopup） 
## 概要
[Task2656: 画面実装（テンプレートファイルアップロードPopup）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2656)

- テンプレートファイルアップロードのAPI呼び出し周りを実装
  - SASトークン付きURL取得
  - Blobストレージへファイルアップロード
  - アップロード完了
- server側
  - `helmet`の`connect-src`を修正
  - SASトークン付きURLが想定と違っていたため修正
  - DBに保存するURLが想定と違っていたため修正

## レビューポイント
- `connect-src`の`self`以外はローカル環境のみの設定でよさそう？
- Popupの挙動で不足している箇所はあるか
  - アップロードファイルでチェックすべき内容等

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

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

## 補足
- 相談、参考資料などがあれば
 2023-09-27 06:36:24 +00:00
水本 祐希
bf4dc1d717 Merged PR 443: 画面実装（アカウント削除確認ポップアップ）マージ用 
## 概要
[Task2738: 画面実装（アカウント削除確認ポップアップ）マージ用](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2738)

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

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-09-26 01:25:39 +00:00
水本 祐希
1a0edee5c9 Merged PR 415: 画面実装（アカウント削除確認ポップアップ） 
## 概要
[Task2669: 画面実装（アカウント削除確認ポップアップ）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2669)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
アカウント削除ボタンの挙動を実装

- このPull Requestでの対象/対象外
・削除後のページ遷移は対象外

- 影響範囲（他の機能にも影響があるか）

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-09-26 00:52:46 +00:00
saito.k
9ca4ae61f8 Merged PR 434: 画面実装（テンプレートファイルアップロードPopupデザイン） 
## 概要
[Task2664: 画面実装（テンプレートファイルアップロードPopupデザイン）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2664)

- デザイン反映
- ファイルピッカーからファイル取得→storeに保存

## レビューポイント
- デザイン反映に不備はあるか
- 想定としてstoreに保持したfileを、Operationsでblobにアップロードする流れにしようとしているがよさそうか

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

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

## 補足
- 相談、参考資料などがあれば
 2023-09-25 07:53:45 +00:00
makabe.t
ecc44e58e0 Merged PR 438: API実装（テンプレートファイルアップロード完了API） 
## 概要
[Task2655: API実装（テンプレートファイルアップロード完了API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2655)

- テンプレートファイルのアップロード完了APIとテストを実装しました。

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

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-09-25 07:50:19 +00:00
makabe.t
f994c23b51 Merged PR 432: API実装（テンプレートファイルアップロード先取得API） 
## 概要
[Task2654: API実装（テンプレートファイルアップロード先取得API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2654)

- テンプレートファイルアップロード先取得APIとテストを実装しました。
  - フォルダパス＋SASトークンの形式で返却する。

## レビューポイント
- 返却URLは適切か
- BlobServiceでSASトークン発行を既存のメソッドとは別で用意したが構成は適切か
- UT用にBlobServiceのoverrideにメソッドを追加したが問題ないか。
- テストケースは適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-09-25 05:45:29 +00:00
saito.k
3f4d4ec436 Merged PR 427: 画面実装（テンプレートファイル一覧画面） 
## 概要
[Task2651: 画面実装（テンプレートファイル一覧画面）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2651)

- テンプレートファイル一覧画面を実装

## レビューポイント
- 取得方法やstoreの構成は問題ないか

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

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

## 補足
- 相談、参考資料などがあれば
 2023-09-22 07:36:23 +00:00
oura.a
f120f4f7e5 Merged PR 436: [PBI1197残]画面レイアウトについて確認する 
## 概要
[Task2712: [PBI1197残]画面レイアウトについて確認する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2712)

アカウント情報画面について、以下のデザイン修正を実施しました。
・「Save changes」ボタンを右側に移動
・表内の値が長かった場合に三点リーダでの省略表示から、改行して全体を表示する

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

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

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

## 補足
なし
 2023-09-22 07:15:54 +00:00
makabe.t
77eeb0ea83 Merged PR 426: ActiveWorktypeIDの画面実装修正 
## 概要
[Task2706: ActiveWorktypeIDの画面実装修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2706)

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

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-09-22 06:05:19 +00:00
makabe.t
f928aa4fd2 Merged PR 424: API実装（テンプレートファイル一覧取得API） 
## 概要
[Task2650: API実装（テンプレートファイル一覧取得API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2650)

- テンプレートファイル一覧取得APIとテストを実装しました。

## レビューポイント
- サービスの配置、リポジトリの呼び出しは適切か
- テストケースは適切か
- テスト用にtemplates配下にテンプレートファイル追加関数を追加したが適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-09-21 08:30:21 +00:00
masaaki
9ac40c00ac Merged PR 431: static_contentsフォルダが空の場合の対応を行う 
## 概要
[Task2719: static_contentsフォルダが空の場合の対応を行う](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2719)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）
  - statis_contents配下が空の場合、gitに認識されなくなってしまうので、コピー処理自体を削除しました
  - 今後復活させたくなった時のため、wikiに手順を追加しておきました。
https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_wiki/wikis/OMDSDictation_wiki/268/%E9%9D%99%E7%9A%84%E3%82%B3%E3%83%B3%E3%83%86%E3%83%B3%E3%83%84%E3%81%AB%E5%AF%BE%E3%81%97%E3%81%A6%E9%9D%99%E7%9A%84%E3%83%95%E3%82%A1%E3%82%A4%E3%83%AB%E3%82%92%E9%85%8D%E7%BD%AE%E3%81%97%E3%81%9F%E3%81%84

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

## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
  - 特にありません

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

## 補足
- 相談、参考資料などがあれば
 2023-09-21 07:59:09 +00:00
masaaki
cda3652817 Merged PR 430: 脆弱性試験ツール対応を元に戻す 
## 概要
[Task2694: 脆弱性試験ツール対応を元に戻す](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2694)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
    - ドメイン確認用のファイルについて削除しました。
- このPull Requestでの対象/対象外
    - すべて対象
- 影響範囲（他の機能にも影響があるか）
    - 無し

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

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

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

## 補足
- 相談、参考資料などがあれば
    - ローカル環境でnpm run build:localを実施していた場合、
       /app/dictation_server/build配下のファイル「YOweYATRY5PBN1G9d….html」を手動で削除する必要があります。
      動作上の悪影響はありませんが、ゴミなので削除願います。
 2023-09-21 06:52:15 +00:00
makabe.t
b38d2c44f7 Merged PR 428: API IF修正 
## 概要
[Task2714: API IF修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2714)

- 以下のAPIのIFを修正しました。
  - テンプレートファイル一覧取得API
  - テンプレートファイルアップロード完了API

## レビューポイント
- 想定通りの修正となっているか。

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-09-21 04:51:00 +00:00
maruyama.t
4a68653c69 Merged PR 423: API IF実装(アカウント削除API) 
## 概要
[Task2668: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2668)

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

## レビューポイント
メソッド名が適切か。

## UIの変更
なし

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

## 補足
- 相談、参考資料などがあれば
 2023-09-20 05:35:55 +00:00
oura.a
e7bd7b52fa Merged PR 425: 親アカウント名表示修正 
## 概要
[Task2687: 親アカウント名表示修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2687)

アカウント情報画面の親アカウント名表示対応を実施

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

## UIの変更
なし

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

## 補足
なし
 2023-09-20 02:34:02 +00:00
saito.k
75393d683d Merged PR 421: OptionItemポップアップサイズを大きくするデザイン依頼 
## 概要
[Task2678: OptionItemポップアップサイズを大きくするデザイン依頼](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2678)

- タイトルの通り

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-09-20 02:14:16 +00:00
makabe.t
cec740f65e Merged PR 418: API IF実装 
## 概要
[Task2649: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2649)

- テンプレートファイル周りで以下のAPIIFを実装し、OpenAPIを更新しました。
  - テンプレートファイル一覧取得API
  - テンプレートファイルアップロード先取得API
  - テンプレートファイルアップロード完了API

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

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-09-20 01:41:14 +00:00
saito.k
41e4fbb8de Merged PR 419: DBマイグレーション 
## 概要
[Task2662: DBマイグレーション](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2662)

- テンプレートファイルテーブルからdeleted_atを削除

## レビューポイント
-  内容に誤りはないか

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

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

## 補足
- 相談、参考資料などがあれば
 2023-09-20 01:38:45 +00:00
saito.k
32db216d6a Merged PR 416: ログアウト時にlogoutRedirectを使うようにする 
## 概要
[Task1829: ログアウト時にlogoutRedirectを使うようにする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1829)

- タイトルの通り
  - 該当箇所をすべて修正

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

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

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

## 補足
- B2Cの設定も修正
   - 不要なアクセストークンは発行しないように修正
 2023-09-20 01:36:57 +00:00
oura.a
8fee108a78 Merged PR 422: 成功時のスナックバー追加 
## 概要
[Task2685: 成功時のスナックバー追加](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2685)

情報更新成功時の成功スナックバー実装が漏れていたため、追加しました。

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

## UIの変更
なし
## 動作確認状況
ローカルで確認

## 補足
なし
 2023-09-19 10:32:33 +00:00
oura.a
2705fcee0a Merged PR 406: 画面実装（アカウント情報画面） 
## 概要
[Task2605: 画面実装（アカウント情報画面）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2605)

アカウント情報画面を実装しました。
※以下の処理は本PBIの対象外となりますので、レビュー対象外でお願いします
・「File Delete Setting」ボタン押下時の処理
・「Delete Account」ボタン押下時の処理

## レビューポイント
もっと簡潔に記載できる箇所はないか

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

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

## 補足
なし
 2023-09-19 10:01:19 +00:00
makabe.t
ec5df50aa0 Merged PR 420: 結合動作確認不具合対応 
## 概要
[Task2684: 結合動作確認不具合対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2684)

- Active WorktypeIDの変更について結合動作確認で見つかった不具合に対応しました。
  - ActiveWorktypeIDの変更がダイアログ表示の前にセレクトボックスに反映される
  - 変更成功時に成功スナックバーを表示する

## レビューポイント
- セレクトボックスの変更時のイベントの取り扱いは適切か

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

## 動作確認状況
- ローカルで確認
 2023-09-19 09:29:04 +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