saito.k
|
a59cb0fffb
|
Merged PR 700: [テストFB対応]User画面でライセンスのStatusの表示を変更したい
## 概要
[Task3506: テスト対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3506)
- ライセンスが割り当たっている状態の表示をLicense Assignedにする
- ライセンスの期限切れ状態の表示をNo Licenseとする
- ヘッダーのOMDSCloudの表記を削除
## レビューポイント
- 修正内容に不足はないか
- 修正の認識ずれはないか
- ほかに修正が影響している箇所はないか
## 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/Task3506?csf=1&web=1&e=g2Hkr3
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2024-01-23 11:02:50 +00:00 |
|
saito.k
|
c4efaf1a1a
|
Merged PR 671: 「LicenseAlert」を消す対応
## 概要
[Task3381: 対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3381)
- 以下の画面から「LicenseAlert」を削除
- ユーザー追加Popup
- ユーザー更新Popup
- ユーザー一覧画面
- 以下のAPIの引数から「LicenseAlert」の設定をするパラメータを削除
- ユーザー作成API(users/signup)
- ユーザー更新API(users/update)
- ユーザー一覧取得APIのレスポンスから「LicenseAlert」を削除
- usersEntityから「LicenseAlert」を削除
- クライアントのAPI呼び出し部分を自動生成するopenapi-generator-cliのバージョンを7.1.0で固定
- 最新バージョン(7.2.0)はaxiosのバージョン1.x用になるため
- https://github.com/OpenAPITools/openapi-generator/releases/tag/v7.2.0
## レビューポイント
- openapi-generator-cliのバージョンを固定するのではなく、axiosのバージョンを上げたほうがよいか
- axiosのバージョン1.xが出たのがおととしになるのでそろそろ上げてもいい?
- このタイミングでメジャーバージョンを上げるのはどうか
- 基本的な仕様について大きくは変わってない模様
- 実際に上げてみて軽く動作確認してみたが問題なかった
## 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/Task3381?csf=1&web=1&e=TpXHES
## 補足
- 相談、参考資料などがあれば
|
2024-01-11 06:58:01 +00:00 |
|
makabe.t
|
a0da277c05
|
Merged PR 607: メールアドレス入力チェックの修正
## 概要
[Task3232: メールアドレス入力チェックの修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3232)
- メールアドレス入力に対して規約に則ったチェックを実施するように修正しました。
- 画面での入力チェックとAPIのバリデーションを修正しています。
## レビューポイント
- クライアントのチェック内容は規約に則った内容になっているでしょうか?
- 正規表現の構文として不自然な点はないでしょうか?
- サーバー側のチェックをIsEmail+禁止文字の追加という形で対応していますが適切でしょうか?
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-12-08 02:14:25 +00:00 |
|
saito.k
|
deb3431d74
|
Merged PR 531: 画面実装(代行操作)
## 概要
[Task2909: 画面実装(代行操作)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2909)
- ディーラーユーザーが代行操作用トークンを取得して、第五階層ユーザーの代行操作ができる実装をしました。
- 代行後に各対象タブが代行操作デザインで表示され、代行対象として操作できるようになっています。
- 各APIの呼び出しについて代行操作用トークンがあればそちらを使うように実装しています。
- 代行操作のタブ、ページ移動時に代行操作を維持するために遷移をリンクから`useNavigate`に変更しました。
## レビューポイント
- 代行操作用トークンの取り扱いについて
- APIからのトークン取得後、`store.auth`に代行操作用トークンを保存し、利用時には関数を使って間接的に呼でいますが構成として不自然な点はないでしょうか?
- トークン取得関数では代行操作用トークンがあればそれを、なければ通常のトークンを取得するようにしています。
- これはAPIのトークンを設定する際にトークンを取得側では代行操作中か意識せずに一つの関数を呼べばいいようにするためです。
- 代行操作用トークンの保存は通常のトークン保存と同様に`operation`から`Slice`に設定したSet関数を呼ぶことでstateに保存していますが、使い方として気になることはないでしょうか?
※代行操作中に表示するタブの制限と代行操作トークンの更新処理は別タスクでの実施予定です。
## UIの変更
- [Task2909](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/Task2909?csf=1&web=1&e=TnwgOb)
## 動作確認状況
- ローカルで確認
|
2023-11-06 08:28:40 +00:00 |
|
水本 祐希
|
9323cd02e4
|
Merged PR 506: AUTHOR_IDを大文字に自動変換してあげる
## 概要
[Task2376: AUTHOR_IDを大文字に自動変換してあげる](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2376)
- 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず)
- 何をどう変更したか、追加したライブラリなど
AUTHOR_IDを大文字に自動変換するよう実装
入力時に大文字に変換するようにしています。
- このPull Requestでの対象/対象外
- 影響範囲(他の機能にも影響があるか)
## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-10-19 06:50:32 +00:00 |
|
水本 祐希
|
3af64fbf8e
|
Merged PR 464: データ取得失敗時に各一覧表示画面の表示がそろっていない
## 概要
[Task2720: データ取得失敗時に各一覧表示画面の表示がそろっていない](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2720)
- 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず)
- 何をどう変更したか、追加したライブラリなど
- 各一覧表示画面で値0件およびデータ取得エラー時に表示する挙動を統一
※Accountに関しては自アカウント情報の表示のため値0件はないとみて、特に本タスクの実装
は必要ないと判断しました。
→対応としてはデータ取得エラー時のスナックバー表示のみ
※Userも自アカウント情報が表示されるため上記と同じ実装にする予定でしたが、スナックバーと0件表示ができました。

・〇はそれぞれの検証条件をクリアできているという意味です。
・「データ取得エラーとする」は通常ではAccount、License、Userは値が0件となることはなく、データベースエラーの場合のみと考えたので、値が0件=データ取得エラーとしました。
・「スナックバー表示のみ」はデータ取得エラーが発生した場合、スナックバー表示のみで対応するという意味です。
・「表示できない」はデータベースを切った状態だと、子アカウントのorderhistoryが表示されないため、表示できないと記載しました。
- dictationSlice.tsでbuilder.addCase(getSortColumnAsync.rejected, (state) => {
state.apps.isLoading = false;
を実装したのは、修正前はデータベースを切った状態だとロードのぐるぐるが消えずにいました。原因はgetSortColumnAsyncにrejectの場合、isLordingをfalseにする実装がなかったためです。
以上のことから上記実装を追加しました。
- このPull Requestでの対象/対象外
- 影響範囲(他の機能にも影響があるか)
## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
## 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/Task2720?csf=1&web=1&e=MBNgO8
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-10-10 06:40:15 +00:00 |
|
masaaki
|
177c8beb41
|
Merged PR 344: 画面実装(ユーザー一覧画面修正)
## 概要
[Task2449: 画面実装(ユーザー一覧画面修正)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2449)
- 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず)
- 何をどう変更したか、追加したライブラリなど
- ユーザー一覧画面を修正し、License Deallocation押下時にライセンス割り当て解除APIを呼び出す処理を追加しました
- このPull Requestでの対象/対象外
- すべて対象になります
- 影響範囲(他の機能にも影響があるか)
- 無し
## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
- 特筆すべきところはありません。
## UIの変更
- 変更なし
## 動作確認状況
- ローカルで確認
## 補足
- 特になし
|
2023-08-24 07:19:08 +00:00 |
|
saito.k
|
fe7581cf10
|
暗号化パスワードの初期値をundefindに修正
|
2023-08-24 12:06:43 +09:00 |
|
masaaki
|
6f92ef9453
|
Merged PR 319: 画面実装(ライセンス割り当てポップアップ)
## 概要
[Task2358: 画面実装(ライセンス割り当てポップアップ)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2358)
- 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず)
- 何をどう変更したか、追加したライブラリなど
- ライセンス割り当てポップアップを新規に追加しました
- ユーザー一覧画面に対して、対象のユーザ情報をstateに格納してライセンス割り当てポップアップを呼び出す処理を追加しました
- このPull Requestでの対象/対象外
- 対象外はありません
- 影響範囲(他の機能にも影響があるか)
- 特にありません
## レビューポイント
- 特にレビューしてほしい箇所
- selectors.tsのselectAllocatableLicensesについて、
UTC変換を伴う時刻算出を行っています。その中で共通関数化やサブ関数化を行っていますが、外だしが妥当か確認いただきたいです。
## 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/Task2358?csf=1&web=1&e=yDndMf
## 動作確認状況
- ローカルで動作確認済
## 補足
- 相談、参考資料などがあれば
|
2023-08-21 07:55:13 +00:00 |
|
saito.k
|
ee783585ae
|
Merged PR 316: dev動作確認で発見した修正対象
## 概要
[Task2368: dev動作確認で発見した修正対象](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2368)
- デザイン周り
- ラジオボタン・チェックボックスのラべルを押下しても状態を変更できるように修正
- 機能
- もともとパスワードがあったとき、フォーカスを当てて伏字がなくなった状態になった場合は、値をチェックするように修正
- Encryption:ON→パスワード入力→Encryption:OFFにしてユーザー追加したとき、入力していたパスワードがServerに送られてしまっていたので、Encryption:OFFにしたらパスワードを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/Task2368?csf=1&web=1&e=pyfaXd
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-08-08 09:43:04 +00:00 |
|
makabe.t
|
ef83477533
|
Merged PR 305: ユーザー編集ポップアップ実装
## 概要
[Task2315: ユーザー編集ポップアップ実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2315)
- ユーザー編集ポップアップを実装しました。
## レビューポイント
- デザイン判定は適切か
- 表示文言は適切か
- エラーチェックは適切か
## UIの変更
- [Task2315](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/Task2315?csf=1&web=1&e=AP7b9M)
## 動作確認状況
- ローカルで確認
|
2023-08-08 02:48:40 +00:00 |
|
saito.k
|
4565d59a51
|
Merged PR 301: ユーザー追加修正(API/画面)
## 概要
[Task2327: ユーザー追加修正(API/画面)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2327)
- ユーザー追加API
- リクエストにencryption,encryptionPassword,promptを追加
- リクエストからtypistGroupIdを削除
- ロールに応じてDBに保存するデータを作成する処理を追加
- リクエストパラメータのバリデーションチェックを追加
- ユーザー追加画面
- TypistGroupの選択欄を削除
- RoleがAuthorの場合、encryption,encryptionPassword,promptを追加
## レビューポイント
- 修正に不足はないか
- 画面のユーザー追加処理を引数を渡さずに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/Task2327?csf=1&web=1&e=uvTYlb
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-08-07 00:11:54 +00:00 |
|
saito.k
|
6a2d2eabde
|
Merged PR 298: ユーザー一覧修正(API・画面)
## 概要
[Task2318: ユーザー追加・一覧修正(API・画面)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2318)
- API
- ユーザー一覧取得APIのレスポンスにUserIDを追加
- OpenAPI生成
- 画面
- コード自動生成
- storeにidを含めたユーザー一覧情報を格納する
## レビューポイント
- 修正内容に認識違いはないか
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認
## 補足
- ユーザー追加については別タスクで対応
|
2023-08-03 00:51:03 +00:00 |
|
saito.k
|
c7d34e1ccb
|
Merged PR 281: ユーザー一覧画面修正
## 概要
[Task2232: ユーザー一覧画面修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2232)
- ユーザー一覧の画面修正
- 新しい項目を追加
- Encryption
- Prompt
- EmailVerified
- ユーザーのRoleに応じて表示内容を切り替える
- author
- Typistgroupをハイフン表示
- typist
- AuthorIDをハイフン表示
- Encryptionをハイフン表示
- Promptをハイフン表示
- Statusに応じて表示を変更
- Alert
- Status, Expiration , Remainingを赤文字表示
- NoLicense
- Statusを赤文字表示
- ログインユーザーのTierによる表示変更
- 行をマウスオーバーすると出てくるボタンの表示非表示
## レビューポイント
- Selectorで画面表示する内容に変換して取得しているが問題ないか
- 表示する内容、条件に漏れはないか
## 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/Task2232?csf=1&web=1&e=nX7ayK
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-08-01 01:54:38 +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 |
|
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 |
|
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
|
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 |
|
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 |
|