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

573 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
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
maruyama.t
afa05f381c Merged PR 628: 第５階層で規約同意画面を表示する際に、DPAに同意が一瞬表示される 
## 概要
[Task3281: 【Commit諸作業終わったら優先対応】第５階層で規約同意画面を表示する際に、DPAに同意が一瞬表示される](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3281)

selectorから別途tierの値が入っているかをselectIsTierLoadingで取得し、まだ入っていない場合(tier===0)Loadingの表示を行うようにした。

```
export const selectIsTierLoading = (state: RootState) =>
  state.terms.domain.tier === 0;
```

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-12-18 04:07:03 +00:00
saito.k
9380d9bfc6 Merged PR 629: STGのパイプラインに環境変数を追加する 
## 概要
[Task3326: STG、PRODのパイプラインに環境変数を追加する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3326)

- StagingのPipelineに環境変数を追加

## レビューポイント

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

## 動作確認状況

## 補足
- 相談、参考資料などがあれば
 2023-12-18 00:58:50 +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
makabe.t
63892bad83 Merged PR 620: テンプレートファイルダウンロードをTypistのみが実行可能にする 
## 概要
[Task3291: テンプレートファイルダウンロードをTypistのみが実行可能にする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3291)

- テンプレートファイルダウンロード先要求APIを実行できるユーザーをTypistのみに修正しました。
  - Authorが実行できないようにしました。

## レビューポイント
- ガードでTypistのみにしたので内部のロールでの分岐処理を削除しましたが問題ないでしょうか？

## UIの変更
- なし

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

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

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

delete処理にSQLコメントを挿入する形にリファクタ
レビューポイント
repository/* に存在するdelete文は全て置き換えたはずだが、漏れはなさそうか
 2023-12-12 09:28:40 +00:00
saito.k
5ef222134e IPアドレス追加に伴う修正漏れ対応 2023-12-12 15:46:37 +09:00
makabe.t
bce9866ba3 Merged PR 612: /files/template/download-locationの対応 
## 概要
[Task3278: /files/template/download-locationの対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3278)

- クエリパラメータが不正な場合にバリデータで処理されるようにしました。

## レビューポイント
- 適用したバリデータは適切でしょうか？

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-12-12 05:56:05 +00: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
水本 祐希
8fe649cb7f Merged PR 618: 画面修正（第５のライセンス一覧の項目順を変更） 
## 概要
[Task3271: 画面修正（第５のライセンス一覧の項目順を変更）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3271)

- 何をどう変更したか、追加したライブラリなど
  - 項目の配置を変更
    - License Inventory→Number of unused licenses→Number of licenses available for reuse→Number of licenses allocatedの順で修正
    - Total number of licenses on orderをlicense情報の一番下に配置
  - 不要な項目を削除
    - Total number of orderを削除
  - レイアウトを最新のものに適用

- このPull Requestでの対象/対象外
項目名の変更はOMDS様が翻訳したものを適用すればよいので、対象外

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

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

## 動作確認状況
- ローカルで確認
 2023-12-12 02:08:18 +00:00
湯本 開
375e8e87a8 Merged PR 621: STG/PRODでSourceMap出力をOFFにしておく 
## 概要
[Task3165: STG/PRODでSourceMap出力をOFFにしておく](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3165)

- STG/PRODでsourcemap（ビルド後の圧縮・結合されたjsからデバッグ用の実行行やコメントを復元するための存在）を出力しないように修正
  - コメント等がユーザーに見えないようにする目的

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

## 動作確認状況
- ローカルで確認
 2023-12-11 06:35:59 +00:00
maruyama.t
f7ec740e7a Merged PR 615: PrivacyNoticeの多言語対応の文言修正 
## 概要
[Task3292: PrivacyNoticeの多言語対応の文言修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3292)

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

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-12-08 05:59:30 +00:00
湯本 開
b1f169def5 Merged PR 594: [TODO対処] メールの内容について編集しやすくする 
## 概要
[Task2163: [TODO対処] メールの内容について編集しやすくする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2163)

- メール文面をハードコードしない構造を試作

## レビューポイント
- メールテンプレートの定義場所、定義形式、読み込み方法などは問題なさそうか

## 動作確認状況
- ローカルで確認
 2023-12-08 05:06:02 +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
masaaki
342baa9826 Merged PR 610: AzureFucntionのpipelineが参照しているDockerFileを修正する 
## 概要
[Task3274: AzureFucntionのpipelineが参照しているDockerFileを修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3274)

- staging用のpipelineに対して、functionのビルド時に参照するdockerFileをappserviceと同様の階層にあるものに修正しました

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

## UIの変更
- 無し

## 動作確認状況
- 未確認。stagingのpipeline動作時に確認します。

## 補足
- 相談、参考資料などがあれば
 2023-12-07 06:09:48 +00:00
saito.k
3ad0d87814 確認用のログを削除 2023-12-06 20:10:00 +09:00
saito.k
9f8e20230b 小文字に修正 2023-12-06 19:43:25 +09:00
saito.k
2d0697f56b リクエストのIPをログに出力して確認する 2023-12-06 19:06:26 +09:00
saito.k
d92cb1d28b クエリログに追跡用のIDをコメントとして追加 2023-12-06 17:15:32 +09:00
水本 祐希
c246586498 Merged PR 602: ライセンス確認画面（第五階層）に会社名を表示する（client側） 
## 概要
[Task3225: ライセンス確認画面（第五階層）に会社名を表示する（client側）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3225)

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

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

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

## 動作確認状況
- ローカルで確認（完了）
 2023-12-06 02:10:01 +00:00
oura.a
4399a61f2b Merged PR 600: [ライセンスアラート改善]リトライ対応 
## 概要
[Task3025: [ライセンスアラート改善]リトライ対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3025)

ライセンスアラート処理にリトライ処理を追加しました。
メールの多重送信を防ぐために、送信成功したメールについてはredisに保存し、送信時にキャッシュをチェックする処理を入れました。

## レビューポイント
処理の流れが妥当か。
redisに保存するキー、値は適切か。
if文のネストが相当深くなってしまったが、改善できるポイントはあるか。

## UIの変更
なし

## 動作確認状況
ローカルで動作確認済み。（テスト用コードで無理やりエラーを発生させての確認）

## 補足
なし
 2023-12-06 00:46:53 +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
makabe.t
62a5155ee1 Merged PR 608: develop動作確認不具合対応 
## 概要
[Task3238: develop動作確認不具合対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3238)

- タスク一覧画面にあるアプリケーションインストールボタンのラベルを翻訳対応しました。

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

## UIの変更
- なし
## 動作確認状況
- ローカルで確認
 2023-12-05 08:10:58 +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
masaaki
f82f202b63 Merged PR 603: [ライセンスアラート]FunctionAPPのprod環境適用（pipeline含む） 
## 概要
[Task2870: [ライセンスアラート]FunctionAPPのprod環境適用（pipeline含む）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2870)

- 本番環境用のpipelineに対して、関数アプリのイメージをデプロイするjobを追加しました

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

## UIの変更
- 無し

## 動作確認状況
- 未実施。本件マージ後にpipeline動作させる予定

## 補足
- デプロイに関する記載についてはstaging環境のpipelineで動作実績があります（job：function_deploy）
 2023-12-05 06:12:20 +00:00
maruyama.t
ebfc05040b Merged PR 604: 画面実装（ファイル情報ポップアップ画面実装） 
## 概要
[Task3195: 画面実装（ファイル情報ポップアップ画面実装）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3195)

- 影響範囲（他の機能にも影響があるか）
画面実装のみのため、なし

## レビューポイント
- 表示される値はDictationタブの表示のされ方と同じだが問題ないか
（拡張子やサイズなど）
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
https://ndstokyo.sharepoint.com/sites/Piranha/Shared%20Documents/Forms/AllItems.aspx?csf=1&web=1&e=hzPw9b&cid=f9d17150%2D1d33%2D49da%2D8bd4%2Df5a27cbbed66&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%2FTask3195&viewid=786a81cf%2Dd15f%2D4dc2%2D9e55%2Dc7a729fbc72f

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

## 補足
- 相談、参考資料などがあれば
 2023-12-05 05:04:25 +00:00
maruyama.t
40da605780 Merged PR 605: DBマイグレーション(PrivacyNotice) 
## 概要
[Task3213: DBマイグレーション](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3213)

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-12-05 04:22:58 +00:00
masaaki
cca9cf6e5e Merged PR 601: ヘルプページ画面実装 
## 概要
[Task3204: ヘッダ修正対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3204)

- ヘルプページを追加して、ヘッダのタブに表示されるようにしました。
  - ヘッダのタブはロールの制限なく見えるようにしています。
  - ヘルプページからさらに個別のヘルプへのリンクは仮のURLにしています。
    - 動作としては別タブで開くようにしています。

## レビューポイント
- パスをタブ名に合わせて`/support`としましたが不自然な点はないでしょうか？
- 仮のURLがxxxとなっているため、別タブで開かれたページがNot Foundとなってしまっていますが問題ないでしょうか？

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

## 動作確認状況
- ローカルで確認
 2023-12-05 02:40:09 +00:00
makabe.t
6db8b4cbd7 Merged PR 599: ライセンスアラート関連：DockerfileFunctionDictation.dockerfileを参照するようファイルとpipelineを修正する 
## 概要
[Task3083: ライセンスアラート関連：DockerfileFunctionDictation.dockerfileを参照するようファイルとpipelineを修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3083)

- Azure Functionsのイメージを`DockerfileFunctionDictation.dockerfile`で定義するように修正しました。
  - マルチステージビルドの形式で必要なファイルだけ取り込んだイメージとなるようにしています。
  - 既定のパス`/home/site/wwwroot`をワークディレクトリとしています。

## レビューポイント
- 生成物イメージに入れるファイルに過不足はないでしょうか？
- 環境変数でのパス指定ができないため、既定のパスで動作するようにしていますが不自然な点はないでしょうか？

## UIの変更
- なし

## 動作確認状況
- develop環境でデプロイして確認
 2023-12-04 08:18:18 +00:00
masaaki
3dd3aeb274 Merged PR 598: ライセンスアラート関連pipelineの最適化 
## 概要
[Task2992: ライセンスアラート関連pipelineの最適化](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2992)

- staging用のpipelineに対してAzureFunction用のjobを追加しました。
  - build用のjobとdeploy用のjobの二つです
- このPull Requestでの対象/対象外
  - 本番環境用のpipelineについては、Azureリソース作成後に実施する必要があるため、
    「タスク 2870: [ライセンスアラート]FunctionAPPのprod環境適用（pipeline含む）」にて実施します。

## レビューポイント
- Appserviceのビルド（backend_build）と比較して、名前の重複やpush先の競合はないか
- jobの追加箇所は適切か

## UIの変更
- 特になし

## 動作確認状況
- 未実施
  マージ後、宣言したのち動作確認を行う予定です。

## 補足
- Functionのdeployで使用したtaskは以下になります。
https://learn.microsoft.com/ja-jp/azure/devops/pipelines/tasks/reference/azure-function-app-container-v1?view=azure-pipelines
 2023-12-01 08:04:31 +00:00
masaaki
fee99a0974 Merged PR 583: [ライセンスアラート改善]AzureAdB2Cアクセスの効率化 
## 概要
[Task3023: [ライセンスアラート改善]AzureAdB2Cアクセスの効率化](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3023)

ADB2Cからユーザーを取得する際に、Redisによるキャッシュ保存・キャッシュからの取得を行う処理を実装しました。

## レビューポイント
処理の妥当性などを全体的にお願いします。

## UIの変更
なし

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

## 補足
なし
 2023-12-01 01:39:18 +00:00
水本 祐希
15fa10e265 Merged PR 595: 各service.tsのoutログ追加 
## 概要
[Task3162: licenses.service.tsのoutログ追加](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3162)

- 何をどう変更したか、追加したライブラリなど
・licenses.service.tsのolicenseOrdersとissueCardLicenseKeysの[OUT]ログを追加
・横展開として他のservice.tsでOUT、INログが不足している場合は追加する

## 動作確認状況
- ユニットテスト
 2023-11-30 04:17:16 +00:00
iwata
ef376d3027 AzireFunctionのpipeline用に作ったが名前が分かりづらいので削除 2023-11-30 09:47:48 +09:00
masaaki
9cbb07efa3 Update azure-pipelines.yml for Azure Pipelines 2023-11-30 00:29:36 +00:00
masaaki
837d6af4cd Set up CI with Azure Pipelines 
[skip ci]
 2023-11-30 00:21:12 +00:00
saito.k
a0ab885526 Azureリソースの構成変更のためテンプレートを更新 2023-11-29 20:04:19 +09:00
saito.k
cbb412218e Azureリソース作成したためテンプレートを更新 2023-11-29 19:28:46 +09:00
水本 祐希
cc36b806c5 Merged PR 596: 第一階層のshortage(-)が赤字となる 
## 概要
[Task3169: 第一階層のshortage(-)が赤字となる](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3169)

- 何をどう変更したか、追加したライブラリなど
 第一階層のshotage"-"が赤文字にならないよう修正

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

## 補足
- 相談、参考資料などがあれば
 2023-11-29 06:06:04 +00:00
水本 祐希
c3750e3c1c Merged PR 591: 修正対応 
## 概要
[Task3158: 修正対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3158)

- 何をどう変更したか、追加したライブラリなど
shotageの値が１以上の場合、赤文字になる

## 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/Task3158?csf=1&web=1&e=3qaLfD

## 動作確認状況
- ローカルで確認
 2023-11-28 06:35:56 +00:00
makabe.t
8b04adf095 Merged PR 587: 画面実装（ダウンロード処理） 
## 概要
[Task3122: 画面実装（ダウンロード処理）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3122)

- ファイルバックアップポップアップからのダウンロード実行処理を実装しました。
  - ファイルURL取得をAdminが適切に実施できるように権限を修正しました。
  - SASトークンの開始時刻を修正しました。
  - ダウンロード処理を実行できるようにするためにblobストレージのCORS設定を修正しました。
- faviconとタイトルを正式なものに差し替えました。

## レビューポイント
- faviconはアセットフォルダに一緒に入れてしまっていますが配置場所として問題ないでしょうか？
- `Operation`内でループで一件ずつダウンロード・バックアップ処理を実行していますが認識あっていますでしょうか？
- SASトークンの生成時の開始時刻10秒前にしていますが問題ないでしょうか？
    - SASトークン付きURLの発行直後にURLでダウンロード実行するとSASトークンエラーとなることがありましたので、その対応です。
      - 発行直後に使うはずなのでSASトークンの開始を10秒前にしても影響はないはずと考えています。
      - ほかの対応としてトークン取得後に数秒待つことも考えましたが、動作全体が遅くなってしまうのとSASトークンの作りの問題だとがんが得ているのでこのような対応をしています。

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

## 動作確認状況
- ローカルで確認
 2023-11-28 06:21:23 +00:00
maruyama.t
aab5bf2af6 Merged PR 592: [システムテスト入ってから対応でOK] ライセンス発行時の有効期限の計算を変更する 
## 概要
[Task2490: [システムテスト入ってから対応でOK] ライセンス発行時の有効期限の計算を変更する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2490)
ライセンスの有効期限にタイムゾーンを加味した時刻８時間を追加した
- 影響範囲（他の機能にも影響があるか）
既存のユニットテスト

## レビューポイント
ライセンスの有効期限がセットされるシチュエーションに対する修正の漏れがないか。

## UIの変更
なし

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

## 補足
- 相談、参考資料などがあれば
 2023-11-28 04:01:09 +00:00
masaaki
501fc5a15d Merged PR 588: 画面実装（タスク一覧画面：タスクキャンセル操作追加） 
## 概要
[Task3131: 画面実装（タスク一覧画面：タスクキャンセル操作追加）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3131)

- タスク 3131: 画面実装（タスク一覧画面：タスクキャンセル操作追加）
- タスク一覧に対してCancel Dictationボタンを実装しました

## レビューポイント
- タスク関連のほかの処理で実施しているものについて、キャンセルで実施必要か確認いただきたいです。
  - キャンセル後のデスクトップアプリ起動
    - 不要の認識
  - キャンセル前のソート条件保存
    - ステータスが変わるため、必要な認識
- ログイン名を取得するためにloginのselectorを使用していますが問題ないか確認いただきたいです
（他フィーチャと余計な関連を作るのはあまりよくないと思いつつ、同じような取得処理を作るのも嫌だと思いつつ。。）

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

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

## 補足
- 相談、参考資料などがあれば
 2023-11-22 01:13:03 +00:00
湯本 開
f38733f9b2 Merged PR 589: ライセンスチェック処理をlicensesまたはusers配下に移動して共通部品化する 
## 概要
[Task3084: ライセンスチェック処理をlicensesまたはusers配下に移動して共通部品化する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3084)

- "チェック"する関数でなく、ライセンスの割当状態を取得する関数に変更
- 利用側では、割当状態を元にして必要なら例外をthrowする実装に変更
- 関数の機能を `LicensesRepository` が提供する形に変更

## レビューポイント
- 関数のシグネチャは問題なさそうか

## 動作確認状況
- テストは通過
 2023-11-21 10:48:57 +00:00
makabe.t
68aaa0e548 Merged PR 586: バックアップAPI実装 
## 概要
[Task3121: バックアップAPI実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3121)

- タスクのバックアップAPIとテストを実装しました。
  - 実装に当たり、タスクエンティティ定義の型を修正しています。

## レビューポイント
- テストケースに過不足はないでしょうか？
- エンティティの値を取得した際に`is_job_number_enabled`が数値となってしまうのでtypeをBooleanに変更しましたが問題ないでしょうか？

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-11-20 08:57:56 +00:00