oura.a
|
3584a65682
|
Merged PR 228: Revert "Merged PR 218: 画面実装(カードライセンス取り込みPU)
Revert "Merged PR 218: 画面実装(カードライセンス取り込みPU)
## 概要
[Task2161: 画面実装(カードライセンス取り込みPU)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2161)
タスク 2161: 画面実装(カードライセンス取り込みPU)
カードライセンス取り込みポップアップを実装しました。
以下の項目については別タスクへ切り出しての対応とし、本タスクでは対象外とさせてください。
・テキストボックスへの4文字区切りの入力
・テキストボックスの文字数制限
・改行コード入力時の挙動(バーコードリーダー対応)
## レビューポイント
なし
## UIの変更
https://ndstokyo.sharepoint.com/:i:/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/Task2161/%E3%82%AB%E3%83%BC%E3%83%89%E3%83%A9%E3%82%A4%E3%82%BB%E3%83%B3%E3%82%B9%E5%8F%96%E3%82%8A%E8%BE%BC%E3%81%BFPU.PNG?csf=1&web=1&e=cRtMX3
## 動作確認状況
ローカルで動作確認済み
## 補足
なし"
Reverted commit `9a66ca02`.
|
2023-07-11 05:50:51 +00:00 |
|
oura.a
|
9a66ca027a
|
Merged PR 218: 画面実装(カードライセンス取り込みPU)
## 概要
[Task2161: 画面実装(カードライセンス取り込みPU)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2161)
タスク 2161: 画面実装(カードライセンス取り込みPU)
カードライセンス取り込みポップアップを実装しました。
以下の項目については別タスクへ切り出しての対応とし、本タスクでは対象外とさせてください。
・テキストボックスへの4文字区切りの入力
・テキストボックスの文字数制限
・改行コード入力時の挙動(バーコードリーダー対応)
## レビューポイント
なし
## UIの変更
https://ndstokyo.sharepoint.com/:i:/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/Task2161/%E3%82%AB%E3%83%BC%E3%83%89%E3%83%A9%E3%82%A4%E3%82%BB%E3%83%B3%E3%82%B9%E5%8F%96%E3%82%8A%E8%BE%BC%E3%81%BFPU.PNG?csf=1&web=1&e=cRtMX3
## 動作確認状況
ローカルで動作確認済み
## 補足
なし
|
2023-07-11 05:06:10 +00:00 |
|
makabe.t
|
773c8894e7
|
Merged PR 208: API実装(音声ファイルDL元)
## 概要
[Task2038: API実装(音声ファイルDL元)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2038)
- 音声ファイルのダウンロードURL取得API&テストを実装しました。
- SASトークンの有効期限を2時間にしています。
- タスク作成APIでURLのSASトークンを取り除く処理を追加しています。
## レビューポイント
- URLの生成に問題はないか
- テストのためにmodule生成処理を追加したが問題ないか
- タスク作成APIでのURL処理は認識通りか
※対象外:テンプレートファイル関連
以下はコード整形による変更なので対象外
- licenses.repository.module.ts
- tasks.service.spec.ts
## UIの変更
なし
## 動作確認状況
- ローカルで確認
|
2023-07-07 06:57:29 +00:00 |
|
saito.k
|
4cf444ab42
|
Merged PR 98: API実装(SASトークン発行)
## 概要
[Task1737: API実装(SASトークン発行)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1737)
- SASトークンを発行する処理を実装
- コンテナ作成処理を実装
- 本来はアカウント登録時に作成されるが、動作検証のためSAS発行時に作成するように実装している
- コンテナ存在確認の処理を実装
## レビューポイント
- 実装した処理に漏れはないか
- エラーの処理に考慮漏れはないか
- テストケースは十分か
## 動作確認状況
- ローカルでコンテナ作成とURL発行を確認
## 補足
- 相談、参考資料などがあれば
|
2023-05-17 00:38:39 +00:00 |
|
makabe
|
50cbcb1d81
|
メール認証URLのドメインを環境変数で設定
|
2023-04-11 17:50:39 +09:00 |
|
saito.k
|
ee2e8dbd5d
|
Merged PR 64: デバイス登録API実装
## 概要
[Task1571: デバイス登録API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1571)
- NotificationHubへのデバイス登録APIを実装
- NotificationHubへの接続
- 登録処理
## レビューポイント
- ディレクトリ構成に問題は無いか
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルでNotificationHubに接続できるところまで確認
## 補足
- 返却するエラーやサーバー側で設定する部分の登録情報については仮です。
- 「プロダクト バックログ項目 1389: デスクトップアプリ/モバイルアプリに様々な通知をしたい」で実装・設計する想定
|
2023-04-10 00:41:32 +00:00 |
|
makabe.t
|
0099614a5f
|
Merged PR 58: API実装(アカウント登録/Azure AD B2C)
## 概要
[Task1550: API実装(アカウント登録/Azure AD B2C)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1550)
- アカウント登録APIでAzure ADB2Cにユーザを登録する処理を追加しました。
- GraphAPIで登録処理をしています。
## レビューポイント
- ADB2Cへ登録する情報は認識通りか
- ADB2CへのGraphAPI接続のためにADB2Cテナントにアプリを追加しているが認証として問題ないか。
- 環境変数にアプリの情報を設定しています
- ADB2C_TENANT_ID=xxxxxxxx
- ADB2C_CLIENT_ID=xxxxxxxx
- ADB2C_CLIENT_SECRET=xxxxxxxx
## UIの変更
無し
## 動作確認状況
- ローカルで確認
- ADB2C、DBに設定項目が追加されていることを確認
|
2023-04-05 09:22:50 +00:00 |
|
makabe.t
|
731c633189
|
Merged PR 57: ログインAPIを修正する
## 概要
[Task1504: ログインAPIを修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1504)
- ログイン時にIDトークンをもとにメールアドレスで認証済みのユーザかをDBから取得して判定する処理を追加しました
## レビューポイント
- IDトークンのSubをもとにUserテーブルの`external_id`でユーザを特定しているが利用法として適切か
- DBからのクエリに問題はないか
- ※アカウント登録の処理が含まれていますが、以下の変更についてご確認をお願いします。
- feature/auth/*
- repositories/users/*
- common/error
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-03-31 07:46:56 +00:00 |
|
湯本 開
|
28a7e246a2
|
Merged PR 60: アカウント登録APIでメール送信するための設定を修正する
## 概要
[Task1565: アカウント登録APIでメール送信するための設定を修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1565)
- .envにも定義されているとそちらを優先して読んでしまうため、環境変数を修正
## レビューポイント
- 情報共有
## 動作確認状況
- ローカルで確認
|
2023-03-29 08:14:34 +00:00 |
|
湯本 開
|
9f5252baf8
|
Merged PR 56: API実装(アカウント登録)
## 概要
[Task1496: API実装(アカウント登録)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1496)
- アカウント登録のAzureAD B2Cへのユーザー作成部分以外について実装
- migration SQLに足りない要素があった箇所を修正(default追加)
- 各種環境変数を追加
- 秘密鍵/公開鍵を取得する方法を環境変数に変更(KeyVaultからWebAppsの機能で環境変数へ流し込む想定)
## レビューポイント
- 実装方法として問題がありそうな箇所が存在しないか
- 可読性の低い箇所が存在しないか
- Moduleの分け方、つなげ方などは問題ないか
- ラフスケッチと違い、Account作成と管理者ユーザー作成を同一トランザクションで行うよう修正したが問題ないか
## 動作確認状況
- メール送信以外はローカルで確認、メール送信部分は未確認
|
2023-03-29 03:55:44 +00:00 |
|
湯本 開
|
c1ed541d87
|
Merged PR 4: タスク 1385: API実装(認証/トークン発行)
## 概要
[Task 1385](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_workitems/edit/1385)
- IDトークンを受け取り、リフレッシュトークンとアクセストークンを発行するAPIを実装
- リフレッシュトークンを受け取り、アクセストークンを発行するAPIを実装
- アクセストークン、リフレッシュトークンの中身は中身を使用するPBIが存在しないので仮のもの
## レビューポイント
- 本PBIで必要な範囲の機能を十分持っているか
- コードに不足部分、問題点などはなさそうか
- テスト項目は妥当か
- アクセストークン、リフレッシュトークンの中身はまだ仮で問題ないか
## 動作確認状況
- ローカルで動作確認
|
2023-03-07 23:35:53 +00:00 |
|
拓海 真壁
|
e686db674f
|
Merged PR 1: タスク 1176: 開発環境コンテナの構築(Client/Server)
[タスク 1176: 開発環境コンテナの構築(Client/Server)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_workitems/edit/1176)
開発用コンテナを構築しました。以下のコンテナを追加しています。
- ライセンス管理
- server
- client
- ディクテーション管理
- server
- client
- DB(MySQL)
- Cache(Redis)
|
2023-01-12 08:56:38 +00:00 |
|