makabe.t
|
0a714f8484
|
Merged PR 870: ファイル情報ポップアップ画面修正
## 概要
[Task4051: ファイル情報ポップアップ画面修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/4051)
- ファイル情報ポップアップを修正し、ファイル名を変更できるように修正しました。
- 音声ファイル名変更APIのファイル名の文字数を修正しました。
- フロントの入力欄で64文字にしたので、プラス拡張子で68文字としました。
## レビューポイント
- 画面イメージは認識通りでしょうか?
- 表示では拡張子を取って、APIに渡す際にはつけているのですが処理として不自然ではないでしょうか?
## UIの変更
- [Task4051](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/Task4051?csf=1&web=1&e=nbYIE0)
## クエリの変更
- なし
## 動作確認状況
- ローカルで確認
- 行った修正がデグレを発生させていないことを確認できるか
- 具体的にどのような確認をしたか
- タスク一覧、ファイル情報ポップアップが正常に見えることを確認
|
2024-04-16 10:12:44 +00:00 |
|
makabe.t
|
e6d6e477d9
|
Merged PR 862: パートナー一覧画面&パートナー編集ポップアップ実装
## 概要
[Task3935: パートナー一覧画面&パートナー編集ポップアップ実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3935)
- パートナー一覧画面からパートナー編集ポップアップを表示して情報を変更できる画面実装をしています。
## レビューポイント
- エラーの表示は適切でしょうか?
- 画面イメージは認識通りでしょうか?
## UIの変更
- [Task3935](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/Task3935?csf=1&web=1&e=FdaUMT)
## クエリの変更
- なし
## 動作確認状況
- ローカルで確認
- 行った修正がデグレを発生させていないことを確認できるか
- 新規機能なので問題なし
|
2024-04-08 07:41:05 +00:00 |
|
masaaki
|
eb6b413adb
|
Merged PR 848: パートナー一覧画面修正
## 概要
[Task3833: パートナー一覧画面修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3833)
- パートナー一覧画面のパートナーにカーソルを合わせた際に、Delete Partnerリンクが表示される。
押下した場合は、本当に削除するのかを確認するメッセージが表示され、どのPartnerを削除するかメッセージ欄に表示されること。
## レビューポイント
- 特筆する点はなし
## UIの変更
https://ndstokyo.sharepoint.com/sites/Piranha/Shared%20Documents/Forms/AllItems.aspx?csf=1&web=1&e=hzPw9b&cid=e8e0702d%2D3730%2D4295%2Dbb9d%2D40e6b1998906&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%2FTask3833&viewid=786a81cf%2Dd15f%2D4dc2%2D9e55%2Dc7a729fbc72f
## クエリの変更
なし
## 動作確認状況
- ローカルで確認(APIからの返却値を直接指定する方式で確認)
- 行った修正がデグレを発生させていないことを確認できるか
新規機能追加のみなので問題なし
## 補足
- 相談、参考資料などがあれば
|
2024-03-22 07:50:58 +00:00 |
|
Kentaro Fukunaga
|
83e297cc9b
|
Merged PR 821: 画面実装(パートナーライセンス一覧画面&階層構造変更ポップアップ)
## 概要
[Task3854: 画面実装(パートナーライセンス一覧画面&階層構造変更ポップアップ)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3854)
- パートナーライセンス一覧に「Change Owner」ボタンを配置し、表示制御およびクリック時にポップアップ表示する処理の実装
- アカウント階層構造変更ポップアップの処理全体実装
- サーバー側のエラーコード定義
## レビューポイント
- 「一括」を表現するためのドロップダウンの構築や処理周りで改善点ないか(to:斎藤くん)
- コンポーネントでの状態管理でお作法に違反しているところないか(to:斎藤くん)
- 修正箇所がほかの機能に影響していないか
- パートナーライセンス一覧の画面表示に何らか悪影響ないか?(to:ガンさん)
## 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/Task3854?csf=1&web=1&e=jBGQrR
## クエリの変更
- なし
## 動作確認状況
- ローカルで確認しました
- 第一階層でログインしてかつ第三または第四視点での一覧を確認しているときにChangeOwnerボタンが表示される
- ボタン押下すると、仕様通りにポップアップの表示が行われる
- ポップアップにて入力項目に入力できる&バリデーション効いている
- ポップアップにて実行ボタン押下するとAPI実行できる&処理結果に応じて仕様通りの挙動をすること
- 行った修正がデグレを発生させていないことを確認できるか
- 具体的にどのような確認をしたか
- パートナーライセンス画面に新規ボタンを配置した&新規ポップアップの実装のみのため、
ポップアップでの処理が正常終了/失敗/何もせず閉じた場合に元の画面の表示が今まで通り動くことを確認済み。
|
2024-03-13 07:41:25 +00:00 |
|
makabe.t
|
da40e8f09c
|
Merged PR 800: 画面実装(一括追加ボタン&ポップアップ画面)
## 概要
[Task3753: 画面実装(一括追加ボタン&ポップアップ画面)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3753)
- ユーザー一括登録画面を実装しました。
- 一括登録ポップアップ
- テンプレートCSVダウンロード
- ファイルインポート
- エラー行表示
## レビューポイント
- 行エラーの条件、内容は認識通りでしょうか?
- 画面の表示内容は認識通りでしょうか?
- CSV変換時にworkerを有効にしているとエラーとなりうまくいかないのでOFFにしてしまいましたが問題ないでしょうか?
- @<湯本 開> さん
## UIの変更
- [Task3753](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/Task3753?csf=1&web=1&e=x5M6hr)
## 動作確認状況
- ローカルで確認
- ファイルチェックするところまで
|
2024-03-06 01:58:32 +00:00 |
|
Kentaro Fukunaga
|
5305984b1a
|
Merged PR 764: 第五階層ライセンス情報画面実装
## 概要
[Task3709: 第五階層ライセンス情報画面実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3709)
- ストレージ使用可否切り替えの画面実装をしました
- 動作確認中に、既存実装でライセンスオーダーするときとカードライセンスアクティベートするときの操作不能化処理に漏れがあったのを修正しました
## レビューポイント
- Redux周りの実装でお作法に違反しているところがないか。もしくは改善点ないか。
- ライセンス情報表示のAPI結果待ち部分のローディング処理で、最低限の改善にしたが現時点ではこれでよいか?(いつ修正するかも未定だけど、実害はないためひとまずこんな感じで。。。)
- `licenseSummarySlice.ts` のコメント部分が該当箇所です
## 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/Task3709?csf=1&web=1&e=bJVzss
## 動作確認状況
- ローカルで動作確認しました。
|
2024-02-27 00:01:02 +00:00 |
|
makabe.t
|
bbbd3e757b
|
Merged PR 743: テンプレートファイル削除画面修正
## 概要
[Task3600: テンプレートファイル削除画面修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3600)
- テンプレートファイル削除の画面を実装しました。
## レビューポイント
- エラー処理は適切でしょうか?
## UIの変更
- [Task3600](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/Task3600?csf=1&web=1&e=E25Kf7)
## 動作確認状況
- ローカルで確認
|
2024-02-13 02:32:17 +00:00 |
|
Kentaro Fukunaga
|
548c7a05e9
|
Merged PR 729: 画面実装(ファイル削除設定ポップアップ画面)
## 概要
[Task3547: 画面実装(ファイル削除設定ポップアップ画面)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3547)
- ファイル削除設定ポップアップの新規実装をしました。
## レビューポイント
- reduxとの連携部分の設計でNGな部分ないか?
- 処理の抜け漏れないか?
## 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/Task3547?csf=1&web=1&e=0pu92Q
## 動作確認状況
- ローカルで一通りの動作確認しました。
## 補足
- とくになし
|
2024-02-06 12:58:55 +00:00 |
|
saito.k
|
4be13e002d
|
Merged PR 710: 画面実装(削除操作)
## 概要
[Task3488: 画面実装(削除操作)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3488)
- ユーザー削除の画面実装
- 確認ダイアログ
- 削除API呼び出し
- エラーハンドリング
- 成功時のメッセージ
- 成功時のユーザー一覧更新
## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認
## 補足
- API呼び出しのエラーハンドリング部分はエラーコードが採番されたら追従します
|
2024-02-05 00:39:53 +00:00 |
|
Kentaro Fukunaga
|
44759b1aac
|
Merged PR 725: タイピストグループ削除 画面実装
## 概要
[Task3536: 画面実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3536)
- タイピストグループ画面に削除ボタン配置
- 削除ボタンクリック時に確認ダイアログ表示→OKで削除API実行&ぐるぐる表示
- API実行結果によってメッセージの出し分け&画面リスト更新
- 翻訳情報追加
## レビューポイント
- エラーコードに認識違いないか?
- 画面実装のお作法にそぐわないところがないか?
- なんか抜けてる実装などあれば
## UIの変更
大した変更ではないので、スクショ置き場に置く手間を省いてここに貼り付けます

## 動作確認状況
- 期待される動作を一通りローカルで確認しました。
- API実行中はぐるぐる表示されること
- API成功または削除済みの場合は成功メッセージ表示され、画面更新されること
- API失敗時、理由によってエラーメッセージが表示分けされること
|
2024-02-05 00:31:47 +00:00 |
|
makabe.t
|
8793606070
|
Merged PR 682: タスク一覧画面修正
## 概要
[Task3458: タスク一覧画面修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3458)
- タスク一覧画面のタスク削除ボタンからタスクを削除する処理を実装しました。
- タスクがInProgress、ユーザーがTypistの場合にはボタンを非活性となるようにしています。
## レビューポイント
- エラーごとの処理内容は適切でしょうか?
- ボタンの活性制御は適切でしょうか?
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2024-01-16 00:17:45 +00:00 |
|
makabe.t
|
81c299dd99
|
Merged PR 680: タスク削除API IF実装
## 概要
[Task3456: タスク削除API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3456)
- タスク削除APIのIFを実装しopenapi.jsonを更新しました。
## レビューポイント
- パス、バリデータは想定通りでしょうか?
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2024-01-11 08:47:28 +00:00 |
|
水本 祐希
|
c246586498
|
Merged PR 602: ライセンス確認画面(第五階層)に会社名を表示する(client側)
## 概要
[Task3225: ライセンス確認画面(第五階層)に会社名を表示する(client側)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3225)
- 何をどう変更したか、追加したライブラリなど
ライセンス確認画面(第5階層)の会社名取得APIを呼び出す実装を追加。
- このPull Requestでの対象/対象外
server側の実装は別ブランチで行うためここでは対象外
## レビューポイント
特になし
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認(完了)
|
2023-12-06 02:10:01 +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 |
|
makabe.t
|
68aaa0e548
|
Merged PR 586: バックアップAPI実装
## 概要
[Task3121: バックアップAPI実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3121)
- タスクのバックアップAPIとテストを実装しました。
- 実装に当たり、タスクエンティティ定義の型を修正しています。
## レビューポイント
- テストケースに過不足はないでしょうか?
- エンティティの値を取得した際に`is_job_number_enabled`が数値となってしまうのでtypeをBooleanに変更しましたが問題ないでしょうか?
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-11-20 08:57:56 +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 |
|
maruyama.t
|
e9541e22a2
|
Merged PR 523: 画面修正(ヘッダー右上にユーザー名を表示)
## 概要
[Task2920: 画面修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2920)
- 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず)
- 何をどう変更したか、追加したライブラリなど
ログイン時に、API(getMyUser)を呼び出し、フラグをtrueにして結果をstateに保存する。
フラグで管理する目的は、ページ遷移時に再度API呼び出しを行わないようにするため。
- このPull Requestでの対象/対象外
サインアウト部分は対象外→別タスクにて対応
- 影響範囲(他の機能にも影響があるか)
既存の各ページのヘッダー部分
## レビューポイント
- component/header配下にSliceを追加すると循環参照になってしまうのを避けるため、feature/login配下にSlice及び各処理を追加した、妥当か。
## 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/Task2920?csf=1&web=1&e=AQgVug
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-10-31 09:02:19 +00:00 |
|
saito.k
|
96848f5e54
|
Merged PR 499: 修正②(files,licenses , Repositoiesのlicenses)
## 概要
[Task2836: 修正②(files,licenses , Repositoiesのlicenses)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2836)
- feature
- files
- licenses
- repositories
- licenses
- users
- worktypes
- entity
- licenses
- audio_files
- audio_option_item
- checkout_permission
- アクセストークンをそのままService層に渡している箇所を修正し、必要なパラメータのみ渡すように修正
- クライアントの型生成
- 割り当て可能ライセンス取得APIのIFが変わったため
## レビューポイント
-
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルでテストが通ることを確認
## 補足
- 相談、参考資料などがあれば
|
2023-10-19 01:04:14 +00:00 |
|
makabe.t
|
74bf434786
|
Merged PR 496: ワークタイプ削除画面処理実装
## 概要
[Task2612: ワークタイプ削除画面処理実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2612)
- ワークタイプ削除の画面実装です。
## レビューポイント
- エラー処理に問題はないか
## UIの変更
- [Task2612](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/Task2612?csf=1&web=1&e=IgC7j3)
## 動作確認状況
- ローカルで確認
|
2023-10-17 06:29:25 +00:00 |
|
oura.a
|
a5b07596c1
|
Merged PR 486: IF不具合修正
## 概要
[Task2840: api.ts最新化](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2840)
IFの修正を行いました。
・/api/termsがPOSTになっていたのでGETに修正
・/api/termsのtypesの記載が足りていなかったので修正
## レビューポイント
なし
## UIの変更
なし
## 動作確認状況
swaggerUIで確認
## 補足
なし
|
2023-10-12 05:37:25 +00:00 |
|
oura.a
|
5b0058b707
|
Merged PR 482: ビルドエラー修正
## 概要
[Task2839: ビルドエラー修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2839)
ビルドエラーを修正しました。
## レビューポイント
なし
## UIの変更
なし
## 動作確認状況
ローカルでビルドが通ることを確認。
## 補足
なし
|
2023-10-11 08:44:07 +00:00 |
|
makabe.t
|
f4293213be
|
Merged PR 475: 画面実装
## 概要
[Task2786: 画面実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2786)
- ワークフロー一覧画面からワークフローを削除する処理を追加しました。
- アカウント画面の実装について想定外の変更となっていた、部品の不要なクラスを削除しました。
- クラスに当たっていた`required`はクラスではなくタグの属性だったため削除して属性に設定しています。
## レビューポイント
- 削除処理は適切か。
- アカウント画面の対応内容に問題はないか
- `select`の`required`は不要だと思いますがいかがでしょうか?
## UIの変更
- [Task2786](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/Task2786?csf=1&web=1&e=0TlIXM)
## 動作確認状況
- ローカルで確認
- 実際にAPIで削除しての動作は未検証です。
|
2023-10-11 08:35:51 +00:00 |
|
saito.k
|
088e6afc85
|
Merged PR 454: ワークフロー一覧画面
## 概要
[Task2735: ワークフロー一覧画面](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2735)
- ワークフロー一覧画面のデザイン反映
- 多言語対応
- 一覧取得API呼び出し
## レビューポイント
- デザイン反映に問題はないか
- フォルダ構成はこれでよいか
- workflow配下に直置き
## 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/Task2735?csf=1&web=1&e=IelNET
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-10-03 02:15:57 +00:00 |
|
水本 祐希
|
bf4dc1d717
|
Merged PR 443: 画面実装(アカウント削除確認ポップアップ)マージ用
## 概要
[Task2738: 画面実装(アカウント削除確認ポップアップ)マージ用](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2738)
- 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず)
- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
- 影響範囲(他の機能にも影響があるか)
## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認、develop環境で確認など
## 補足
- 相談、参考資料などがあれば
|
2023-09-26 01:25:39 +00:00 |
|
水本 祐希
|
1a0edee5c9
|
Merged PR 415: 画面実装(アカウント削除確認ポップアップ)
## 概要
[Task2669: 画面実装(アカウント削除確認ポップアップ)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2669)
- 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず)
- 何をどう変更したか、追加したライブラリなど
アカウント削除ボタンの挙動を実装
- この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/Task2669?csf=1&web=1&e=VbkLlR
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-09-26 00:52:46 +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
|
e7bd7b52fa
|
Merged PR 425: 親アカウント名表示修正
## 概要
[Task2687: 親アカウント名表示修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2687)
アカウント情報画面の親アカウント名表示対応を実施
## レビューポイント
なし
## UIの変更
なし
## 動作確認状況
ローカルで確認
## 補足
なし
|
2023-09-20 02:34:02 +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 |
|
makabe.t
|
b5ecd6de15
|
Merged PR 410: 画面実装(ActiveWorktypeID設定セレクトボックス)
## 概要
[Task2623: 画面実装(ActiveWorktypeID設定セレクトボックス)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2623)
- WorktypeID設定画面でのActiveWorktypeID選択処理を実装しました。
## レビューポイント
- WorkTypeIDの変更時の処理に問題はないか
- 画面の表示に問題はないか
## UIの変更
- [Task2623](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/Task2623?csf=1&web=1&e=sUPTbC)
## 動作確認状況
- ローカルで確認
|
2023-09-15 08:28:18 +00:00 |
|
saito.k
|
bdbb0dfe5d
|
Merged PR 394: 画面実装(オプションアイテム編集Popup)
## 概要
[Task2594: 画面実装(オプションアイテム編集Popup)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2594)
- オプションアイテム編集Popupを実装
- Popup表示時にWorktypeIDに紐づくOptionItemを取得
- 入力値のバリデーション
## レビューポイント
- エラーの表示の仕方を変えてみたが、問題ないか
-「 タスク 2630: 入力のエラーラベルの表示が不自然なので対応」の参考となる実装として
- 保存可能な文字種ルールの位置
- エラーラベルの位置
## 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/Task2594?csf=1&web=1&e=9Ygf6d
## 動作確認状況
- ローカルで確認
## 補足
- オプションアイテム更新の確認は未実施
|
2023-09-13 06:39:55 +00:00 |
|
makabe.t
|
3f17ec615c
|
Merged PR 402: API IF実装
## 概要
[Task2620: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2620)
- ActiveWorktypeAPIのIFを実装しました。
- クライアントはIF反映だけしています。
## レビューポイント
- IFはラフスケッチの認識通りか
- 制約に問題はないか。
- クライアントの対応に問題はないか
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-09-12 09:34:00 +00:00 |
|
makabe.t
|
f14c086980
|
Merged PR 386: ワークタイプ編集ポップアップ実装
## 概要
[Task2570: ワークタイプ編集ポップアップ実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2570)
- ワークタイプ編集ポップアップを実装しました。
- ローディング中処理をタスク追加にも追加しています。
## レビューポイント
- 画面の表示は想定通りか
- ローディング中処理の追加に問題はないか
## UIの変更
- [Task2570](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/Task2570?csf=1&web=1&e=bcRjsJ)
## 動作確認状況
- ローカルで確認
|
2023-09-07 08:58:36 +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 |
|
masaaki
|
9a67c513d9
|
Merged PR 367: 画面実装(注文履歴画面修正)
## 概要
[Task2497: 画面実装(注文履歴画面修正)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2497)
- 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず)
- 何をどう変更したか、追加したライブラリなど
- 注文履歴画面に対して、Issue Cancel押下時にライセンス発行キャンセルAPIを呼び出す処理を追加しました
- Issue Cancelボタンの表示条件を追加し、自分が第一または第二階層、かつ対象が第五階層の場合に表示するよう対応しました
- このPull Requestでの対象/対象外
- api.tsおよび言語毎のメッセージファイルに一部別機能のものが含まれています
- 影響範囲(他の機能にも影響があるか)
## レビューポイント
- 特にレビューしてほしい箇所
- Issue Cancelボタンの表示条件について、2行程度ですが外出ししています。変であればコメントいただければと思います。
## UIの変更
- 変更なしです
## 動作確認状況
- ローカルで確認済です
## 補足
- 相談、参考資料などがあれば
|
2023-08-31 07:35:22 +00:00 |
|
saito.k
|
e82f66e32a
|
Merged PR 365: 画面実装(TypistGroup編集ポップアップ&TypistGroup設定画面)
## 概要
[Task2458: 画面実装(TypistGroup編集ポップアップ&TypistGroup設定画面)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2458)
- タイピストグループ編集Popup作成
- Popupを呼び出す処理を追加
- 実行ボタンを「Save」に統一
- タスク一覧にも同様のボタンがあったので一緒に修正
## レビューポイント
- Popupを呼び出すときに選択したTypistGroupのIdを渡すようにしているが問題ないか
- 引数として渡した方がわかりやすいと思ったのでこの実装にしました
- Popupの項目をクリーンアップするタイミングをPopupが閉じたときに統一したが問題ないか
## 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/Task2458?csf=1&web=1&e=urD3Fa
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-08-30 02:37:01 +00:00 |
|
masaaki
|
ef8e6c44ec
|
Merged PR 360: 画面実装(注文履歴画面修正)
## 概要
[Task2483: 画面実装(注文履歴画面修正)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2483)
- 何をどう変更したか、追加したライブラリなど
Order Cancelボタン押下時にライセンス注文キャンセルAPIを呼び出す処理を追加。
- このPull Requestでの対象/対象外
- 影響範囲(他の機能にも影響があるか)
特になし
## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
## UIの変更
特になし
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
特になし
|
2023-08-29 01:29:34 +00:00 |
|
makabe.t
|
15d7119306
|
Merged PR 348: 画面実装(TypistGroup追加ポップアップ&TypistGroup設定画面)
## 概要
[Task2424: 画面実装(TypistGroup追加ポップアップ&TypistGroup設定画面)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2424)
- TypistGroup追加ポップアップを追加しました。
- 言語対応を実装しています。
- 入力エラー時のメッセージ表示を実装しています。
## レビューポイント
- エラーメッセージの出し方は問題ないか
- デザイン適用は適切か
## UIの変更
- [Task2424](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/Task2424?csf=1&web=1&e=IVxM43)
## 動作確認状況
- ローカルで確認
※実際のAPIを呼んでの登録は未検証です
|
2023-08-25 05:33:05 +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 |
|
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 |
|
makabe.t
|
adbd57bca6
|
Merged PR 333: 動作確認不具合修正
## 概要
[Task2419: 動作確認不具合修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2419)
- ディーラーをURLで指定して見つからなかった場合にスナックバーでメッセージを表示するように修正しました。
## レビューポイント
- UIの挙動として問題ないか
- メッセージは適切か
## UIの変更
- [Task2419](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/Task2419?csf=1&web=1&e=v1C0ki)
## 動作確認状況
- ローカルで確認
|
2023-08-17 08:08:28 +00:00 |
|
makabe.t
|
1145debabf
|
Merged PR 315: アカウント登録画面修正
## 概要
[Task2350: アカウント登録画面修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2350)
- アカウント登録画面のディーラー周りの実装を追加しました。
- クエリパラメータからアカウントIDを拾って入力画面に反映するようにしています。
- 言語情報をクエリパラメータから取得できるようにしました。
## レビューポイント
- 画面の挙動として不自然な点はないか
- ディーラー情報の取り扱いは適切か
## UIの変更
- [Task2350](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/Task2350?csf=1&web=1&e=fP9xdO)
## 動作確認状況
- ローカルで確認
|
2023-08-14 09:29:16 +00:00 |
|
oura.a
|
b59a5df44c
|
Merged PR 303: PBI1224_dev動作確認指摘対応
## 概要
[Task2325: PBI1224_dev動作確認指摘対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2325)
前PBIで指摘のあった箇所を修正しました。
・IssueDateが空白の場合「-」とするように修正
・ページ上部の「EFGI Legal」に会社名を入れるように修正
・上記に伴い、アカウント情報取得API(accounts/me)で会社名を取得できるように修正
・ページネーションの位置を修正
・ページネーション部分に表示されている「xxx licenses」を「xxx histories」に修正
・ページネーション周りの挙動を修正
## レビューポイント
なし
## 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/Task2325?csf=1&web=1&e=g6hYEy
## 動作確認状況
ローカルで動作確認済み
## 補足
なし
|
2023-08-07 01:52:03 +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 |
|
水本 祐希
|
1f5bfdaf3e
|
Merged PR 253: 画面実装(パートナーライセンス一覧画面)(初期表示部分)
## 概要
[Task2212: 画面実装(パートナーライセンス一覧画面)(初期表示部分)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2212)
パートナーライセンス一覧画面の初期表示を実装しました。
本タスクでのレビュー対象・対象外は以下となりますので、ご確認をお願いします。
■対象
・全体の画面レイアウト
・「Order License」「Order History」「License Card」「Card History」ボタンのログインtierによる表示制御
・「Order License」「License Card」ボタンの押下時の挙動
・表内の値(固定値)
■対象外
・パンくずリストの表示内容、押下時の挙動
・「return」ボタンの表示制御、押下時の挙動
・ページネーション周りの挙動
・「Order History」「Card History」ボタンの押下時の挙動
・表内の「Order History」「View Details」ボタンの表示制御、押下時の挙動
・表内のアカウント押下時の挙動
## レビューポイント
なし
## 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/Task2212?csf=1&web=1&e=dASrbE
## 動作確認状況
ローカルで動作確認済み
## 補足
パンくずリスト・表・ページネーション部分は、挙動を作成するタスクで外出しして別コンポーネントにする予定です。
|
2023-07-25 07:57:38 +00:00 |
|
maruyama.t
|
28a88a1b3d
|
Merged PR 255: 画面実装(パートナーアカウント追加PU)
## 概要
[Task2156: 画面実装(パートナーアカウント追加PU)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2156)
- 何をどう変更したか、追加したライブラリなど
パートナー画面のAdd 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/Task2156?csf=1&web=1&e=JdT2Bx
## 動作確認状況
- ローカルで表示確認
先にDONEのAPIと合わせてローカル環境で、パートナー追加の一連の流れが行えることを確認。
## 補足
- 相談、参考資料などがあれば
|
2023-07-19 09:25:24 +00:00 |
|
maruyama.t
|
4bbd9b371d
|
Merged PR 214: 第五階層の画面レイアウトを最新の状況にする
## 概要
[Task2034: 第五階層の画面レイアウトを最新の状況にする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2034)
- 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず)
- 何をどう変更したか、追加したライブラリなど
クライアント側の画面表示内容の修正。
AccountLock→StorageAvailable
サーバ側の処理でAccountLockと定義している変数名を適切な名称に修正。
- 影響範囲(他の機能にも影響があるか)
openapi.json変更に伴い、api.tsがdev環境の最新版になっています。
- 本PBIの対象外
第五階層用ライセンス情報画面以外の機能
scssファイルの修正箇所は、次回デザイナさんの修正時に正式に展開される想定です。
## レビューポイント
- 特にレビューしてほしい箇所
## 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/Task2034?csf=1&web=1&e=gfSwl3
## 動作確認状況
ローカル環境で画面の表示内容が変更されていることを確認
## 補足
- 相談、参考資料などがあれば
|
2023-07-10 04:03:03 +00:00 |
|
saito.k
|
3a7bf60f3e
|
Merged PR 202: 画面実装(PlayBackボタン)
## 概要
[Task1997: 画面実装(PlayBackボタン)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1997)
- Playbackボタン押下時の挙動を実装
- typist
- 自身が割り当て候補となっているタスクをPlayBackする
- 成功時、カスタムURLスキームでデスクトップアプリを起動する
- author
- 自身のAuthorIDと一致するタスクをPlayBackする
- 成功時、カスタムURLスキームでデスクトップアプリを起動する
- ログイン時の、カスタムURLスキームを実際のデスクトップアプリのスキーム名に修正
## レビューポイント
- playbackAsyncのなかでソート条件更新APIを一緒に呼び出しているが問題ないか
- ソート条件を更新するタイミングはここで問題ないか
- ユーザーがTypistの時のみ更新するようにしたが問題ないか
## 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/Task1997?csf=1&web=1&e=9kLaxo
## 動作確認状況
- ローカルで確認
## 補足
- Authorの挙動はAPI側の実装が完了していないので、未確認
|
2023-07-04 06:06:37 +00:00 |
|