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

14 Commits

Author SHA1 Message Date
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
saito.k
566da623bf Merged PR 873: dev動作確認不具合修正 
## 概要
[Task4131: dev動作確認不具合修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/4131)

- メール送信対象のアカウント取得条件を修正
  - 第五階層はDPAの規約に同意することがないため、accepted_dpa_versionは常にNULLとなる
    - 取得条件にaccepted_dpa_versionがNOTNULLを追加するとカラムに値が入る契機がないのでアカウントを取得できなくなってしまっていた。

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

## 動作確認状況
- ローカルで確認
- 行った修正がデグレを発生させていないことを確認できるか
  - ほかのテストケースで使用しているユーザーデータをaccepted_dpa_versionはNULLの状態で作成するようにし、テストがすべて通ることを確認

## 補足
- 相談、参考資料などがあれば
 2024-04-17 01:01:31 +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
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
makabe.t
1524ec2473 Merged PR 701: パイプラインエラー対応 
## 概要
[Task3531: パイプラインエラー対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3531)

- パイプラインエラー解消

## レビューポイント
- 共有
 2024-01-24 00:48:21 +00:00
saito.k
46fdef854e Merged PR 689: licenseAlertのテスト修正 
## 概要
[Task3275: licenseAlertのテスト修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3275)

- redisをモックに切り替えてテストを行うように修正
  - redisのモックは`redis-mock`というライブラリを使用

## レビューポイント
- 使用したライブラリに問題はありそうか
  - https://www.npmjs.com/package/redis-mock/v/0.56.3
  - テスト用に使用しているだけなので大きな問題になることはない認識

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

## 動作確認状況
- ローカルで確認、Pipeline上でテストが通ることを確認

## 補足
- 相談、参考資料などがあれば
 2024-01-17 06:02:20 +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
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
masaaki
40162ef3af Merged PR 568: Azure Function App上でのDBアクセス・メール送信疎通実施 
## 概要
[Task2867: Azure Function App上でのDBアクセス・メール送信疎通実施](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2867)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
  - ライセンスアラート処理がAzureFunction上で動作するよう構成変更を行いました
    - DBアクセス、SendGrid、AADB2Cアクセスに必要な環境変数を追加
    - kv-odms-secret-devのアクセスポリシにfunc-odms-dictation-devを追加
    - ネットワーク設定にvnet-odms-network-dev/snet-odms-integration-devを追加
  - プロジェクトの内容もいくつか修正しています
    - 最新のpackage.jsonに合わせてpackage-lock.jsonを生成
    - licenseAlertのスケジュールを毎日1:00（UTC）に変更
    - ビルド用にDockerfileを復活
- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）

## レビューポイント
- 特段注視する部分はありません。ソース上の修正はスケジュールの変更のみです。

## UIの変更
- 無し

## 動作確認状況
- unitテスト実施、ローカルで確認

## 補足
- 相談、参考資料などがあれば
 2023-11-14 12:08:30 +00:00
oura.a
aeb2eb392a Merged PR 572: メール内容多言語対応、可変文字列対応 
## 概要
[Task3022: メール内容多言語対応、可変文字列対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3022)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
  - 仮だったメール内容について、ユーザ提示の内容とし、可変部分の対応を行いました。
- このPull Requestでの対象/対象外
  - 複数言語の対応について、現状は各言語の内容を縦に並べる形としています。
    11/14現在、奥澤さんが調査・調整中で、最終的な他言語対応は
  「プロダクト バックログ項目 1660: メール送信機能の実装」の中で実施になります。
  - [メール内容の外出しなど、編集しやすくする対応についても「プロダクト バックログ項目 1660: メール送信機能の実装」にて実施になります。
- 影響範囲（他の機能にも影響があるか）

## レビューポイント
- 特にレビューしてほしい箇所
  - 特筆してみていただきたい部分はありません。全体見ていただき違和感なければOKと思っています。
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載

## 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/Task3022?csf=1&web=1&e=0PB6iI

## 動作確認状況
- ユニットテスト実施＋ローカルで確認

## 補足
- 相談、参考資料などがあれば
 2023-11-14 06:51:13 +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