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 |
|
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 |
|
湯本 開
|
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 |
|
saito.k
|
555b048ca9
|
Merged PR 12: タスク 1465: 画面実装(アカウント作成画面)
## 概要
[Task: 1465](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_sprints/taskboard/OMDSDictation%20%E3%83%81%E3%83%BC%E3%83%A0/OMDSDictation/%E3%82%B9%E3%83%97%E3%83%AA%E3%83%B3%E3%83%88%204-1?workitem=1465)
- アカウント登録
## レビューポイント
-
## 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/Task1465?csf=1&web=1&e=bHB7XJ
## 動作確認状況
- 型チェック
- ローカルで動作確認
## 補足
|
2023-03-14 07:50:25 +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 |
|
makabe.t
|
41f0213fe9
|
Merged PR 1: タスク 1359: API実装(認証/IDトークン検証)
## 概要
[タスク 1359: API実装(認証/IDトークン検証)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_workitems/edit/1359)
- IDトークンを受け取って内容を検証し、デコードしたペイロードを返すサービスを実装しました。
## レビューポイント
- サービスの処理の流れが認識とあっているか。
- ADB2CのAPI呼び出しを別サービスにしているが問題ないか
- 公開鍵の変換処理を別サービスに切り出しているが構成に問題はないか。
- トークンの検証をエラーごとに処理しているがエラー内容は認識通りか
## UIの変更
- なし
## 動作確認状況
- テストが通ることを確認
# 備考
- IDトークンを検証して中身を返すまでの実装です。
|
2023-03-07 01:30:13 +00:00 |
|
斎藤 快斗
|
3b4b3c59e7
|
Merged PR 4: API実装のためのディレクトリを構成
## 概要
[Task: 1358](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_sprints/taskboard/OMDSDictation%20%E3%83%81%E3%83%BC%E3%83%A0/OMDSDictation/%E3%82%B9%E3%83%97%E3%83%AA%E3%83%B3%E3%83%88%202_2?workitem=1358)
- API実装を行うためのベースとなる実装を行いました。
- ディレクトリ構成
- ビルドの設定
- openapi.jsonの配置場所
- openapi.jsonの生成コマンド作成
## レビューポイント
- ベース実装として足りているか
- openapi.jsonを生成するコードの配置場所は`src/api/generate.ts`で良いか
## UIの変更
- なし
## 動作確認状況
- ローカルで`/health`が200 OKを返却することを確認
## 補足
|
2023-02-20 08:12:36 +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 |
|