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

603 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
masaaki
7f8e2d5411 Merged PR 667: ロックが効かなくなる 
## 概要
[Task3420: ロックが効かなくなる](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3420)

- 「タスク 3400: テスト修正」対応の影響で正しくlockが行われなくなっていた事象を修正

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

## UIの変更
- 無し

## 動作確認状況
- ユニットテスト実施、ローカル動作確認時、sqlにfor updateが付与されることを確認

## 補足
- 相談、参考資料などがあれば
 2023-12-27 10:23:54 +00:00
oura.a
4a73d5f8c4 Merged PR 665: マイグレーションファイル修正 
## 概要
[Task3416: マイグレーションファイル修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3416)

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

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

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

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

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

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

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-12-27 02:24:39 +00:00
oura.a
9852004a36 Merged PR 636: ライセンス発行処理が遅い問題の解決およびトランザクションが効いてなければ効くよう修正する 
## 概要
[Task3243: ライセンス発行処理が遅い問題の解決およびトランザクションが効いてなければ効くよう修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3243)

ライセンス発行処理が重複して実行できてしまう不具合を修正しました。
■修正内容
・ライセンス注文テーブルの情報を取得する際に、行ロックを取得するよう修正
・ライセンス注文テーブルに、「注文元アカウントID、POナンバー」の２カラムを対象とするインデックスを作成
・以前に外部キー制約をつけた際、自動で作成されていたインデックスを削除

■ロックについて
共有ロックと排他ロックがある
・共有ロック：共有ロック取得中でも、他のトランザクションが共有ロックを取得できる
　　　　　　　排他ロックは取得できない
・排他ロック：排他ロック取得中は、他のトランザクションは共有ロック・排他ロック共に取得できない
今回の修正では、デフォルト設定で共有ロックを取得していた箇所を、明示的に排他ロックを取得するようにした。

■行ロックについて
・インデックス行に対してロックをかけている
　→インデックスが作成されていない、検索条件にヒットしないなどでうまく動かない
　　例）インデックスが作成されていないと、テーブル全体のロックとなってしまう
・上記の都合で検索条件が範囲指定のものにロックをかける際は注意が必要。（今回は一意指定なので問題なし）

■SQLiteを使ったユニットテストが`pessimistic_write`に対応していない件について
`process.env.NODE_ENV`の値を参照（テスト実行中は`test`、ビルドした環境で動かすと`undifind`）し、
テスト実行の場合`pessimistic_write`を付与しないようクエリを修正した。

## レビューポイント
インデックスについて懸念点があるか？

## UIの変更
なし

## 動作確認状況
ローカルで以下を確認
■発行処理について
・同じ注文に対し複数タブで発行処理を実行し、後発の処理が「ライセンス発行済みエラー」となることを確認
・同一アカウントからの異なるPOナンバーの注文を同時に発行し、行ロックによる待ちが発生せず並列に処理されることを確認
・別アカウントからの同一POナンバーの注文を同時に発行し、行ロックによる待ちが発生せず並列に処理されることを確認
■インデックスについて
同一アカウントからの異なるPOナンバーの注文を同時に発行
・インデックスを作成している状態で、行ロックによる待ちが発生せず並列に処理されることを確認
・インデックスを削除した状態で、行ロックによる待ちが発生することを確認
・migrate:up/downが正しく動作することを確認

## 補足
以前のアカウント削除PBIで一時的に設定した外部キー制約の作成時に、自動でインデックスも作成されていたようです。
必要ないインデックスはどこかで削除する必要があるかと思っています。
 2023-12-27 02:01:24 +00:00
makabe.t
6f62a016d4 Merged PR 663: develop不具合対応 
## 概要
[Task3404: develop不具合対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3404)

- ライセンス割り当て通知メールのTOにAdminが入るように修正しました。
- 不要なメソッド`createMailContentFromEmailConfirm`を削除しました。

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

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-12-26 07:24:10 +00:00
湯本 開
f2ef2228c3 Merged PR 661: APIバリデータの漏れを修正 
## 概要
[Task3402: APIバリデータの漏れを修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3402)

- Optinal指定が必要なパラメータにOptionalが抜けていた箇所を修正
  - 以前はバリデータ自体が無かったので、IsOptionalが抜けていても偶然動いていた
- 他requestにも同様の漏れがないか `?:` で検索したが、この1件のみだと思われる

## レビューポイント
- 他に同様の記載はなさそうか
- `?:` による省略可能表記以外の記法で省略パラメータは定義されていない認識だが、合っているか

## 動作確認状況
- npm run testは通過
 2023-12-25 07:34:49 +00:00
makabe.t
f6c3f69801 Merged PR 657: ユーザー認証完了通知の実装 [U-113] 
## 概要
[Task3322: ユーザー認証完了通知の実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3322)

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

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

## UIの変更
- なし

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

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

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

## UIの変更
- なし
## 動作確認状況
- ローカルで確認
 2023-12-25 05:38:28 +00:00
湯本 開
95c058265e Merged PR 655: 外部連携以外のAPIバリデータ見直し(/accounts/*以外) 
## 概要
[Task3285: 外部連携以外のAPIバリデータ見直し](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3285)

- バリデータが不足していた部分について、バリデータを追加

## レビューポイント
- 現行の動きが変更されるようなバリデータが設定されていないか
- `AUTHORの時のみ省略不可能` 等のバリデータが設定されているプロパティに関しては、`hogehoge` や `"or ‘1’=’1’"` が設定されたリクエストが来てもバリデータで弾けないが許容可能そうか

## 動作確認状況
- npm run build / npm run test 成功
 2023-12-25 05:22:01 +00:00
湯本 開
6deaa37df7 Merged PR 652: 外部連携以外のAPIバリデータ見直し(/accounts/*) 
## 概要
[Task3285: 外部連携以外のAPIバリデータ見直し](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3285)

- accounts/* 配下のRequest型に不足しているバリデータを追加

## レビュー対象外
- 他features配下のRequest型のバリデータ
  - 1個のレビューで出すとレビュー負荷が高すぎそうであったため、レビューを分割します

## レビューポイント
- 追加したバリデーションが適切か（主な対応は以下）
  - 数値に型変換していなかったものに型変換を追加
  - IsInt()を追加
  - マイナス値がセットされるとDBリクエストでコケる類のものに制限を追加
  - 文字列はDBやAzure AD B2Cで受付不能な文字数を弾く

## 動作確認状況
- ビルドとテストが成功することは確認
 2023-12-25 05:21:35 +00:00
makabe.t
fce3214650 Merged PR 659: ライセンスアラートのメール文言を最新に修正 
## 概要
[Task3318: ライセンスアラートのメール文言が最新の仕様か確認](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3318)

- ライセンスアラート周りのメール文言を修正しました。
  - スペイン語は不要なので文言から削除しました。

## レビューポイント
- メッセージの修正内容は適切でしょうか？

## UIの変更
- なし
## 動作確認状況
- ローカルで確認
 2023-12-25 04:56:22 +00:00
maruyama.t
9d8c736d92 Merged PR 648: ライセンス自動割り当て処理実装（リトライ対応） 
## 概要
[Task3296: ライセンス自動割り当て処理実装（リトライ対応）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3296)

- licenseAutoAllocationProcessingに任意引数で日付を追加。
　日付がある場合はその日付を実行日としてライセンス自動割り当てを行う。

## レビューポイント
- 未来日日付が指定された場合エラーにしなくてよいか？
（運用で使う想定はない）

## 動作確認状況
- ユニットテストで確認、devlopで確認
・引数なしで手動実行した場合に、実行日でライセンス自動割り当てが処理されること。
・引数ありで手動実行した場合に、引数の日付でライセンス自動割り当てが処理されること。
・（未来日で）引数ありで手動実行した場合に、引数の日付でライセンス自動割り当てが処理されること。
（あったらうれしいかもしれないのでリトライ処理の機能として可能な状態にしておいています。）

## 補足
- 実際にサポートの担当が行う作業は以下になります。
①AzureFunctionのlicenseAutoAllocationManualRetryにアクセスする。
![image (3).png](https://dev.azure.com/ODMSCloud/6023ff7b-d41c-4fa7-9c6f-f576ba48c07c/_apis/git/repositories/302da463-a2d7-40f9-b2bb-6e8edf324fa9/pullRequests/648/attachments/image%20%283%29.png)
②左カラムの「コードとテスト」を押下し、「テストと実行」を押下する。
![image (4).png](https://dev.azure.com/ODMSCloud/6023ff7b-d41c-4fa7-9c6f-f576ba48c07c/_apis/git/repositories/302da463-a2d7-40f9-b2bb-6e8edf324fa9/pullRequests/648/attachments/image%20%284%29.png)
③入力欄の、「クエリ」にdateと日付をハイフン区切りで入力して、「実行」を押下する。
![image (2).png](https://dev.azure.com/ODMSCloud/6023ff7b-d41c-4fa7-9c6f-f576ba48c07c/_apis/git/repositories/302da463-a2d7-40f9-b2bb-6e8edf324fa9/pullRequests/648/attachments/image%20%282%29.png)
 2023-12-25 04:33:12 +00:00
水本 祐希
1c18bf03b6 Merged PR 640: 壊れたライセンステストをDBテストで書き直す 
## 概要
[Task3356: 壊れたライセンステストをDBテストで書き直す](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3356)

- 何をどう変更したか、追加したライブラリなど
ライセンスのモック使ったたUTをDBテストになるよう修正
- ライセンス注文が完了する
  - ライセンス注文が完了する
  - ユーザID取得できなかった場合、エラーとなる
  - 親ユーザID取得できなかった場合、エラーとなる
  - POナンバー重複時、エラーとなる

- カードライセンス発行が完了する
  - カードライセンス発行に失敗した場合、エラーになる

- カードライセンス取り込みが完了する
  - カードライセンス取り込みに失敗した場合、エラーになる（DBエラー)
  - カードライセンス取り込みに失敗した場合、エラーになる（ライセンスが存在しないエラー）
  - カードライセンス取り込みに失敗した場合、エラーになる（ライセンスが既に取り込まれているエラー）

※カードライセンス発行、カードライセンス取り込みが完了する、取込可能なライセンスのみが取得できる
以上の３点はすでにDBテストになっていたので、特に修正・追加していません。

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

## 動作確認状況
- ユニットテスト
 2023-12-25 00:17:17 +00:00
makabe.t
f455cd6262 Merged PR 653: ユーザー情報変更完了通知 [U-115] の実装 
## 概要
[Task3311: ユーザー情報変更完了通知 [U-115] の実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3311)

- ユーザー情報変更時のメール通知を実装しました。

## レビューポイント
- メールの送信先に不自然な点はないでしょうか？

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-12-22 08:39:11 +00:00
makabe.t
2f2e401ae5 Merged PR 647: ユーザー認証完了のお願い [U-114] の実装 
## 概要
[Task3310: ユーザー認証完了のお願い [U-114] の実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3310)

- ユーザー追加後の認証メール送信について正規のメール文面を出すようにしました。

## レビューポイント
- メールに渡す情報の取得内容で不自然な点はないでしょうか？
- メール送信処理に失敗した場合には仮登録したユーザーを削除する処理をそのままにしていますが問題ないでしょうか？

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-12-22 08:23:01 +00:00
水本 祐希
f4da949bfd Merged PR 656: Revert 'タスク一覧のパフォーマンス調査' 
## 概要
[Task3346: タスク一覧のパフォーマンス調査](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3346)

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

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

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

Reverts !651
 2023-12-22 07:09:18 +00:00
水本 祐希
01dda79d72 Merged PR 651: タスク一覧のパフォーマンス調査 
## 概要
[Task3346: タスク一覧のパフォーマンス調査](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3346)

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

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

## 動作確認状況
- ローカルで確認
 2023-12-22 06:12:13 +00:00
makabe.t
3fc184b4af Merged PR 646: アカウント情報変更完了通知 [U-112] の実装 
## 概要
[Task3308: アカウント情報変更完了通知 [U-112] の実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3308)

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

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

## UIの変更
- なし
## 動作確認状況
- ローカルで確認
 2023-12-22 00:18:11 +00:00
saito.k
16f743c4c4 Merged PR 650: 最新の翻訳情報をWebアプリに適用する 
## 概要
[Task3357: 最新の翻訳情報をWebアプリに適用する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3357)

- 最新の翻訳情報を反映

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

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

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

## 補足
- 以下の翻訳情報はまだもらえていないので、残っている
```
タスク一覧
dictationPage.message.backupFailedError
dictationPage.message.cancelFailedError
dictationPage.label.fileBackup
dictationPage.label.downloadForBackup
dictationPage.label.applications

共通
common.label.headerSupport

ヘルプページ全般

ライセンス情報
LicenseSummaryPage.label.licenseLabel
LicenseSummaryPage.label.storageLabel
 2023-12-21 08:52:45 +00:00
Kentaro Fukunaga
9baae2d2dc Merged PR 644: Dictation Workflow完了通知 [U-117] の実装 
## 概要
[Task3313: Dictation Workflow完了通知 [U-117] の実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3313)

- 文字起こし完了時にメール送信する機能を実装しました。
- npm run formatで変更あった箇所も入っています。

## レビューポイント
- SendGridServiceのIFを「こうしたほうがいいかも」とかあれば。
- メール送信に必要な内容取得で効率的にできそうな部分ないか？

## UIの変更
- なし

## 動作確認状況
- ローカルでnpm run testが通ることを確認
- ローカルでメール送信されることを確認
 2023-12-21 06:49:30 +00:00
makabe.t
a6f56d71ee Merged PR 645: アカウント情報消去完了通知 [U-111] の実装 
## 概要
[Task3307: アカウント情報消去完了通知 [U-111] の実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3307)

-

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

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

## 動作確認状況
- ローカルで確認
 2023-12-20 08:50:16 +00:00
湯本 開
1bc6618a6d Merged PR 643: ライセンス引き戻し完了通知 [U-109] の実装 
## 概要
[Task3306: ライセンス引き戻し完了通知 [U-109] の実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3306)

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

## レビューポイント
- テンプレートの適用内容に不自然な点はないか
- アカウントのFromとToとCCに関わる部分で認識違いはないか
  - `orderedAccountId` という引数には注文したアカウント=下位階層のアカウントが入るという理解であっているか等

## UIの変更
- なし
## 動作確認状況
- npm run testは通過
 2023-12-20 07:54:47 +00:00
maruyama.t
60bb8f9e20 Merged PR 649: Revert "手動実行できるように変更" 
Revert "手動実行できるように変更"

Reverted commit `57fc7a17`.
 2023-12-20 06:17:32 +00:00
maruyama.t
57fc7a17b5 手動実行できるように変更 2023-12-20 14:23:28 +09:00
oura.a
32a452bdb2 Merged PR 631: ライセンス自動割り当て処理実装（メイン処理） 
## 概要
[Task3294: ライセンス自動割り当て処理実装（メイン処理）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3294)

ライセンス自動割り当て処理を実装しました。

ラフスケッチでは１回のクエリでアカウント・ユーザーを両方取得する設計でしたが、実装難度・可読性の面から、
アカウントとユーザーを別々に取得するよう変更しています。

## レビューポイント
処理内容に過不足がないか。
DBからのデータ取得時の条件に過不足がないか。

## UIの変更
なし

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

## 補足
なし
 2023-12-20 01:24:31 +00:00
makabe.t
9e1bc8944f Merged PR 641: ライセンス割当完了通知 [U-108] の実装 
## 概要
[Task3305: ライセンス割当完了通知 [U-108] の実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3305)

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

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

## UIの変更
- なし
## 動作確認状況
- ローカルで確認
 2023-12-19 09:57:07 +00:00
masaaki
48ff009d39 Merged PR 630: ファイルプロパティをcloseするとたすく一覧が更新される 
## 概要
[Task3333: ファイルプロパティをcloseするとたすく一覧が更新される](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3333)

- File Property画面にて「Close」ボタンを押下した際に画面がリロードされる問題について対応しました。
「Close」ボタン押下時に「href=""」としていたため、デフォルトのURL遷移（/dictations）が行われていたのを、
onClickによるポップアップ閉じる処理を呼び出す実装に変更しました。

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

## UIの変更
- 変更なし

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

## 補足
- 相談、参考資料などがあれば
 2023-12-19 05:13:32 +00:00
makabe.t
b2fef69ea9 Merged PR 635: アカウント登録完了通知 [U-101] の実装 
## 概要
[Task3301: アカウント登録完了通知 [U-101] の実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3301)

- アカウント登録完了（認証完了）後にメール送信をする機能を追加しました。
  - 合わせてテスト修正をしています。

## レビューポイント
- テンプレートの適用は適切でしょうか。
- テスト修正で対象Sendgridメソッドを上書きしていますが対応として不自然な点はないでしょうか？

## UIの変更
- なし

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

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

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

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-12-19 01:25:57 +00:00
makabe.t
4d325d1751 Merged PR 638: ライセンス発行依頼受付通知 [U-105] の実装 
## 概要
[Task3302: ライセンス発行依頼受付通知 [U-105] の実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3302)

- ライセンス注文時のメール通知処理を既存処理に追加しました。

## レビューポイント
- テンプレートの適用に問題はないでしょうか？
- メールに渡す値の取得処理は認識通りでしょうか？

## UIの変更
- なし

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

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

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

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

## 動作確認状況
- ローカルで確認
 2023-12-18 04:54:53 +00:00
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