saito.k
|
e448e8d249
|
Merged PR 806: メール文面の修正反映
## 概要
[Task3792: メール文面の修正反映](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3792)
- メール文面を修正
- ドイツ語の部分
## レビューポイント
- タスクに添付しているエクセルの内容と修正した内容を照らし合わせて確認していただきたいです。
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## クエリの変更
- Repositoryを変更し、クエリが変更された場合は変更内容を確認する
- Before/Afterのクエリ
- クエリ置き場
## 動作確認状況
- ローカル環境でメールを確認
- メール文面のみの修正のため、他機能に影響はない
## 補足
- 相談、参考資料などがあれば
|
2024-03-07 02:26:15 +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 |
|
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 |
|
maruyama.t
|
658de1c1ae
|
Merged PR 691: ライセンス自動割り当てのテストコードの結果が異常になる
## 概要
[Task3486: ライセンス自動割り当てのテストコードの結果が異常になる](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3486)
現在時刻と明示的に定義した日付の比較を行っていたことで、年を跨ぎ異常となっていた
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認、develop環境で確認など
## 補足
- 相談、参考資料などがあれば
|
2024-01-16 09:51:37 +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 |
|
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にアクセスする。
②左カラムの「コードとテスト」を押下し、「テストと実行」を押下する。
③入力欄の、「クエリ」に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 |
|
masaaki
|
342baa9826
|
Merged PR 610: AzureFucntionのpipelineが参照しているDockerFileを修正する
## 概要
[Task3274: AzureFucntionのpipelineが参照しているDockerFileを修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3274)
- staging用のpipelineに対して、functionのビルド時に参照するdockerFileをappserviceと同様の階層にあるものに修正しました
## レビューポイント
- 特にありません
## UIの変更
- 無し
## 動作確認状況
- 未確認。stagingのpipeline動作時に確認します。
## 補足
- 相談、参考資料などがあれば
|
2023-12-07 06:09:48 +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 |
|
makabe.t
|
6db8b4cbd7
|
Merged PR 599: ライセンスアラート関連：DockerfileFunctionDictation.dockerfileを参照するようファイルとpipelineを修正する
## 概要
[Task3083: ライセンスアラート関連：DockerfileFunctionDictation.dockerfileを参照するようファイルとpipelineを修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3083)
- Azure Functionsのイメージを`DockerfileFunctionDictation.dockerfile`で定義するように修正しました。
- マルチステージビルドの形式で必要なファイルだけ取り込んだイメージとなるようにしています。
- 既定のパス`/home/site/wwwroot`をワークディレクトリとしています。
## レビューポイント
- 生成物イメージに入れるファイルに過不足はないでしょうか？
- 環境変数でのパス指定ができないため、既定のパスで動作するようにしていますが不自然な点はないでしょうか？
## UIの変更
- なし
## 動作確認状況
- develop環境でデプロイして確認
|
2023-12-04 08:18:18 +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 |
|
oura.a
|
86d17d6729
|
Merged PR 552: dictation_serverからソースコードを複製
## 概要
[Task2977: dictation_serverからソースコードを複製](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2977)
新たに構築したdictation_functionで以下のことをできるようにしました。
・RDBへの接続
・sendgridでのメール送信
・jestでのテスト実行
※とりあえず動くことを目標としているため、DB接続のパラメータやsendgridのAPIキーなどがベタ打ちになっています。機能実装時には外出ししたファイルから読み込めるようにします。
## レビューポイント
・フォルダ構成は適切か
・RDB、sendgrid、jestに関するもので不足しているものがないか
・今後機能開発を始めるにあたり、他に必要なものがないか
## UIの変更
なし
## 動作確認状況
ローカルでjestによるテストを実施し、以下を確認
・RDBからデータが取得できる
・メールが送信され、設定したアドレスで受信できる
## 補足
なし
|
2023-11-08 00:46:26 +00:00 |
|
oura.a
|
42dab2a45d
|
Merged PR 532: function構築の動作確認用ブランチ
## 概要
[Task2954: function構築の動作確認用ブランチ](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2954)
- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- dev環境にfunctionAppが動作する環境を構築しました。
- 新規にDICTATION_FUNCTIONプロジェクトを追加
- 関数アプリリソース「funcapp-odms-dictation-dev」を追加
- 「funcapp-odms-dictation-dev」に付随してストレージアカウント「saomdsdevelopfuncapp」を追加
- pipeline「ODMS Function Deploy」を追加
- 構築の流れをwikiに更新しています。
https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_wiki/wikis/OMDSDictation_wiki/280/FunctionApp%E3%81%AE%E4%BD%9C%E6%88%90
- このPull Requestでの対象/対象外
- configurationsについて、結構差分がたまっている状態でした。本タスクで更新したのは以下になり、それ以外は対象外です。
- dev-application-rg.json：funcapp-odms-dictation-dev
- dev-application-rg.json：saomdsdevelopfuncapp
- pipelineについては精査が必要です。「タスク 2992: pipelineの最適化」にて対応予定
- 影響範囲（他の機能にも影響があるか）
## レビューポイント
- 特にレビューしてほしい箇所
- .devcontainerの内容について、VSCodeからコンテナで動かすためDICTATION_SERVERから取得しています。入れておくと問題のある設定があれば指摘いただければと思います。
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
## UIの変更
- 無し
## 動作確認状況
- ローカルで確認＋develop環境で確認
## 補足
- 相談、参考資料などがあれば
|
2023-11-06 00:29:47 +00:00 |
|
oura.a
|
911d028073
|
Merged PR 510: [sprint20完了]新規Dockerコンテナの作成
## 概要
[Task2864: [sprint20完了]新規Dockerコンテナの作成](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2864)
- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）
## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認、develop環境で確認など
## 補足
- 相談、参考資料などがあれば
|
2023-10-25 07:03:48 +00:00 |
|
