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 |
|
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 |
|
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 |
|