金村 勇祐
2ded5b8498
Merged PR 1058: File Lengthのソートが正しく行われない_Funtion側修正漏れの修正
...
Funtion側もデータ型の変更に合わせる修正をいたしました。
動作確認状況
```
Test Suites: 8 passed, 8 total
Tests: 46 passed, 46 total
Snapshots: 0 total
Time: 85.851 s
Ran all test suites.
```
2025-04-23 02:57:38 +00:00
makabe
23f8b54011
Merge branch 'main' into develop
2024-05-07 12:04:03 +09:00
saito.k
af0ba78ae9
Merged PR 883: Functions修正
...
## 概要
[Task4132: Functions修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/4132 )
- 自動割り当てを行うライセンスの取得条件を変更
- 有効期限が近いライセンスまたは有効期限が設定されていないライセンス（新規ライセンス）を取得する
- 有効期限が近いものから割り当てを行うので、ソートはサーバー側で行うようにした
- メール送信処理を追加
## レビューポイント
- テンプレート取得からメール送信までの実装で漏れはないか
- テストケースは足りているか
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## クエリの変更
- ２行目で割り当てるライセンスを取得しているが、その条件を修正した
- https://ndstokyo.sharepoint.com/:u:/r/sites/Piranha/Shared%20Documents/General/OMDS/%E3%82%AF%E3%82%A8%E3%83%AA/task4132/after.log?csf=1&web=1&e=jh49c3
## 動作確認状況
- ローカルで確認、develop環境で確認など
- 行った修正がデグレを発生させていないことを確認できるか
- 追加したメール送信処理を確認するように各テストを修正し、テストが通っている
- テストの観点を拡充したうえでテストが通っていることを確認
- ライセンス割り当て履歴の内容をより詳細に確認するようにした
## 補足
- 相談、参考資料などがあれば
2024-04-25 08:56:03 +00:00
SAITO-PC-3\saito.k
b24059b538
Merge branch 'develop' into ccb
...
# Conflicts:
# dictation_client/src/translation/de.json
# dictation_client/src/translation/en.json
# dictation_client/src/translation/es.json
# dictation_client/src/translation/fr.json
# dictation_function/src/test/common/utility.ts
# dictation_server/src/features/files/test/utility.ts
2024-04-18 09:57:37 +09:00
makabe.t
69241ed36c
Merged PR 871: Function（音声ファイル自動削除）修正
...
## 概要
[Task4060: Function（音声ファイル自動削除）修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/4060 )
- 自動ファイル削除で生ファイル名を利用するように修正しUTを追従しました。
## レビューポイント
- 生ファイル名を使う方法は適切でしょうか？
- テストの修正は適切でしょうか？
## UIの変更
- なし
## クエリの変更
- select対象をraw_file_nameに修正
## 動作確認状況
- ローカルで確認
- 行った修正がデグレを発生させていないことを確認できるか
- 具体的にどのような確認をしたか
- テストが通ることを確認
2024-04-17 00:23:08 +00:00
saito.k
f975ecf551
Merged PR 869: Functions修正
...
## 概要
[Task4085: Functions修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/4085 )
- メール送信対象のアカウント取得条件を修正
- プライマリー管理者の規約同意用のカラムがNULLであった場合は、そのアカウントにはメール送信しない。
- Entityの`@Column`で設定する型が実際のパラメータの型と異なっていたため修正
- 文字列のところをDateTimeとしていた
## レビューポイント
- 修正内容の認識あっているか
- テストケースは足りているか
## クエリの変更
- Repositoryを変更し、クエリが変更された場合は変更内容を確認する
- Before/Afterのクエリ
- https://ndstokyo.sharepoint.com/:f:/r/sites/Piranha/Shared%20Documents/General/OMDS/%E3%82%AF%E3%82%A8%E3%83%AA/4085?csf=1&web=1&e=WRec5O
- 35行目に変更あり（規約系のカラムがNULLではないという条件を追加）
## 動作確認状況
- ローカルで確認
- 行った修正がデグレを発生させていないことを確認できるか
- 既存のテスト結果に影響なし
## 補足
- 相談、参考資料などがあれば
2024-04-16 05:08:04 +00:00
湯本 開
ac3d523c0e
Merged PR 826: Azure Functions実装（音声ファイル削除）
...
## 概要
[Task3880: Azure Functions実装（音声ファイル削除）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3880 )
- 自動音声ファイル削除を実装
- 上記のテストを実装
- テストにMySQLを使用する仕組みを導入
## レビューポイント
- テストケースは十分か
- テスト内容は妥当か
- developにデプロイする前の動作確認・ユニットテストとして十分か
## クエリの変更
- 新規処理のため、既存からの変更はなし
## 動作確認状況
- DBが空の状態でローカル環境で実行し、0件削除のログが出ることを確認
- 削除対象が正しいか等はdevelopでチェック予定
- 行った修正がデグレを発生させていないことを確認できるか
- 既存処理の変更はなし
2024-03-19 07:36:03 +00:00
maruyama.t
415fd2eb58
Merged PR 825: AzureFunctions実装２（取得したデータをCSV用に変換する）
...
## 概要
[Task3844: AzureFunctions実装２（取得したデータをCSV用に変換する）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3844 )
transferData()の実装
ccbの最新よりマージした状態での引数に変更
各ライセンス数をCSV配列に出力できるところまでを実装
（実際にローカルにCSV出力して中身を確認済み）
※UIの変更ではない為、ここにそのまま張り付けさせていただきます。
## レビューポイント
- 関数化の範囲は適切か。
テストコードは最低限の記述になるが、問題ないか
（33行*7項目の突合せをコード上に実装するのは時間的余裕がないためやってない）
詳細な動作確認は、別タスク
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%2029-2?workitem=3861
で行います。
## 補足
- 相談、参考資料などがあれば
2024-03-12 05:43:49 +00:00
masaaki
84fc89071a
Merged PR 811: AzureFunctions実装１（DBから必要な情報を取得する）
...
## 概要
[Task3842: AzureFunctions実装１（DBから必要な情報を取得する）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3842 )
- ライセンス数推移情報CSV出力機能のAzureFuntion用関数を追加しました
- DBから必要な情報を取得する処理を実装しました
## レビューポイント
- 特にレビューしてほしい箇所
- DBアクセス時の結合・検索条件はラフスケッチの条件に対して過不足ないか
- テストケースに不足はないか
- 関数名、構造は分かりづらくないか
## UIの変更
- 無し
## クエリの変更
- 新規のため無し
## 動作確認状況
- ローカルで処理が正常終了することを確認
- ユニットテストが通ることを確認
- 行った修正がデグレを発生させていないことを確認できるか
- Function全体のunittestを実施し通ることを確認
## 補足
- 相談、参考資料などがあれば
2024-03-11 06:04:02 +00:00
maruyama.t
0d0f624a3f
Merged PR 733: ライセンス自動割り当てのFunctionsがエラーになっている
...
## 概要
[Task3631: ライセンス自動割り当てのFunctionsがエラーになっている](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3631 )
ユーザーテーブルからライセンスアラートを削除したが、Functionsの追従が漏れていた
## レビューポイント
- とくになし
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
2024-02-07 07:34:53 +00: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
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
