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

515 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
saito.k
ad49a19f04 Merged PR 484: NotificationHubsの通知フォーマット 
## 概要
[Task2820: NotificationHubsの通知フォーマット](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2820)

- NotificationHubsで通知を行う処理を修正
  - 通知内容を変更
    - デスクトップ側でレンダリングを行うため、こちらからは必要なデータを送信する

- 通知処理を読んでいる箇所を修正
  - タスクキャンセル
  - チェックアウト候補変更
  - 音声ファイルアップロード完了（タスク作成）

## レビューポイント
- taskService内で通知をする箇所が複数あったのでプライベートメソッドとして切り出したが、fileServiceの通知処理もまとめたほうが良いか。
  - まとめる場合は、どこに切り出すか。
    - sendNotify(今回作成したプライベートメソッド)に、TasksRepositoryServiceやUserGroupsRepositoryServiceを引数に追加して共通関数としてCommonのどこかに配置する？
  - notificationhub.service.tsのnotifyメソッドの中で通知内容のための情報取得～通知内容作成まで全部行うようにする？

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

## 補足
- 相談、参考資料などがあれば
 2023-11-15 02:33:03 +00:00
maruyama.t
36401d0542 Merged PR 579: pipeline不具合対応 
## 概要
[Task3086: pipeline不具合対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3086)

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

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-11-14 13:27:28 +00:00
maruyama.t
eb3c7e55bd Merged PR 573: 音声ファイルアップロードAPI修正 
## 概要
[Task3069: 音声ファイルアップロードAPI修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3069)
[Task3070: 音声ファイルダウンロードAPI修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3070)
[Task3071: テンプレートファイルダウンロードAPI修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3071)
修正内容かぶるため、3本まとめてレビューお願いします。
- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 音声ファイルアップロードAPIを修正
　・第五階層の場合のみチェックを追加
　　・アカウントがロックされている場合、エラー
　　・ユーザーにライセンスが未割当の場合、エラー
　　・ユーザーに紐づいたライセンスが有効期限切れの場合、エラー
- 音声ファイルダウンロード、テンプレートファイルダウンロードAPIを修正
　・第五階層の場合のみチェックを追加
　　・ユーザーにライセンスが未割当の場合、エラー
　　・ユーザーに紐づいたライセンスが有効期限切れの場合、エラー
- 外部連携アプリ側の挙動の変化については考慮しない。
- ログ強化は別タスクで対応中。
- 影響範囲（他の機能にも影響があるか）
ファイル操作以外は影響なし。
旧式のユニットテストを修正。

## レビューポイント
- 音声ファイルアップロードのユニットテストを最新の状態にしたが、不足していないか。
~~- users.repositoryにユーザに紐づくライセンスが現在有効かどうかの判定を入れ込み、共通的に呼び出すようにしたが使いづらくないか（ライセンスが紐づいていない場合と有効期限切れの場合エラーとし、それ以外はtrueが帰る点について）~~

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

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

## 補足
- 相談、参考資料などがあれば
 2023-11-14 13:16:46 +00:00
masaaki
40162ef3af Merged PR 568: Azure Function App上でのDBアクセス・メール送信疎通実施 
## 概要
[Task2867: Azure Function App上でのDBアクセス・メール送信疎通実施](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2867)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
  - ライセンスアラート処理がAzureFunction上で動作するよう構成変更を行いました
    - DBアクセス、SendGrid、AADB2Cアクセスに必要な環境変数を追加
    - kv-odms-secret-devのアクセスポリシにfunc-odms-dictation-devを追加
    - ネットワーク設定にvnet-odms-network-dev/snet-odms-integration-devを追加
  - プロジェクトの内容もいくつか修正しています
    - 最新のpackage.jsonに合わせてpackage-lock.jsonを生成
    - licenseAlertのスケジュールを毎日1:00（UTC）に変更
    - ビルド用にDockerfileを復活
- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）

## レビューポイント
- 特段注視する部分はありません。ソース上の修正はスケジュールの変更のみです。

## UIの変更
- 無し

## 動作確認状況
- unitテスト実施、ローカルで確認

## 補足
- 相談、参考資料などがあれば
 2023-11-14 12:08:30 +00:00
oura.a
aeb2eb392a Merged PR 572: メール内容多言語対応、可変文字列対応 
## 概要
[Task3022: メール内容多言語対応、可変文字列対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3022)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
  - 仮だったメール内容について、ユーザ提示の内容とし、可変部分の対応を行いました。
- このPull Requestでの対象/対象外
  - 複数言語の対応について、現状は各言語の内容を縦に並べる形としています。
    11/14現在、奥澤さんが調査・調整中で、最終的な他言語対応は
  「プロダクト バックログ項目 1660: メール送信機能の実装」の中で実施になります。
  - [メール内容の外出しなど、編集しやすくする対応についても「プロダクト バックログ項目 1660: メール送信機能の実装」にて実施になります。
- 影響範囲（他の機能にも影響があるか）

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-11-14 06:51:13 +00:00
makabe.t
effbfe9d46 Merged PR 575: タイピスト割り当てで、未認証のユーザーは割り当てられないようにする 
## 概要
[Task2818: タイピスト割り当てで、未認証のユーザーは割り当てられないようにする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2818)

- 以下のAPIについて未認証のタイピストを割り当て内容にユーザー取得処理とテストを修正しました。
  - Typist一覧取得
  - Workflow追加
  - Workflow編集
  - ユーザーグループ追加
  - ユーザーグループ編集
  - 割り当て候補変更

## レビューポイント
- 対応箇所は適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-11-14 02:30:46 +00:00
makabe.t
5c0d5f6476 Merged PR 574: 未認証のAuthorIDは選択できないようにする 
## 概要
[Task2819: 未認証のAuthorIDは選択できないようにする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2819)

- Author取得APIではメール認証済みのユーザーのみ取得するように修正しました。
- Workflowの追加/編集APIでメール未認証のAuthorを指定して実行するとエラーとなるように修正しました。
- 修正APIに対応したテストケースを追加しました。

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

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-11-14 02:10:32 +00:00
makabe.t
ee161a405f Merged PR 560: ユーザー情報取得API実装 
## 概要
[Task3036: ユーザー情報取得API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3036)

- ユーザー関連情報取得APIのIFについてレスポンスを修正しています。
  - AuthorIDを非必須のパラメータにしています。
- ユーザー関連情報取得APIの中身を実装しました。

## レビューポイント
- オプションアイテムのタイプを定数に置いたDictionaryで数値に変換していますが、定数の置き方として不自然ではないでしょうか。

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-11-14 01:47:27 +00:00
maruyama.t
35923f84e2 Merged PR 576: dev環境不具合修正 
## 概要
[Task3046: dev環境不具合修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3046)

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

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-11-13 06:01:49 +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
oura.a
7c16e7c358 Merged PR 559: ライセンスアラート処理実装（メール内容固定） 
## 概要
[Task3021: ライセンスアラート処理実装（メール内容固定）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3021)

ライセンスアラート処理を実装しました。

## レビューポイント
取得している情報に過不足はないか。
処理の構成に問題がないか。
※redis対応は別タスクとなりますので、adb2cへのアクセス効率はレビュー対象外でお願いします
※メールの内容は別タスクで作成しますので、レビュー対象外でお願いします。

## UIの変更
なし

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

## 補足
UTでメールを送信した、していないを判断する方法が分からず、ひとまずconsoleログの出力の有無で判断しています。
 2023-11-10 07:57:18 +00:00
湯本 開
7421203bc4 Merged PR 567: テスト用Attributeを追加 
## 概要
[Task3060: テスト用Attributeを追加](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3060)

- E2Eテストから指定しやすいように代表的な要素（ログアウトボタン等）に一意の名前を付ける実装を追加
  - `data-*` はカスタムデータ属性と呼ばれるもので、「標準的な取り扱い方」が存在しない属性
    - https://developer.mozilla.org/ja/docs/Learn/HTML/Howto/Use_data_attributes
  - 一般的に、E2Eテストで「このボタン」とか「このラベル」とかを指定したい時に使われる
    - https://docs.cypress.io/guides/references/best-practices
  - 本番環境などでも属性としては残したままにする想定だが、`data-test-id` 等の見るからにテスト用の名前を指定すると、テスト処理が入っているような変な誤解を招きそうなので名前は `data-tag` とした
    - 実際にはテストにも使える、副作用が特にない属性

## レビューポイント
- 動作が変わるような変更が入り込んでしまっていないか
- タグ名は妥当か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-11-10 03:50:33 +00:00
saito.k
4b0110856c Merged PR 566: ｘｘｘのサンプルページを消し去る 
## 概要
[Task3055: ｘｘｘのサンプルページを消し去る](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3055)

- 仮で置いてあったサンプルページ削除
  - タブからも削除
  - ルーティングからも削除
- ログイン成功時、階層とロールに応じて遷移する画面を変更する処理を追加

## レビューポイント
- 遷移先を変更する処理は関数に切り出した方が良いか？
  - ほかで使わないからべた書きでも良いかなと思い、こうしました。

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

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

## 補足
- 相談、参考資料などがあれば
 2023-11-10 02:22:34 +00:00
makabe.t
6d163a9067 Merged PR 565: 代行操作中はサインアウトを非活性or非表示にする 
## 概要
[Task3066: 代行操作中はサインアウトを非活性or非表示にする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3066)

- 代行操作中はサインアウトボタンが非活性となるように修正しました。
- ログアウト時に自ユーザー情報取得APIを呼ばないように修正しました。
  - アクセストークンの削除後に呼ばれ、401エラーになるため

## レビューポイント
- 代行操作中はサインアウトボタンのマウスイベントを無効にすることで非活性としていますが対応として適切でしょうか？
- ログアウト時（アクセストークンがない場合）はヘッダで自ユーザー情報取得APIを呼ばないように修正しましたが、処理として適切でしょうか？
  - エラーになる呼び出しはしないようにしたい＆使っているのはヘッダでのみなのでほかの箇所への影響はない認識です。

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

## 動作確認状況
- ローカルで確認
 2023-11-09 06:40:14 +00:00
水本 祐希
b5449640be Merged PR 561: 認証メールのURLの末尾に"が入っていて不正なリンクになっている問題を修正する 
## 概要
[Task3000: 認証メールのURLの末尾に"が入っていて不正なリンクになっている問題を修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3000)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
認証メールの末尾にある「”」を削除し適切な認証メールに修正

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

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-11-09 02:38:17 +00:00
oura.a
09ff4327ee Merged PR 564: Revert "Merged PR 548: ログイン回り修正 
Revert "Merged PR 548: ログイン回り修正

## 概要
[Task1828: IDトークンを一度しか使えないようにする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1828)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
ログイン時にRedisを参照し、同じIDトークンでアクセスされていた場合はアクセスを拒否する。
初めて認証で使われるIDトークンの場合は、Redisに保存する（有効期限は３００秒（IDトークンの有効期限の最小値））
ログイン後、クライアントはローカルストレージの不要な情報を破棄する（accessToken,refreshToken,displayInfo以外）
- 影響範囲（他の機能にも影響があるか）
新規機能のため、なし。

## レビューポイント
- keyの形式id-token:{idトークンの中身}は想定通りか。
- ログイン後のローカルストレージの状態が想定通りか。
- controllerやservice層の実装箇所が妥当か。
- stg環境、本番環境のIDトークンの有効期限を最小値に設定するタスクを備忘用PBIに切り出しました。
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%2021-1?workitem=3041

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

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

## 補足
- 相談、参考資料などがあれば"

Reverted commit `0cdb0f42`.
 2023-11-08 02:55:28 +00:00
saito.k
f79d8d395d dev動作確認の不具合修正 2023-11-08 11:39:41 +09:00
水本 祐希
0cdb0f4267 Merged PR 548: ログイン回り修正 
## 概要
[Task1828: IDトークンを一度しか使えないようにする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1828)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
ログイン時にRedisを参照し、同じIDトークンでアクセスされていた場合はアクセスを拒否する。
初めて認証で使われるIDトークンの場合は、Redisに保存する（有効期限は３００秒（IDトークンの有効期限の最小値））
ログイン後、クライアントはローカルストレージの不要な情報を破棄する（accessToken,refreshToken,displayInfo以外）
- 影響範囲（他の機能にも影響があるか）
新規機能のため、なし。

## レビューポイント
- keyの形式id-token:{idトークンの中身}は想定通りか。
- ログイン後のローカルストレージの状態が想定通りか。
- controllerやservice層の実装箇所が妥当か。
- stg環境、本番環境のIDトークンの有効期限を最小値に設定するタスクを備忘用PBIに切り出しました。
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%2021-1?workitem=3041

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

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

## 補足
- 相談、参考資料などがあれば
 2023-11-08 02:22:20 +00:00
oura.a
86d17d6729 Merged PR 552: dictation_serverからソースコードを複製 
## 概要
[Task2977: dictation_serverからソースコードを複製](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2977)

新たに構築したdictation_functionで以下のことをできるようにしました。
・RDBへの接続
・sendgridでのメール送信
・jestでのテスト実行
※とりあえず動くことを目標としているため、DB接続のパラメータやsendgridのAPIキーなどがベタ打ちになっています。機能実装時には外出ししたファイルから読み込めるようにします。

## レビューポイント
・フォルダ構成は適切か
・RDB、sendgrid、jestに関するもので不足しているものがないか
・今後機能開発を始めるにあたり、他に必要なものがないか

## UIの変更
なし

## 動作確認状況
ローカルでjestによるテストを実施し、以下を確認
・RDBからデータが取得できる
・メールが送信され、設定したアドレスで受信できる

## 補足
なし
 2023-11-08 00:46:26 +00:00
saito.k
1e4a545bf8 Merged PR 562: キャンセルAPI修正 
## 概要
[Task2972: キャンセルAPI修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2972)

- キャンセル処理に自動ルーティングを追加

## レビューポイント
- 追加したテストケースは足りているか
- 自動ルーティングを修正したが修正箇所は問題ないか（To : 福永さん）
  - 特にworktypeが空文字だった時の挙動を修正したので、そこが業務要件とあっているか
    - コメントがある場所

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

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

## 補足
- 相談、参考資料などがあれば
 2023-11-08 00:25:45 +00:00
makabe.t
c9124f8661 Merged PR 563: develop動作確認不具合修正 
## 概要
[Task3042: develop動作確認不具合修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3042)

- ライセンスページで代行操作中の挙動を修正しました。
  - 自アカウント取得APIにガードを設定して実行可能に修正
  - 代行操作中は第五階層ユーザーとして扱われるように修正

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

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-11-07 10:23:15 +00:00
saito.k
36716dc408 Merged PR 547: 音声ファイルアップロード完了API修正（repository実装含む） 
## 概要
[Task2971: 音声ファイルアップロード完了API修正（repository実装含む）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2971)

- 音声ファイルアップロード完了API修正
  - 自動ルーティング処理を追加
    - authorIDとworktypeの組み合わせでワークフロー（ルーティングルール）を取得し、そのワークフローに従って、タスクのチェックアウト候補を設定する。
  - チェックアウト候補に設定したユーザーに対して通知を行う処理を追加

## レビューポイント
- 自動ルーティング処理を実装しているメソッドのメソッド名はこれでよいか
  - ほかに思いつかなかったので
- AudioOptionItemのentityの定義はあっている？
  - がタスクにあるaudio_file_idに紐づいている感じになっている
- 自動ルーティング処理で失敗したときの挙動は認識あっているか
  - エラーログだけ出してAPIとしては成功とする
- テストケースは足りているか
- 古い形式で記述されていたタスク作成のテストを新しい形で作り替えたが、反映漏れている部分はあるか

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

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

## 補足
- 相談、参考資料などがあれば
 2023-11-07 08:48:53 +00:00
makabe.t
8a2ca2b786 Merged PR 519: 代行操作で使用するAPIのガード修正＋ログ修正 
## 概要
[Task2907: 代行操作で使用するAPIのガード修正＋ログ修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2907)

- 代行操作するAPIについてガードに代行操作許可の設定を追加しました。
  - ガードの引数で設定したAPIだけが代行操作許可となるように設定しています。
- APIのバリデーションで意図せぬエラーが発生する問題に対処しました。

※ログは未適用なのでガードの設定だけ確認をお願いします。

## レビューポイント
- ガードを設定する対象は適切か
- ガード内での代行操作許可のロジックはホワイトリスト形式の実装として適切か
  - アクセストークンが代行操作用の（代行操作ユーザーIDがある）場合には、ガードで代行操作が許可されているかをチェックするようにしています。

※ログの代行操作への適用は未実装です。

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-11-07 08:41:47 +00:00
makabe.t
0212c61bbc Merged PR 550: 画面実装（トークンを定期的に更新する仕組み） 
## 概要
[Task2910: 画面実装（トークンを定期的に更新する仕組み）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2910)

- タイマーで定期的に代行操作用のアクセストークンを更新する処理を実装しました。

## レビューポイント
- 通常のアクセストークンのタイマー内で同じタイミングでチェックするように実装していますが分けたほうがいいなどありますでしょうか？

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

## 動作確認状況
- ローカルで確認
 2023-11-07 07:15:25 +00:00
makabe.t
ef4f22029b Merged PR 551: 画面実装（代行操作中のヘッダー表示） 
## 概要
[Task2911: 画面実装（代行操作中のヘッダー表示）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2911)

- 代行操作中に表示されるヘッダのタブを以下のタブのみに設定しました。
  - User
  - Workflow
  - License

## レビューポイント
- 代行操作タブ表示の判断材料としてヘッダコンポーネント内でselectorで代行操作の有無を判断してタブ表示用メソッドに渡していますが、データの取り扱いとして不自然なところはないでしょうか？

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

## 動作確認状況
- ローカルで確認
 2023-11-07 06:25:42 +00:00
makabe.t
bc110712b0 Merged PR 558: 代行操作の開始/終了時にダイアログを表示 
## 概要
[Task3032: 代行操作の開始/終了時にダイアログを表示](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3032)

- 代行操作の開始/終了時に確認ダイアログを表示するように修正しました。

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

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

## 動作確認状況
- ローカルで確認
 2023-11-07 05:01:06 +00:00
makabe.t
e38112924d Merged PR 557: ヘッダの表示文言がクッションページで正常に出るように修正 
## 概要
[Task3031: ヘッダの表示文言がクッションページで正常に出るように修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3031)

- 未ログイン状態のヘッダについて、翻訳文言の反映が漏れていたので対応しました。

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

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

## 動作確認状況
- ローカルで確認
 2023-11-07 01:29:09 +00:00
湯本 開
0da1aeab81 Merged PR 555: Stagingパイプラインを作成する（デプロイ）/修正 
## 概要
[Task2843: Stagingパイプラインを作成する（デプロイ）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2843)

- Staging環境Pipelineで部分的に間違ってprod指定したいた箇所を修正

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

## 動作確認状況
- なし
 2023-11-06 08:49:18 +00:00
湯本 開
f4e2d47bdd Merged PR 553: Stagingパイプラインを作成する（デプロイ） 
## 概要
[Task2843: Stagingパイプラインを作成する（デプロイ）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2843)

- StagingビルドだけしていたPipelineにデプロイ処理を追加
- adminによるmigrationが反映されていなかったので反映

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

## 動作確認状況
- なし
 2023-11-06 08:44:24 +00:00
saito.k
deb3431d74 Merged PR 531: 画面実装（代行操作） 
## 概要
[Task2909: 画面実装（代行操作）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2909)

- ディーラーユーザーが代行操作用トークンを取得して、第五階層ユーザーの代行操作ができる実装をしました。
  - 代行後に各対象タブが代行操作デザインで表示され、代行対象として操作できるようになっています。
    - 各APIの呼び出しについて代行操作用トークンがあればそちらを使うように実装しています。
 - 代行操作のタブ、ページ移動時に代行操作を維持するために遷移をリンクから`useNavigate`に変更しました。

## レビューポイント
- 代行操作用トークンの取り扱いについて
  - APIからのトークン取得後、`store.auth`に代行操作用トークンを保存し、利用時には関数を使って間接的に呼でいますが構成として不自然な点はないでしょうか？
    - トークン取得関数では代行操作用トークンがあればそれを、なければ通常のトークンを取得するようにしています。
      - これはAPIのトークンを設定する際にトークンを取得側では代行操作中か意識せずに一つの関数を呼べばいいようにするためです。
  - 代行操作用トークンの保存は通常のトークン保存と同様に`operation`から`Slice`に設定したSet関数を呼ぶことでstateに保存していますが、使い方として気になることはないでしょうか？

※代行操作中に表示するタブの制限と代行操作トークンの更新処理は別タスクでの実施予定です。

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

## 動作確認状況
- ローカルで確認
 2023-11-06 08:28:40 +00:00
oura.a
42dab2a45d Merged PR 532: function構築の動作確認用ブランチ 
## 概要
[Task2954: function構築の動作確認用ブランチ](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2954)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
  - dev環境にfunctionAppが動作する環境を構築しました。
      - 新規にDICTATION_FUNCTIONプロジェクトを追加
      - 関数アプリリソース「funcapp-odms-dictation-dev」を追加
      - 「funcapp-odms-dictation-dev」に付随してストレージアカウント「saomdsdevelopfuncapp」を追加
      - pipeline「ODMS Function Deploy」を追加
  - 構築の流れをwikiに更新しています。
    https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_wiki/wikis/OMDSDictation_wiki/280/FunctionApp%E3%81%AE%E4%BD%9C%E6%88%90
- このPull Requestでの対象/対象外
  - configurationsについて、結構差分がたまっている状態でした。本タスクで更新したのは以下になり、それ以外は対象外です。
    - dev-application-rg.json：funcapp-odms-dictation-dev
    - dev-application-rg.json：saomdsdevelopfuncapp
  - pipelineについては精査が必要です。「タスク 2992: pipelineの最適化」にて対応予定
- 影響範囲（他の機能にも影響があるか）

## レビューポイント
- 特にレビューしてほしい箇所
  - .devcontainerの内容について、VSCodeからコンテナで動かすためDICTATION_SERVERから取得しています。入れておくと問題のある設定があれば指摘いただければと思います。
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載

## UIの変更
- 無し

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

## 補足
- 相談、参考資料などがあれば
 2023-11-06 00:29:47 +00:00
makabe.t
d0f4971dc9 Merged PR 544: ヘッダの表示文言（タブ・タイトル）を翻訳ファイルを参照するように修正する 
## 概要
[Task2895: ヘッダの表示文言（タブ・タイトル）を翻訳ファイルを参照するように修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2895)

- ヘッダに表示するタイトルとタブの文言を翻訳ファイルに定義したラベルを利用するように修正しました。

## レビューポイント
- タブのフィルタ管理用に`key`プロパティを追加したが、構成として適切か
  - 表示項目の制御には従来通りの定数を利用したいのでkeyプロパティとして設定
  - labelプロパティにタブに表示する文言の翻訳文言を参照するように設定
- 対応ラベルの箇所は適切か（各タブ名＋タイトル名）

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

## 動作確認状況
- ローカルで確認
 2023-11-02 09:41:43 +00:00
湯本 開
18757e3243 Merged PR 549: Usersタブを開けないBugの修正 
## 概要
[Task3019: 修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3019)
- 単体ユーザー取得の際、Selectを用いて必要なパラメータを指定していなかったので必須パラメータが欠けた状態で取得され、それがキャッシュされてしまった
  - ※キャッシュ実装時、Selectなしだと全部取れると勘違いしていた

## レビューポイント
- 修正内容は妥当か？

## 動作確認状況
- ローカルで確認
 2023-11-02 06:00:36 +00:00
makabe.t
da31ae441f Merged PR 546: Revert '調査する' 
## 概要
[Task3013: 調査する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3013)

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

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

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

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

## 補足
- 相談、参考資料などがあれば

Reverts !545
 2023-11-01 09:38:29 +00:00
makabe.t
2d01f3025c Merged PR 545: 調査する 
## 概要
[Task3013: 調査する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3013)

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

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-11-01 09:29:31 +00:00
水本 祐希
c089060162 Merged PR 542: 画面修正（サインアウト本実装） 
## 概要
[Task2963: 画面修正（サインアウト本実装）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2963)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
**headerにサインアウトの実装を追加**
- この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/Task2963?csf=1&web=1&e=XbInfz

## 動作確認状況
- ローカルで確認
管理者ユーザーかつ第1～5階層、一般ユーザーでログインし、headerに表示されかつサインアウトできることを確認。

## 補足
- 相談、参考資料などがあれば
 2023-10-31 10:28:43 +00:00
masaaki
93e86e952b Merged PR 541: StaticWebConfigApp.jsonの定義を追加する 
## 概要
[Task2928: StaticWebConfigApp.jsonの定義を追加する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2928)

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

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

## UIの変更
- 無し

## 動作確認状況
- ローカルで確認済。ただし、ローカルはもともと"X-Frame-Options": "SAMEORIGIN"となっていたのでdev反映後に再度確認実施予定。

## 補足
- 相談、参考資料などがあれば
 2023-10-31 09:11:46 +00:00
maruyama.t
e9541e22a2 Merged PR 523: 画面修正（ヘッダー右上にユーザー名を表示） 
## 概要
[Task2920: 画面修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2920)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
ログイン時に、API（getMyUser）を呼び出し、フラグをtrueにして結果をstateに保存する。
フラグで管理する目的は、ページ遷移時に再度API呼び出しを行わないようにするため。

- このPull Requestでの対象/対象外
サインアウト部分は対象外→別タスクにて対応

- 影響範囲（他の機能にも影響があるか）
既存の各ページのヘッダー部分

## レビューポイント
- component/header配下にSliceを追加すると循環参照になってしまうのを避けるため、feature/login配下にSlice及び各処理を追加した、妥当か。

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

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

## 補足
- 相談、参考資料などがあれば
 2023-10-31 09:02:19 +00:00
makabe.t
976271ab92 Merged PR 543: デプロイパイプラインyamlを修正&リポジトリの修正 
## 概要
[Task2995: デプロイパイプラインyamlを修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2995)

- STGデプロイパイプラインの定義yamlについて以下の環境変数に0から具体的な値を設定しました。
 - REFRESH_TOKEN_LIFETIME_WEB: 86400000
 -  REFRESH_TOKEN_LIFETIME_DEFAULT: 2592000000
 -  ACCESS_TOKEN_LIFETIME_WEB: 7200000
- ユーザーリポジトリの`findSameAccountUsers`についてentityManagerを使っていない箇所を使うように修正しました。

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

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-10-31 06:52:29 +00:00
湯本 開
83add51148 Merged PR 538: Azure AD B2Cの結果をCacheManagerにキャッシュするよう修正 
## 概要
[Task2967: Azure AD B2Cの結果をCacheManagerにキャッシュするよう修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2967)

- キャッシュを扱うRedisServiceを追加
- AdB2cServiceでRedisServiceを使って結果をキャッシュする実装を追加
- ADB2Cの呼び出しと、キャッシュからの取得が行われた時にログを出す実装を追加
  - Azure Monitorで呼び出しコストでアラート出したくなった時のための予防的追加
- 開発環境でローカルのredisを操作する用途のredis-cliをインストールする設定を追加&スクリプトを追加
- `getUser` と `getUsers` の返り値の方を統一 & 使用されなくなった方の型を削除
- AdB2Cの`ttl` に設定する用の値を環境変数に追加
  - 今後実装予定のトークンのキャッシュとはTTLを別にしたかったため
- 複数ユーザー削除処理内でのindex処理が不適切と思われる箇所があったので修正

## レビューポイント
- **Redisへのget/set/delが失敗した際に、エラーログだけ出して成功 or 取得対象なしと同様の動作をするように作成したが、問題なさそうか**
    - これは速度向上用のキャッシュが死んでいても業務は動くべきではないか、という考えによるもの
    - 通信できない=障害中であると想定されるので、失敗しても良いような気もするので相談
- **AdB2cService内でキャッシュを扱う箇所のコードの可読性に問題はないか**
    - 更にWrapしてキャッシュの具体的な動きを隠蔽することも考えたが、詳細なエラーの制御をしづらくなりそうだったので具体的な引数の変換等以上のことはしない形で実装
    - AdB2cServiceが十分に末端の処理なので詳細な処理を生で書いていても認知負荷はそう変わらない可能性がある
- **キャッシュする値の性質によってTTLを変えられる仕組みを前提に設計・実装したが、懸念点はないか**
- **TTLに設定する値は妥当そうか**
- **`Aadb2cUser` を削除したが問題ないか**
- **`deleteUsers` 内のログ処理の変更は適切か**
    - to 岩田さん

## 動作確認状況
- ローカルで確認
- npm run testが通過することを確認
 2023-10-31 03:45:31 +00:00
makabe.t
01d92b2408 Merged PR 537: API実装（代行操作用トークン更新API） 
## 概要
[Task2906: API実装（代行操作用トークン更新API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2906)

- アクセストークン更新APIとテストを実装しました。

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

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-10-31 01:47:00 +00:00
makabe.t
f33af7a9cd Merged PR 539: 動作確認不具合修正 
## 概要
[Task2978: 動作確認不具合修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2978)

- 次タスク取得時のパラメータをNumberで取得できるように修正

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

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-10-30 07:06:31 +00:00
makabe.t
e6da791406 Merged PR 530: API実装（代行操作用トークン生成API） 
## 概要
[Task2905: API実装（代行操作用トークン生成API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2905)

- 代行操作用トークン生成APIとテストを実装しました。

## レビューポイント
- リポジトリの処理は適切か
  - アカウントの取得⇒管理者ユーザ取得としているためUsersリポジトリ配下に配置していますが構成として問題ないでしょうか。
- テストケースは適切か
- アクセストークン生成は既存と別に代行操作用のメソッドを用意していますが想定とあっていますでしょうか。

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-10-30 00:58:46 +00:00
makabe.t
b314fe4b46 Merged PR 513: 次タスク取得API実装 
## 概要
[Task2874: 次タスク取得API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2874)

- 次タスク取得APIとテストを実装しました。

## レビューポイント
- リポジトリからのタスク取得ロジックは適切か
- テストケースは適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-10-26 10:46:15 +00:00
masaaki
f3dde1874d Merged PR 512: ADB2Cユーザが一括で削除できないことに対する対応優先度の数値根拠だし 
## 概要
[Task2831: ADB2Cユーザが一括で削除できないことに対する対応優先度の数値根拠だし](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2831)

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

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-10-26 09:19:03 +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
湯本 開
9c9404367b Merged PR 535: STGビルドの環境変数が間違ったファイルを読まないよう修正する(2) 
## 概要
[Task2956: STGビルドの環境変数が間違ったファイルを読まないよう修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2956)

- ビルドコマンドでの引数の渡し方に問題があった部分を修正

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

## 動作確認状況
- ローカルで確認
 2023-10-26 08:46:30 +00:00
湯本 開
2a867f2a71 Merged PR 533: STGビルドの環境変数が間違ったファイルを読まないよう修正する 
## 概要
[Task2956: STGビルドの環境変数が間違ったファイルを読まないよう修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2956)

- .envを上書きする方法から、VITEが本来想定しているビルド指定方法へ修正
- STGパイプラインでも `npm run build:stg` を使用するよう修正

## レビューポイント
- 修正方針に問題はないか

## 動作確認状況
- ローカルで動作する所まで確認
 2023-10-26 08:20:32 +00:00
saito.k
375e3a5a3b Merged PR 524: 画面実装（代行操作中表示コンポーネント＋代行操作中に表示する画面のデザイン） 
## 概要
[Task2908: 画面実装（代行操作中表示コンポーネント＋代行操作中に表示する画面のデザイン）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2908)

- 代行操作中に表示する画面に対して、代行操作中のデザインを反映
- 代行操作中に画面上部に表示するバーのコンポーネントを作成
- 代行操作用のトークンをauthStateに追加

## レビューポイント
- 代行操作バーのコンポーネントに表示する会社名をpropsとしたが良いか
  - このコンポーネントを表示する画面で会社名を取得して、会社名とトークンがあれば表示するという風にしたいから
- 代行操作用トークンはauthStateでよいと思っているが認識会うか

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

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

## 補足
- 相談、参考資料などがあれば
 2023-10-26 06:35:59 +00:00
湯本 開
8474c6a4f5 Merged PR 529: Pipelineの仕組みを修正する 
## 概要
[Task2949: Pipelineの仕組みを修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2949)

- .env.(環境名) の環境変数を用いてフロント側ビルドを行うよう修正
- ビルドパイプラインでstaging/production用の両方のフロント側ビルドを行うよう変更

## レビューポイント
- 仕組み上の不明点はないか
- 情報共有

## 動作確認状況
- 本番環境へデプロイされ、本当の最低限だけ動作することを確認
 2023-10-26 04:37:58 +00:00