masaaki
|
cca9cf6e5e
|
Merged PR 601: ヘルプページ画面実装
## 概要
[Task3204: ヘッダ修正対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3204)
- ヘルプページを追加して、ヘッダのタブに表示されるようにしました。
- ヘッダのタブはロールの制限なく見えるようにしています。
- ヘルプページからさらに個別のヘルプへのリンクは仮のURLにしています。
- 動作としては別タブで開くようにしています。
## レビューポイント
- パスをタブ名に合わせて`/support`としましたが不自然な点はないでしょうか?
- 仮のURLがxxxとなっているため、別タブで開かれたページがNot Foundとなってしまっていますが問題ないでしょうか?
## UIの変更
- [Task3204](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/Task3204?csf=1&web=1&e=N9yn0D)
## 動作確認状況
- ローカルで確認
|
2023-12-05 02:40:09 +00:00 |
|
saito.k
|
4b0110856c
|
Merged PR 566: xxxのサンプルページを消し去る
## 概要
[Task3055: xxxのサンプルページを消し去る](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3055)
- 仮で置いてあったサンプルページ削除
- タブからも削除
- ルーティングからも削除
- ログイン成功時、階層とロールに応じて遷移する画面を変更する処理を追加
## レビューポイント
- 遷移先を変更する処理は関数に切り出した方が良いか?
- ほかで使わないからべた書きでも良いかなと思い、こうしました。
## UIの変更
- Before/Afterのスクショなど
- 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/Task3055?csf=1&web=1&e=B0E9Ud
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-11-10 02:22:34 +00:00 |
|
水本 祐希
|
7196491cf0
|
Merged PR 472: 画面作成(利用規約同意画面)
## 概要
[Task2802: 画面作成(利用規約同意画面)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2802)
- 何をどう変更したか、追加したライブラリなど
- 利用規約同意画面の実装を行いました
- このPull Requestでの対象/対象外
- api.tsおよびstyles
- 影響範囲(他の機能にも影響があるか)
- ありません
## レビューポイント
- 特にレビューしてほしい箇所
- URLの妥当性(動作確認のため別タスクで追加していますが、内容は本タスクで見てほしいです)
違和感ないか確認お願いします。
<Route path="/accept-to-use" element={<AcceptToUsePage />} />
- 各処理のエラーハンドリングについて
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
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/Task2802?csf=1&web=1&e=otF5YX
## 動作確認状況
- ローカルで確認済
## 補足
- 相談、参考資料などがあれば
|
2023-10-17 07:15:49 +00:00 |
|
masaaki
|
897bad289b
|
Merged PR 480: 画面修正(ログイン画面)
## 概要
[Task2801: 画面修正(ログイン画面)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2801)
- 以下の修正を実施しました
- ログイン画面について、未同意バージョンがある場合、利用規約同意画面に遷移する処理を実装
- 利用規約同意画面(ADB2C以外の画面)からログイン画面に遷移した際も処理継続できるよう対応を実施
- このPull Requestでの対象/対象外
- AcceptToUsePageについては、遷移確認用のダミーページなので対象外でお願いします。
- 影響範囲(他の機能にも影響があるか)
- ありません。
## レビューポイント
- 特にレビューしてほしい箇所
1. 既存のLoginPageを以下のように分割しています。
実装内容のイメージあっているか確認お願いします。
- LoginPage→AADB2Cからのリダイレクトを元にLocalStorageアクセス用のキーを生成
- TokenSettingPage→LocalStorageアクセス用のキーを使用してidTokenを取得し各種token生成を実施
1. TokenSettingPage/index.tsxにて、型ガード(isErrorObject)を作成し使用しています。
使い方やガードの実装が妥当か確認お願いします。
## UIの変更
- 無し
## 動作確認状況
- ローカルで確認を実施
## 補足
- 相談、参考資料などがあれば
|
2023-10-16 06:52:08 +00:00 |
|
水本 祐希
|
5cfe069b58
|
Merged PR 444: 画面実装(アカウント削除成功ページ)
## 概要
[Task2715: 画面実装(アカウント削除成功ページ)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2715)
- 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず)
- 何をどう変更したか、追加したライブラリなど
・アカウント削除成功後に、アカウント削除成功ページに遷移するよう実装
・ログオフ状態にする
- このPull Requestでの対象/対象外
- 影響範囲(他の機能にも影響があるか)
## レビューポイント
- 特にレビューしてほしい箇所
・レイアウトやメッセージの文言
・ログオフ状態にするタイミング
→Back to TOP Pageを押下時にログオフ状態になるようにしています
・Delete Accountボタン(ポップアップ上の)押下時のページ遷移のコード
・その他、漏れがないか
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
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/Task2715?csf=1&web=1&e=0M85t6
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-10-02 01:19:18 +00:00 |
|
saito.k
|
3f4d4ec436
|
Merged PR 427: 画面実装(テンプレートファイル一覧画面)
## 概要
[Task2651: 画面実装(テンプレートファイル一覧画面)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2651)
- テンプレートファイル一覧画面を実装
## レビューポイント
- 取得方法やstoreの構成は問題ないか
## 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/Task2651?csf=1&web=1&e=MAaOJd
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-09-22 07:36:23 +00:00 |
|
oura.a
|
2705fcee0a
|
Merged PR 406: 画面実装(アカウント情報画面)
## 概要
[Task2605: 画面実装(アカウント情報画面)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2605)
アカウント情報画面を実装しました。
※以下の処理は本PBIの対象外となりますので、レビュー対象外でお願いします
・「File Delete Setting」ボタン押下時の処理
・「Delete Account」ボタン押下時の処理
## レビューポイント
もっと簡潔に記載できる箇所はないか
## 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/Task2605?csf=1&web=1&e=p0a4EP
## 動作確認状況
ローカルで動作確認済み
## 補足
なし
|
2023-09-19 10:01:19 +00:00 |
|
saito.k
|
7a453c80f8
|
Merged PR 372: 画面実装(WorktypeID設定画面)
## 概要
[Task2506: 画面実装(WorktypeID設定画面)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2506)
- WorktypeID設定画面を実装
- WorktypeID一覧を表示する
- 各種ボタンを表示する
- ActiveWorktypeIDのセレクトボックスを表示する
- 言語切り替えの対応で同一の文言をcommonにまとめ、該当箇所を修正
## レビューポイント
- stateのdomain配下にあるAPIからの戻り値をOptionalにしたが問題ないか
- 必須にすると画面初期表示時に「0件です」がちらつくため
## 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/Task2506?csf=1&web=1&e=IHIC1Y
## 動作確認状況
- ローカルで確認
## 補足
- Returnボタン以外のボタンの挙動はレビュー対象外
- Active WorkTypeIDの挙動はレビュー対象外
|
2023-09-01 07:27:36 +00:00 |
|
makabe.t
|
c85ecbfc90
|
Merged PR 341: 画面実装(ルーティングルール一覧画面)
## 概要
[Task2410: 画面実装(ルーティングルール一覧画面)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2410)
- ワークフロー画面と遷移先のタイピストグループ設定画面を実装しました。
- ワークフロー画面はタブ制御とリンク以外は仮です。
- 管理者ユーザーでログインした時だけタブが見えるようにしています。
- タイピストグループ設定画面は空の仮画面です。
- 画面コンポーネントをルーティングしました。
- /workflow
- /workflow/typist-group
## レビューポイント
- タブ制御は適切か
- URL設計は適切か
## UIの変更
- [Task2410](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/Task2410?csf=1&web=1&e=10GUjM)
## 動作確認状況
- ローカルで確認
|
2023-08-23 02:32:36 +00:00 |
|
maruyama.t
|
8be20b7ca8
|
Merged PR 269: 画面実装(注文履歴画面)
## 概要
[Task2260: 画面実装(注文履歴画面)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2260)
- 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず)
- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
Order Cancel/Issue/Issue Cancel機能について、はこちらで実装予定[注文履歴画面_画面遷移対応](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%2014-1?workitem=2273)
Return機能の挙動
userSlice.tsの修正部分(openapi.jsonからapi.ts生成時に画面側のDomainが原因でエラーになるため、一時的に修正)
- 影響範囲(他の機能にも影響があるか)
## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
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/Task2260?csf=1&web=1&e=PBcZUw
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-08-01 01:35:19 +00:00 |
|
水本 祐希
|
6e3c9d5c81
|
Merged PR 244: Partner一覧仮画面
## 概要
[Task2229: Partner一覧仮画面](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_workitems/edit/2229/)
- 何をどう変更したか、追加したライブラリなど
・Partner一覧の仮画面を作成しました。
ボタン追加とアカウント階層の画面表示
- このPull Requestでの対象/対象外
・仮画面なので、レイアウトに関しては対象外です。
・46行目にonClickをコメントアウトしています。
後々使用すると思い、わざと消していないですが、レビューで消すか否かを
コメントいただければと思います。
## レビューポイント
- 特にレビューしてほしい箇所
・ヘッダのpartnerタブ押下により画面が表示できること
・第1~3階層で「Add Account」ボタンが表示されるか
・第4階層で「Add Account」ボタンが非表示かどうか
・第1~4階層で正しく階層が表示されるか
例:第1階層でログイン→Tier:1と表示
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
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/Task2155?csf=1&web=1&e=d1MRi8
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-07-19 00:43:09 +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 |
|
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 |
|
masaaki
|
785535e4e6
|
Merged PR 125: licenseのAppRouterについて、タスク1791対応後にRouteAuthGuardの設定を入れる
## 概要
[Task1789: licenseのAppRouterについて、タスク1791対応後にRouteAuthGuardの設定を入れる](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1789)
- licenseページについてRouteAuthGuardを実装しました
※「バグ 1791: 言語情報を切り替えるとログインできない」の影響で多言語対応の確認ができないので、一時的にRouteAuthGuardを外していたのを、バグ解消により修正しました。
## レビューポイント
- 修正内容に問題ないか
## UIの変更
- 特になし
## 動作確認状況
- ローカルで確認済
## 補足
- 特になし
|
2023-06-02 04:26:08 +00:00 |
|
masaaki
|
713d587abf
|
Merged PR 112: 画面実装(ライセンス情報)
## 概要
[Task1683: 画面実装(ライセンス情報)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1683)
- ライセンス情報画面を仮実装。ライセンス注文を起動するためのボタンを用意しました。
- 画面デザイン上に登場する文言については、多言語対応を実施しています。
- このPull Requestで対象外
- 細かな画面の実装部分は別PBIのため対象外
- ライセンス注文のボタン実装については、「タスク 1684: 画面実装(ライセンス注文ポップアップ)」で実装するため対象外
- AppRouter.tsxの定義について、本来、ログイン後の画面のため、RouteAuthGuardの設定を行う必要がありますが、タスク1786の事象により多言語でのログインが出来なく、多言語対応の確認ができないため一時的に外しています。タスク1789で対応予定です。(ソースコメントにも記載)
## レビューポイント
- 特にありません
## 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/Task1683?csf=1&web=1&e=agPOmO
## 動作確認状況
- ローカルで確認済
## 補足
- 無し
|
2023-05-22 01:11:00 +00:00 |
|
masaaki
|
c5ebd6ec50
|
Merged PR 106: ユーザ一覧のurlを修正
## 概要
[Task1746: ユーザ一覧のurlを修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1746)
- ユーザ一覧のURLについて、userListとしていましたが、ヘッダコンポーネントのタブ押下時に呼び出すURL(user)に合わせて修正を行いました。
## レビューポイント
- AppRouter.tsxの記載を見たところ、仮でのuserページの定義上は「RouteAuthGuard」という記載がありました。現状は、初回作成時のまま「RouteAuthGuard」を指定しない定義としていますが、問題ないか確認をお願いします。
## UIの変更
- 無し
## 動作確認状況
- ローカルで確認済
## 補足
- 無し
|
2023-05-15 05:47:07 +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
|
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
|
b4cd0208e6
|
Merged PR 51: タスク 1468: 部品component作成(ヘッダー・フッター)
## 概要
[Task: 1468](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-2?workitem=1468)
- ヘッダーcomponentを作成
- ログイン前とログイン後でヘッダーが異なるので各ページに配置するようにした
- 呼び出すcomponentは一つとして作成し、内部でヘッダーを切り替えるようにした
- フッターcomponentを作成
- ログイン前とログイン後でページのデザインが異なるのでヘッダー同様、各ページに配置することにした
## レビューポイント
- ヘッダーの作成方法に問題はないか
## 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/Task1468?csf=1&web=1&e=K2tFjK
## 動作確認状況
- 型チェック
- ローカルで動作確認
## 補足
|
2023-03-27 12:05:56 +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
|
a69d8b19f3
|
Merged PR 13: タスク 1470: 画面実装(トップページ)
## 概要
[タスク 1470: 画面実装(トップページ)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_workitems/edit/1470)
- トップページのデザインを適用しました。
## レビューポイント
- デザイン適用は適切か
- CSSの使い方に問題はないか
## UIの変更
- トップページのデザイン適用
- [Task1470](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/Task1470?csf=1&web=1&e=BgUfMX)
## 動作確認状況
- ローカルで起動してデザイン確認
|
2023-03-14 07:17:36 +00:00 |
|
makabe.t
|
6bcbabc897
|
Merged PR 7: タスク 1477: 存在しないURLを指定した場合の挙動を実装する
## 概要
[タスク 1477: 存在しないURLを指定した場合の挙動を実装する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_workitems/edit/1477)
- 存在しないURLを指定した場合にNot Foundページへ遷移するようルーティングしました。
- エラーページ自体は仮デザインです。
## レビューポイント
- エラー時の対応として問題ないか
## UIの変更
- エラーページ追加
- [Task1477](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/Task1477?csf=1&web=1&e=nc0yx5)
## 動作確認状況
- 画面を起動して存在しないURLでエラーページに遷移することを確認
|
2023-03-09 08:47:13 +00: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 |
|
斎藤 快斗
|
bfe91c64c9
|
Merged PR 8: タスク 1360: 画面実装(TOPページ)
[Task1360](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-1?workitem=1360)
- Azure AD B2Cのログインを行う実装
- B2Cへの接続設定
- ログイン後のリダイレクト先設定
## レビューポイント
- B2Cへの接続設定で過不足はないか
## UIの変更
- https://ndstokyo.sharepoint.com/:f:/r/sites/Piranha/Shared%20Documents/General/%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/Task1360?csf=1&web=1&e=8bzmdc
## 動作確認状況
- 型チェックだけしています。
## 補足
|
2023-02-24 07:47:15 +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 |
|