OMDSCloud

nik/OMDSCloud

Fork 0

Commit Graph

Author	SHA1	Message	Date
makabe	50cbcb1d81	メール認証URLのドメインを環境変数で設定	2023-04-11 17:50:39 +09:00
makabe.t	50f4cf5070	Merged PR 65: 画面実装（ユーザー認証画面） ## 概要 [Task1495: 画面実装（ユーザー認証画面）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1495) - メールの認証URLから、認証を実行して結果を表示するまでの画面を実装しました。 - 認証画面 - 認証完了画面 - 成功 - 失敗 - 認証済み - エラーハンドリング用のメソッドを`common`に追加しました。 - メールに送信される認証URLのパスを認証画面のパスに修正しました。 ## レビューポイント - エラーハンドリング用の処理は適切でしょうか？ - 改行を画面に対応させるために暫定の処置を入れています。対応に問題はないでしょうか。 - アカウント登録のメール送信について、パスを対象となる画面のパス`mail-confirm/`に変更しました。 - 対応として適切でしょうか？ ## UIの変更 - [Task1495](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/Task1495?csf=1&web=1&e=bqT7nz) ## 動作確認状況 - ローカルで確認	2023-04-10 09:03:21 +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

Author

SHA1

Message

Date

makabe

50cbcb1d81

メール認証URLのドメインを環境変数で設定

2023-04-11 17:50:39 +09:00

makabe.t

50f4cf5070

Merged PR 65: 画面実装（ユーザー認証画面）

## 概要
[Task1495: 画面実装（ユーザー認証画面）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1495)

- メールの認証URLから、認証を実行して結果を表示するまでの画面を実装しました。
  - 認証画面
  - 認証完了画面
    - 成功
    - 失敗
    - 認証済み
- エラーハンドリング用のメソッドを`common`に追加しました。
- メールに送信される認証URLのパスを認証画面のパスに修正しました。

## レビューポイント
- エラーハンドリング用の処理は適切でしょうか？
- 改行を画面に対応させるために暫定の処置を入れています。対応に問題はないでしょうか。
- アカウント登録のメール送信について、パスを対象となる画面のパス`mail-confirm/`に変更しました。
  - 対応として適切でしょうか？

## UIの変更
- [Task1495](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/Task1495?csf=1&web=1&e=bqT7nz)

## 動作確認状況
- ローカルで確認

2023-04-10 09:03:21 +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

3 Commits