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: ｘｘｘのサンプルページを消し去る
## 概要
[Task3055: ｘｘｘのサンプルページを消し去る](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 |
|
makabe.t
|
ef4f22029b
|
Merged PR 551: 画面実装（代行操作中のヘッダー表示）
## 概要
[Task2911: 画面実装（代行操作中のヘッダー表示）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2911)
- 代行操作中に表示されるヘッダのタブを以下のタブのみに設定しました。
- User
- Workflow
- License
## レビューポイント
- 代行操作タブ表示の判断材料としてヘッダコンポーネント内でselectorで代行操作の有無を判断してタブ表示用メソッドに渡していますが、データの取り扱いとして不自然なところはないでしょうか？
## UIの変更
- [Task2911](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/Task2911?csf=1&web=1&e=zFW1zj)
## 動作確認状況
- ローカルで確認
|
2023-11-07 06:25:42 +00:00 |
|
makabe.t
|
d0f4971dc9
|
Merged PR 544: ヘッダの表示文言（タブ・タイトル）を翻訳ファイルを参照するように修正する
## 概要
[Task2895: ヘッダの表示文言（タブ・タイトル）を翻訳ファイルを参照するように修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2895)
- ヘッダに表示するタイトルとタブの文言を翻訳ファイルに定義したラベルを利用するように修正しました。
## レビューポイント
- タブのフィルタ管理用に`key`プロパティを追加したが、構成として適切か
- 表示項目の制御には従来通りの定数を利用したいのでkeyプロパティとして設定
- labelプロパティにタブに表示する文言の翻訳文言を参照するように設定
- 対応ラベルの箇所は適切か（各タブ名＋タイトル名）
## UIの変更
- [Task2895](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/Task2895?csf=1&web=1&e=Xs3Tac)
## 動作確認状況
- ローカルで確認
|
2023-11-02 09:41:43 +00:00 |
|
水本 祐希
|
a4dd5addde
|
Merged PR 507: 第五のユーザでログインした際、アカウントタブが表示される
## 概要
[Task2886: 第五のユーザでログインした際、アカウントタブが表示される](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2886)
- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
一般ユーザーがログイン時にアカウントタブを表示しないようにする。
- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）
## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-10-23 07:15:53 +00:00 |
|
makabe.t
|
84e7deb52a
|
Merged PR 343: 画面実装（TypistGroup設定画面）
## 概要
[Task2411: 画面実装（TypistGroup設定画面）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2411)
- タイピストグループの設定画面を実装しました。
- タイピストグループの一覧を取得できるようにしています。
- return ボタンでワークフロー画面に遷移します。
- Edit、Add Groupは対象外です。
- ヘッダタブ表示のため、ヘッダを修正しています。
## レビューポイント
- パスの構成は問題ないか
- ヘッダの対応に問題はないか
## UIの変更
- [Task2411](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/Task2411?csf=1&web=1&e=cxxeCf)
## 動作確認状況
- ローカルで確認
|
2023-08-23 08:44:01 +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 |
|
oura.a
|
fabcdc16f5
|
Merged PR 223: ヘッダ修正（partnerタブ追加）
## 概要
[Task2154: ヘッダ修正（partnerタブ追加）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2154)
- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
・Partnerヘッダータグを追加
- この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/Task2154?csf=1&web=1&e=jv7CDb
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-07-18 06:13:23 +00:00 |
|
masaaki
|
9c0f457e9f
|
Merged PR 220: [12-2]RoleGuradにTierに対するチェックを実装する
## 概要
[Task1951: [12-2]RoleGuradにTierに対するチェックを実装する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1951)
- 階層の指定について宣言的にできるように対応しました。
- 階層の定数について配列化しました。
（分かりやすさとroleguards側の実装しやすさのため）
- 使用していない宣言がいくつか見られたので、気づいた範囲で削除しました。
- 影響範囲（他の機能にも影響があるか）
- これまで処理内で階層のチェックを行っていた箇所について、宣言的にチェックするよう修正しました。
修正対象の洗い出しについては「補足」参照
- 階層のみチェックする場合を考慮し、既存のrolesに対するチェックを任意指定にしています。
これに伴い、rolesが指定されなかった場合を考慮して修正を行っています。
## レビューポイント
- 各コントローラを見ていただき、階層チェックのやり方について使いづらさがないか
## UIの変更
- 無し
## 動作確認状況
- ローカルで確認済
## 補足
- 修正対象の洗い出しは以下の通り実施しています。
https://ndstokyo.sharepoint.com/:u:/r/sites/Piranha/Shared%20Documents/General/OMDS/%E3%83%A9%E3%83%95%E3%82%B9%E3%82%B1%E3%83%83%E3%83%81/PBI1189_%5B%E9%9A%8E%E5%B1%A4%E5%91%A8%E3%82%8A%E6%95%B4%E7%90%86%5D%E7%AC%AC%E4%B8%80%EF%BD%9E%E7%AC%AC%E5%9B%9B%E9%9A%8E%E5%B1%A4%E3%81%A8%E3%81%97%E3%81%A6%E3%80%81%E3%83%91%E3%83%BC%E3%83%88%E3%83%8A%E3%83%BC%E5%90%91%E3%81%91Web%E3%82%B5%E3%83%BC%E3%83%93%E3%82%B9%E3%81%AB%E3%83%AD%E3%82%B0%E3%82%A4%E3%83%B3%E3%81%97%E3%81%9F%E3%81%84.drawio?csf=1&web=1&e=h3Sbf6
|
2023-07-12 02:06:16 +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 |
|
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 |
|
