oura.a
|
709db0d2ec
|
Merged PR 187: 画面実装（カードライセンス発行PU）
## 概要
[Task1991: 画面実装（カードライセンス発行PU）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1991)
カードライセンス発行ポップアップを新規作成。
上記ポップアップを起動するボタンを第１～４階層用のライセンスページに仮配置。
## レビューポイント
特筆するものはなし。
ライセンスページのポップアップ起動ボタンは仮配置なので、レビュー対象外でお願いします。
## 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/Task1991/%E3%82%AB%E3%83%BC%E3%83%89%E3%83%A9%E3%82%A4%E3%82%BB%E3%83%B3%E3%82%B9%E7%99%BA%E8%A1%8CPU.PNG?csf=1&web=1&e=NtuRgu
## 動作確認状況
ローカルで動作確認済み
## 補足
・csvファイルダウンロード確認のポップアップは、現在は表示しないつくりになっています。PBIの受け入れ条件には表示するように記載があるのですが、月曜日のDSで奥澤さんに確認予定です。
・ライセンス数入力のテキストボックスで「.」「-」「+」が入力できてしまう事象がありました。こちらですがライセンス注文ポップアップでも同様の仕様となっているため、本PBI内の別タスクで纏めての修正とさせてください。
|
2023-07-03 05:35:19 +00:00 |
|
makabe.t
|
e4bc4776b0
|
Merged PR 184: ローディング表現に関する実装等を行う
## 概要
[Task2015: [Task1895完了後][Sp11-2着手] ローディング表現に関する実装等を行う](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2015)
- 現在実装中のテーブル、ポップアップについてローディング表現を実装しました。
- ローディング中にローディング中を示すぐるぐるを表示
- ローディング中はボタンを非活性にする
## レビューポイント
- 対応箇所は適切か
- 表示内容に問題はないか
## UIの変更
- [Task2015](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/Task2015?csf=1&web=1&e=TmrOXa)
## 動作確認状況
- ローカルで確認
|
2023-06-30 05:51:14 +00:00 |
|
saito.k
|
f20fb5b362
|
Merged PR 188: 処理成功時、スナックバーを表示する
## 概要
[Task2088: 処理成功時、スナックバーを表示する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2088)
- タイピスト割り当て変更処理の成功時、成功メッセージを表示する
- 他画面に同様の文言が存在していたため、共通のメッセージとして位置づけを変更
- 同様の文言を表示していた箇所のメッセージ変数名を修正
## レビューポイント
## 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/Task2088?csf=1&web=1&e=g2r2ca
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-06-28 08:47:23 +00:00 |
|
makabe.t
|
33fc741eee
|
Merged PR 156: タイピスト割り当てポップアップ実装＆画面修正
## 概要
[Task1933: タイピスト割り当てポップアップ実装＆画面修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1933)
- タイピスト割り当てポップアップを実装しました。
- ポップアップをタスクがUploaded時のみ開けるようにしています。
- タイピスト割り当てを変更・保存できるようにしています。
## レビューポイント
- 画面実装のデザインは適切か
- データの保持方法は適切か
## UIの変更
- [Task1933](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/Task1933?csf=1&web=1&e=4ghlqe)
## 動作確認状況
- ローカルで確認
- API連携は未確認
|
2023-06-23 04:57:54 +00:00 |
|
saito.k
|
2155539365
|
Merged PR 175: タスク一覧画面でタスクステータスのフィルターをすべて外すとエラーとなる
## 概要
[Task2037: 対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2037)
- フィルターの選択をすべて未選択にするとエラーになる挙動を修正
- フィルターがすべて未選択時に画面側から空文字が渡されていた。
- APIはstetusに値が入っていると判断して、バリデーションを行っていたためエラーになっていた。
- フィルターがすべて未選択の場合は画面側からundefinedを渡すように修正した。
## レビューポイント
- 修正箇所は画面側でよいか
- 修正内容に問題はあるか
## 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/Bug2037?csf=1&web=1&e=pJJk6P
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-06-22 08:07:31 +00:00 |
|
Kentaro Fukunaga
|
97bb9c4190
|
Merged PR 174: アカウント登録時・ユーザー追加時のパスワードポリシーを、AADB2C標準に合わせる
## 概要
[Task1958: アカウント登録時・ユーザー追加時のパスワードポリシーを、AADB2C標準に合わせる](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1958)
- 利用可能な文字種を修正
- 開発規約上「推奨」となっていたルールを排除
- フロント側の潜在バグ（パスワードの長さチェック）を修正かつリファクタ
## レビューポイント
- 無駄な書き方になっているようなところはないか
## UIの変更
- 無し
## 動作確認状況
- ローカルでアカウント登録時、パスワードポリシー満たしていないものであれば次画面遷移できないことを確認
- ADB2Cユーザーフロー画面でパスワードリセット時に通過するルールと一致していることを確認しました。
## 補足
- 相談、参考資料などがあれば
|
2023-06-22 06:28:57 +00:00 |
|
maruyama.t
|
f0e0d5e6b1
|
Merged PR 162: API IF修正（LicenseSummaryInfo）
## 概要
[Task1961: API IF修正（LicenseSummaryInfo）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1961)
- 何をどう変更したか、追加したライブラリなど
API実装時に一時的にLicenseSummaryInfo2と命名していた定義を、LicenseSummaryInfoに修正。
それに伴い、既存のLicenseSummaryInfoはレスポンスに直接記載するよう修正。
openapi.jsonを再生成し、画面側のパラメータの取得の記載を修正。
- このPull Requestでの対象/対象外
storageSizeとusedSizeの取得はPBI対象外
- 影響範囲（他の機能にも影響があるか）
なし
## レビューポイント
- クライアント側に対する修正漏れがないか。
## UIの変更
なし
## 動作確認状況
- ローカルでAPIの戻り値の確認と、画面にその値が表示されていることを確認。
## 補足
- 相談、参考資料などがあれば
|
2023-06-19 04:06:44 +00:00 |
|
masaaki
|
1203d6bb99
|
Merged PR 148: 画面実装（第5階層用ライセンス情報）
## 概要
[Task1842: 画面実装（第5階層用ライセンス情報）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1842)
- 第5階層用ライセンス情報画面の本実装を行いました
- ボタン押下確認のため、遷移後の画面（注文履歴画面、カードライセンス取り込みポップアップ）も仮実装しています
- このPull Requestでの対象外
・api.ts、app.module.scss → openapi.json及びデザイナさんcssを取り込んだもののため
・cardLicenseImportPopup.tsx、LicenseOrderHistoryPage → 仮実装画面のため
・licenseOrderフォルダ構成を変更しただけのため
## レビューポイント
- License周りの構成について見直しを行っています。
これまではfeatures/licenseの直下にstateやoperationを配置していましたが、
LicenseページとURLを同じくする画面が増えるため、サブフォルダを作っています。
features/license
/licenseOrder // これまで直下にあったもの
/licenseSummary // 今回作成したもの
構成について問題ないか確認をお願いします。
## 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/Task1842?csf=1&web=1&e=OdR2mR
## 動作確認状況
- ローカルで確認
## 補足
- 特にありません
|
2023-06-13 07:07:06 +00:00 |
|
maruyama.t
|
25072f558f
|
Merged PR 132: 画面実装（ページヘッダ修正）
## 概要
[Task1840: 画面実装（ページヘッダ修正）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1840)
- JWTからアクセストークン内のロールを確認する
- 管理者権限（admin）を持たない場合はライセンスタブを表示しない
## レビューポイント
- ライセンスタブを表示しない処理を.filterで行ってるが、問題ないか
## UIの変更
- ライセンスタブの表示非表示
## 動作確認状況
- ローカルで確認、adminユーザで表示されること
standardユーザで表示されないことを確認
## 補足
- `// userRole: "user" | "partner"; ログインユーザーのロールに応じてタブの活性非活性に使用する想定`
の記述は今回使わなかったので削除しました。
|
2023-06-08 09:22:06 +00:00 |
|
makabe.t
|
3dca5c604b
|
Merged PR 126: 画面実装（タスク一覧画面）
## 概要
[Task1834: 画面実装（タスク一覧画面）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1834)
- タスク一覧画面を実装しました。
- 表示項目、フィルターを制御できるようにしています
- 表示項目はlocalStorageに保存するようにしています
- ソート順の取得、変更、更新をする処理を追加
## レビューポイント
- デザインの反映は適切か
- フィルター、表示の反映ロジックは適切か
- ソートの反映を追加したので適切か確認をお願いします。
## UIの変更
- [Task1834](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/Task1834?csf=1&web=1&e=zBcQ6i)
## 動作確認状況
- ローカルで確認
- API呼び出しは未検証
|
2023-06-08 08:04:34 +00:00 |
|
makabe.t
|
e4f84f78ba
|
Merged PR 121: リフレッシュトークンの発行方法・発行内容が不適切な問題の修正
## 概要
[Task1904: リフレッシュトークンの発行方法・発行内容が不適切な問題の修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1904)
セキュリティ・権限管理の要であるリフレッシュトークンのロール設定に関する修正ですので、開発メンバー全員にご確認いただき、変更内容について認識を合わせていただきたいです。
- リフレッシュトークンの発行で使用するロールがクライアントから送られた値をそのまま利用する実装になっていましたので、APIのロールの取り扱いを修正しました
- ユーザー追加APIで想定している（author/typist/none）のみを受け付けるようにする
- 想定外のロールの場合は400エラー
- クライアントからも想定通りのロールを送信するようロールの定数値を修正
- リフレッシュトークンを発行する際にDBの値をそのまま使わず、想定値のどれかを判断して定数を設定する
- DBに登録されているロールが想定外の文字列の場合は500エラーとなる
## レビューポイント
- トークンのロール設定が安全にできるようになっていることをメンバー全員で合意
- APIのパラメータを受け付ける際にIsInで想定ロール文字列のみを受け付けるようにしているが問題はないか
- リフレッシュトークンの生成時に直接DBのロールを設定しないようにしたが、トークンのロール設定として問題はないか
- 意図しないトークンへの権限の付与などは起こりえないか。
- クライアントからadminなどのユーザー追加時に想定していない権限を付与しようとしてもAPIではじかれるか
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
## 補足
- ユーザー追加時に登録されるロールが変更となったのでマージ後に利用する場合は再度の登録をお願いします。
|
2023-06-02 04:55:28 +00:00 |
|
oura.a
|
81dca16d5f
|
Merged PR 113: 画面実装（ライセンス注文ポップアップ）
## 概要
[Task1684: 画面実装（ライセンス注文ポップアップ）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1684)
- ライセンス注文ポップアップ画面について実装を行いました
- ライセンス情報画面について、ボタン押下時にポップアップを表示する実装も併せて実施しております
- ポップアップの表示について、表示/非表示を制御するのではなくそもそも表示しない場合はhtmlとして生成しないよう制御を入れています。
- ポップアップ表示の対応により、ポップアップクローズ時の対応が簡易化しています。（これまでは、ポップアップを閉じる際に値を初期化しなおす必要があったのですが、生成しなおす形になるので初期生成の考慮のみでよくなる）
- cssについて、5/26時点で最新のファイルを取り込みました
## レビューポイント
- ポップアップ生成周りの考え方、初期化の実装内容
- images、cssについては、最新のデザイナさんのものを取り込んだ形なので、レビュー対象外でお願いします
## 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/Task1684?csf=1&web=1&e=xLRn4D
## 動作確認状況
- ローカルで確認
## 補足
- 無し
|
2023-05-30 01:07:52 +00:00 |
|
oura.a
|
4a5307842c
|
Merged PR 104: ユーザ作成API呼び出し時に、アクセストークンを渡すように修正
## 概要
[Task1745: ユーザ作成API呼び出し時に、アクセストークンを渡すように修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1745)
タスク 1745: ユーザ作成API呼び出し時に、アクセストークンを渡すように修正
ユーザ作成API4（signup）呼び出し時の引数にアクセストークンを追加。
（/app/dictation_client/src/features/user/operations.ts）
## レビューポイント
アクセストークンの渡し方が正しいかどうか
不足している処理はないか
## UIの変更
なし
## 動作確認状況
ローカルでの動作確認にてheader内にauthorizationが追加されたことを確認
## 補足
なし
|
2023-05-15 02:30:32 +00:00 |
|
makabe.t
|
476c810cc3
|
Merged PR 79: 画面実装（ユーザー追加ダイアログ）
## 概要
[Task1596: 画面実装（ユーザー追加ダイアログ）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1596)
- ユーザ一覧画面にユーザ追加ポップアップを追加しました。
## レビューポイント
- 入力エラーチェックは適切か
- Roleを切り替えた際の内容は適切か
- タイピストの選択はデザインのみです。
## UIの変更
- [Task1596](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/Task1596?csf=1&web=1&e=iiTOxd)
## 動作確認状況
- ローカルで確認
|
2023-04-26 00:04:48 +00:00 |
|
saito.k
|
16b7416de0
|
Merged PR 80: 画面実装
## 概要
[Task1618: 画面実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1618)
- login処理が成功した時にデスクトップアプリを起動するように実装
- デスクトップアプリを起動するURLは確認済み
## レビューポイント
- デスクトップアプリを起動するタイミングは問題ないか
- 実装を追加した場所は問題ないか
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認
## 補足
- useEffectの依存関係からloginを削除
- これでAPI呼び出しが複数回行われることは無くなったが再度調査が必要そう
|
2023-04-25 10:18:00 +00:00 |
|
masaaki
|
469eb6542c
|
Merged PR 78: 画面実装（ユーザー一覧）
## 概要
[Task1595: 画面実装（ユーザー一覧）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1595)
- デザイン部門のHTMLをもとに画面レイアウトを作成
- ルーティング処理を実装
- ユーザー一覧取得APIを呼び出してユーザ情報の一覧を取得
- 取得したユーザ情報を画面の一覧に表示
- 多言語対応
## レビュー対象外
- ユーザ追加ボタン押下時の挙動については[Task 1596: 画面実装（ユーザー追加ダイアログ）](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%207-1?workitem=1596)にて実装のため本タスク対象外
## レビューポイント
- ユーザ一覧の実装についてはPBI対象外ですが、ユーザ登録の動作確認をするうえで問題ないかどうか確認おねがいします
## 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/Task1595?csf=1&web=1&e=236oE5)
## 動作確認状況
- ローカル環境で確認済
## 補足
- 初実装なので、各コンポーネント（state,operation,selectors...）の使い方が誤っていないかも見ていただきたいです
|
2023-04-25 00:29:41 +00:00 |
|
makabe.t
|
a2a0778dfb
|
Merged PR 81: 画面実装（ユーザー認証画面/認証完了画面）
## 概要
[Task1597: 画面実装（ユーザー認証画面/認証完了画面）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1597)
- アカウントへのユーザ追加でメール認証URLから認証を実行した際の画面を実装しました。
- 完了後の画面はアカウント登録のものをそのまま流用しています。
## レビューポイント
- 認証APIからのレスポンスはアカウント登録と同様のものを想定して、完了画面をそのまま流用していますが問題ないでしょうか。
- 画面のパスを`/mail-confirm/user`としましたが問題ないでしょうか？
## UIの変更
- アカウント登録の認証完了画面と同様
## 動作確認状況
- ローカルで確認
- 認証APIでアカウント登録と同様のレスポンスを想定
|
2023-04-19 00:52:47 +00:00 |
|
makabe.t
|
33509fb228
|
Merged PR 69: 画面実装（スナックバー）
## 概要
[Task1506: 画面実装（スナックバー）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1506)
- スナックバーを実装し、アカウント登録のエラー時にエラースナックバーを表示するようにしました。
## レビューポイント
- スナックバーの構成に問題はないか
- 別タスクのコードが混ざっているので、スナックバー実装周りのご確認をお願いします。
- component/snackbar
- App.tsx
- features/ui
- features/signup/operations
## UIの変更
- [Tack1506](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/Task1506?csf=1&web=1&e=xina6Q)
## 動作確認状況
- ローカルで確認
|
2023-04-10 10:34:59 +00: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 |
|
makabe.t
|
444cfda3b2
|
Merged PR 61: 画面実装（入力確認画面＆登録完了画面）
## 概要
[Task1466: 画面実装（入力確認画面＆登録完了画面）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1466)
- アカウント登録の確認画面と完了画面を実装しました。
## レビューポイント
- ページコンポーネントの配置、遷移に問題はないか
- API呼び出しは適切か
- エラー表示はスナックバー実装時にやる想定です。
- デザイン反映は適切か
## UIの変更
- Before/Afterのスクショなど
- [Task1466](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/Task1466?csf=1&web=1&e=DGLJXs)
## 動作確認状況
- ローカルで確認
|
2023-04-06 09:45:59 +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 |
|
makabe.t
|
8822ddaee4
|
Merged PR 6: タスク 1484: 言語切り替えの仕組みをいれる
## 概要
[タスク 1484: 言語切り替えの仕組みをいれる](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_workitems/edit/1484)
- トップ画面での言語切り替え機能を実装しました。
- 英語、ドイツ語、フランス語、スペイン語で切り替えできるようにしています。
## レビューポイント
- 言語切り替えとして機能に不足はないか
- デザインは仮組なので対象外
- コンボボックスで言語切り替えできるところのみ確認をお願いします。
## UIの変更
- 言語切り替え追加
- [Task1484](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/Task1484?csf=1&web=1&e=e3lu7p)
## 動作確認状況
- 画面上で言語切り替えできることを確認
## 補足
- デザインはタグだけの仮組ですので無視してください。
|
2023-03-09 08:46:31 +00:00 |
|
斎藤 快斗
|
a292eee67d
|
pathを変更
|
2023-03-08 11:40:36 +09:00 |
|
斎藤 快斗
|
80d96110ca
|
dev環境のbaseパスを修正
|
2023-03-08 11:19:34 +09:00 |
|
斎藤 快斗
|
edae0ea939
|
develop用の環境変数を修正
|
2023-03-08 10:52:30 +09:00 |
|
斎藤 快斗
|
588e60b762
|
競合解決ミスを修正
|
2023-03-08 10:02:36 +09:00 |
|
saito.k
|
4ce2bbf823
|
Merged PR 5: タスク 1471: 画面実装（トークン系）
## 概要
[Task: 1471](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%203-2?workitem=1471)
- アクセストークンの自動更新処理を実装
- UpdateTokenTimerで定期実行を行う
- 未ログインまたはトークンが期限切れの状態で、ログイン後の画面にアクセスした場合、Topページにリダイレクトする処理を実装
- RouteAuthGuard.tsx
- APIからのレスポンスが401だった時にTopページにリダイレクトする処理を実装
- App.tsx
## レビューポイント
- 今の実装だとトークンの自動更新に失敗した場合、画面上では何も起こらないようにになっている
- 更新が失敗し続け、アクセストークンが切れた段階でRouteAuthGuardではじかれてTopへリダイレクトする
- トークンの期限を確認する間隔を3分にしているが問題なさそうか
## UIの変更
-
## 動作確認状況
- ローカルで動作確認
## 補足
|
2023-03-08 00:57:55 +00:00 |
|
saito.k
|
a1ddc64d2b
|
Merged PR 2: タスク 1361: 画面実装（ログインページ/ログイン済みページ/エラーページ）
## 概要
[Task: 1361](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%203-2?workitem=1361)
- B2Cログイン後、IDトークンを引数にログインAPIを呼ぶ
- ログインAPIレスポンスのアクセストークン・リフレッシュトークンをlocalStorage/storeにセット
- ログインAPIが成否でページ遷移先を変更する
## レビューポイント
- IDトークンの取得方法は下記リンクを参考にした
- https://github.com/AzureAD/microsoft-authentication-library-for-js/blob/dev/lib/msal-react/FAQ.md#how-do-i-handle-the-redirect-flow-in-a-react-app
## 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/Task1361?csf=1&web=1&e=Fo5NQZ
## 動作確認状況
- ローカルで動作確認
## 補足
- WIPです。
- 以下の実装は行っていません。
- アクセストークンの更新処理
- トークンの期限が切れていた場合、Topページにリダイレクトする処理
|
2023-03-08 00:18:38 +00:00 |
|
