makabe.t
|
33d4ab3d2f
|
Merged PR 863: （Sprint31対応）一部のentityでMySQL上の型がbigintのものに対してbigintTransformerで変換する処理が入っていない
## 概要
[Task3928: （Sprint31対応）一部のentityでMySQL上の型がbigintのものに対してbigintTransformerで変換する処理が入っていない](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3928)
- 現在、bigint型で変換を適用していない部分へのTransformerの適用はうまくいかないため保留。
- 既存処理については問題ありません
- Functionのテストで日付に依存している箇所があったので修正しています。
## レビューポイント
- 共有
## UIの変更
- なし
## クエリの変更
- なし
## 動作確認状況
- ローカルで確認
- 行った修正がデグレを発生させていないことを確認できるか
- テストの修正のみ
|
2024-04-11 04:29:16 +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
|
cab7a75ec1
|
Merged PR 849: 有効なアカウント内の削除されたユーザーの割り当て履歴が集計されない
## 概要
[Task3929: 有効なアカウント内の削除されたユーザーの割り当て履歴が集計されない](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3929)
- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）
## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認、develop環境で確認など
## 補足
- 相談、参考資料などがあれば
|
2024-03-15 12:19:16 +00:00 |
|
maruyama.t
|
2b68a9f054
|
Merged PR 824: AzureFunctions実装3（CSVをストレージアカウントに配置する）
## 概要
[Task3846: AzureFunctions実装3（CSVをストレージアカウントに配置する）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3846)
outputDataを追加
→outputAnalysisLicensesDataに変更（アラートルールなどでログを見るので、何の処理か理解できるように）
blobstorageService.tsに以下を追加
- uploadFileAnalysisLicensesCSV
（ライセンスCSVを配置する）
- createContainerAnalysisを追加
（コンテナーを作成する）
環境変数の追加
## レビューポイント
- 今回追加されたJP-EASTのストレージアカウントのコンテナーが、第一階層のアカウントのものであるかどうかはソース上は特に意識していないが問題ないでしょうか。
## 動作確認状況
- ローカルで確認（モックでソース上処理が通ることのみ確認のみ）
詳細なテストは別タスクで行う。
## 補足
- 相談、参考資料などがあれば
|
2024-03-13 07:54:10 +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 |
|
