makabe.t
|
279a9ab037
|
Merged PR 889: function修正
## 概要
[Task4164: function修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/4164)
- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）
## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
- 修正箇所がほかの機能に影響していないか
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## クエリの変更
- Repositoryを変更し、クエリが変更された場合は変更内容を確認する
- Before/Afterのクエリ
- クエリ置き場
## 動作確認状況
- ローカルで確認、develop環境で確認など
- 行った修正がデグレを発生させていないことを確認できるか
- 具体的にどのような確認をしたか
- どのケースに対してどのような手段でデグレがないことを担保しているか
## 補足
- 相談、参考資料などがあれば
|
2024-05-07 07:11:39 +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 |
|
masaaki
|
340aa73bde
|
Merged PR 827: CSV出力が失敗したときに手動で起動するためのFunctionsを作成
## 概要
[Task3860: CSV出力が失敗したときに手動で起動するためのFunctionsを作成](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3860)
- analysisLicensesについて手動で起動できる処理（analysisLicensesManualRetry）を追加しました
- データベース接続の初期化処理について共通化しました
## レビューポイント
- 特筆する部分はありません
## UIの変更
- 無し
## クエリの変更
- 無し
## 動作確認状況
- ローカルでpostmanからリクエストを行うことで起動できることを確認
- 行った修正がデグレを発生させていないことを確認できるか
- 具体的にどのような確認をしたか
- unit testが通ることを確認
- タイマを暫定的に1分にして、ローカル環境で各タイマ処理が正常終了することを確認（DBアクセスが全処理行われることを確認）
## 補足
- 相談、参考資料などがあれば
|
2024-03-12 04:55:04 +00:00 |
|
makabe.t
|
8aa45baee8
|
Merged PR 697: DB関連コード修正
## 概要
[Task3509: DB関連コード修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3509)
- 参照するDBがCCB用のスキーマとなるようにserverの参照DB名を変更しました。
対象：
- server
- .env
- app.module.ts
- function
- .env
- functions/*
※マージの取り込み分も入ってしまったので他は無視してください。
## レビューポイント
- 対応箇所は適切でしょうか。
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
- ローカルにスキーマを追加・マイグレーションしてserverを起動できることを確認
|
2024-01-24 02:52:13 +00:00 |
|
maruyama.t
|
0a9f125d76
|
Merged PR 668: 有効期限＞翌日になっているのを＞当日に修正する
## 概要
[Task3421: 有効期限＞翌日になっているのを＞当日に修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3421)
- 有効期限＞翌日になっているのを＞当日に修正
翌日が有効期限のものは割り当ての対象とする。
念のためテストケースを追加
## レビューポイント
- とくになし
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-12-28 04:27:19 +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にアクセスする。
②左カラムの「コードとテスト」を押下し、「テストと実行」を押下する。
③入力欄の、「クエリ」にdateと日付をハイフン区切りで入力して、「実行」を押下する。
|
2023-12-25 04:33:12 +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 |
|
