makabe.t
|
cbf7622909
|
Merged PR 675: File Propertyの「priority」をHighかNormalで表示する
## 概要
[Task3433: File Propertyの「priority」をHighかNormalで表示する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3433)
- FilePropertyポップアップのpriorityの項目の表示を数字からHigh/Normalとなるように修正しました。
## レビューポイント
- 表示内容は認識通りでしょうか？
## UIの変更
- [Task3433](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/Task3433?csf=1&web=1&e=5iWTtn)
## 動作確認状況
- ローカルで確認
|
2024-01-10 06:12:15 +00:00 |
|
maruyama.t
|
afa05f381c
|
Merged PR 628: 第５階層で規約同意画面を表示する際に、DPAに同意が一瞬表示される
## 概要
[Task3281: 【Commit諸作業終わったら優先対応】第５階層で規約同意画面を表示する際に、DPAに同意が一瞬表示される](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3281)
selectorから別途tierの値が入っているかをselectIsTierLoadingで取得し、まだ入っていない場合(tier===0)Loadingの表示を行うようにした。
```
export const selectIsTierLoading = (state: RootState) =>
state.terms.domain.tier === 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/Task3281?csf=1&web=1&e=D9t5di
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-12-18 04:07:03 +00:00 |
|
maruyama.t
|
94f34a0fde
|
Merged PR 606: 画面修正（Terms画面）
## 概要
[Task3210: 画面修正（Terms画面）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3210)
[Task3211:API修正（バージョン取得API）](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%2023-1?workitem=3211)
[Task3212:API修正（バージョン更新API））](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%2023-1?workitem=3212)
- このPull Requestでの対象/対象外
Click here to read the terms of use.の文言は多言語対応の対象のため、現在一律同じ文言がでます。
第一～第四階層は
上からEULA,PrivacyNotice,DPAが表示されています
第五階層は、
上から、PrivacyNotice,DPAが表示されています
- 影響範囲（他の機能にも影響があるか）
ユーザアーカイブテーブルにPrivacyNoticeのバージョンを追加
## レビューポイント
同意済みプライバシーポリシーはユーザーアーカイブの対象だと認識しているが正しいか。
termsテーブルのdocument_typeの値をPrivacyNoticeにしているが、PRIVACY_NOTICEにしたほうがよいか。
ユニットテストに不足はないか。
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
https://ndstokyo.sharepoint.com/sites/Piranha/Shared%20Documents/Forms/AllItems.aspx?csf=1&web=1&e=hzPw9b&cid=7737ed1b%2D0eb4%2D4331%2Da238%2D14dd35b27e18&FolderCTID=0x012000C0DCEE65AC2177479C3C761CD137C9C9&id=%2Fsites%2FPiranha%2FShared%20Documents%2FGeneral%2FOMDS%2F%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%2FTask3210&viewid=786a81cf%2Dd15f%2D4dc2%2D9e55%2Dc7a729fbc72f
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-12-05 09:10:49 +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 |
|
水本 祐希
|
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 |
|