saito.k
|
de13753ef6
|
Merged PR 447: [Sp18-2 or 19-1]翻訳情報を翻訳Excelに適用し、Webアプリにも適用する
## 概要
[Task2166: [Sp18-2 or 19-1]翻訳情報を翻訳Excelに適用し、Webアプリにも適用する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2166)
- 不要な文言を削除
- 削除した文言が使用されている箇所を修正
## レビューポイント
- 特になし
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-09-27 08:12:25 +00:00 |
|
saito.k
|
8265ca38c8
|
Merged PR 440: 画面実装(テンプレートファイルアップロードPopup)
## 概要
[Task2656: 画面実装(テンプレートファイルアップロードPopup)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2656)
- テンプレートファイルアップロードのAPI呼び出し周りを実装
- SASトークン付きURL取得
- Blobストレージへファイルアップロード
- アップロード完了
- server側
- `helmet`の`connect-src`を修正
- SASトークン付きURLが想定と違っていたため修正
- DBに保存するURLが想定と違っていたため修正
## レビューポイント
- `connect-src`の`self`以外はローカル環境のみの設定でよさそう?
- 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/Task2656?csf=1&web=1&e=iU1huG
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-09-27 06:36:24 +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
|
9ca4ae61f8
|
Merged PR 434: 画面実装(テンプレートファイルアップロードPopupデザイン)
## 概要
[Task2664: 画面実装(テンプレートファイルアップロードPopupデザイン)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2664)
- デザイン反映
- ファイルピッカーからファイル取得→storeに保存
## レビューポイント
- デザイン反映に不備はあるか
- 想定としてstoreに保持したfileを、Operationsでblobにアップロードする流れにしようとしているがよさそうか
## 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/Task2664?csf=1&web=1&e=optFai
## 動作確認状況
- ローカルで確認、develop環境で確認など
## 補足
- 相談、参考資料などがあれば
|
2023-09-25 07:53:45 +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
|
8fee108a78
|
Merged PR 422: 成功時のスナックバー追加
## 概要
[Task2685: 成功時のスナックバー追加](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2685)
情報更新成功時の成功スナックバー実装が漏れていたため、追加しました。
## レビューポイント
なし
## UIの変更
なし
## 動作確認状況
ローカルで確認
## 補足
なし
|
2023-09-19 10:32:33 +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
|
ec5df50aa0
|
Merged PR 420: 結合動作確認不具合対応
## 概要
[Task2684: 結合動作確認不具合対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2684)
- Active WorktypeIDの変更について結合動作確認で見つかった不具合に対応しました。
- ActiveWorktypeIDの変更がダイアログ表示の前にセレクトボックスに反映される
- 変更成功時に成功スナックバーを表示する
## レビューポイント
- セレクトボックスの変更時のイベントの取り扱いは適切か
## UIの変更
- [Task2648](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/Task2648?csf=1&web=1&e=rxWAF0)
## 動作確認状況
- ローカルで確認
|
2023-09-19 09:29:04 +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 |
|
水本 祐希
|
606ff6de9b
|
Merged PR 379: 画面実装(パートナー一覧画面本実装)
## 概要
[Task2539: 画面実装(パートナー一覧画面本実装)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2539)
- 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず)
- 何をどう変更したか、追加したライブラリなど
パートナー一覧画面でパートナーの一覧が表示されるように実装
- このPull Requestでの対象/対象外
・Add Accountボタンは前PBIのため対象外
・Dealer Managementボタンの挙動は対象外
・Delete Accountボタンの挙動は対象外
- 影響範囲(他の機能にも影響があるか)
特になし
## レビューポイント
- 特にレビューしてほしい箇所
・Dealer Management、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/Task2539?csf=1&web=1&e=PNI5bw
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-09-12 01:14:26 +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
|
2d378b8e66
|
Merged PR 380: 画面実装(ワークタイプID追加Popup)
## 概要
[Task2517: 画面実装(ワークタイプID追加Popup)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2517)
- ワークタイプID追加Popupを実装
- 入力チェック
- ワークタイプID作成API呼び出し
- 多言語対応
## レビューポイント
- 画面にワークタイプIDのルールを追加したがデザイン的に問題ありそうか
## 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/Task2517?csf=1&web=1&e=Fm7bpm
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-09-05 02:41:39 +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 |
|
saito.k
|
fe7581cf10
|
暗号化パスワードの初期値をundefindに修正
|
2023-08-24 12:06:43 +09: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 |
|
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
|
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
|
28210968e0
|
Merged PR 318: develop動作確認
## 概要
[Task2343: develop動作確認](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2343)
エラーメッセージを修正しました。
## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認、develop環境で確認など
## 補足
- 相談、参考資料などがあれば
|
2023-08-09 01:49:16 +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 |
|
水本 祐希
|
c304271494
|
Merged PR 302: 画面修正(注文履歴)
## 概要
[Task2339: 画面修正(注文履歴)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2339)
注文履歴画面について、以下の修正を行いました。
・Issueボタン押下時の処理を追加
・Issueボタン、Issue Cancelボタン、Order Cancelボタンのレイアウト修正
※Issue Cancelボタン、Order Cancelボタン押下時の動作は今回は対象外です。
## レビューポイント
なし
## 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/Task2339?csf=1&web=1&e=sUjf1e
## 動作確認状況
ローカルで動作確認済み
## 補足
なし
|
2023-08-08 04:46:59 +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 |
|
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 |
|
oura.a
|
eb8c31cf05
|
Merged PR 300: view detailsで第5階層の情報を参照した際、ログインアカウントではなく第5のアカウント情報を表示できるようにする
## 概要
[Task2335: view detailsで第5階層の情報を参照した際、ログインアカウントではなく第5のアカウント情報を表示できるようにする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2335)
パートナーライセンス一覧画面から「viewDetails」ボタンを押下した際の挙動を実装しました。
具体的な対応は以下になります。
・第五以外でも第五の情報を取得できるよう、ライセンス情報APIを呼び出す際の処理を修正
・ライセンス情報APIのRoleGuardからtierの制限を削除
・第五以外から第五の情報を表示する際に、「Order License」「Activate License Key」ボタンを表示しないよう修正
・第五以外から第五の情報を表示する際に、「Return」ボタンを追加
## レビューポイント
なし
## 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/Task2335?csf=1&web=1&e=SPJKzU
## 動作確認状況
ローカルで動作確認済み
## 補足
なし
|
2023-08-03 08:56:39 +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 |
|
masaaki
|
9ef4752bc1
|
Merged PR 293: 注文履歴画面_画面遷移対応
## 概要
[Task2273: 注文履歴画面_画面遷移対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2273)
- 注文履歴画面について、画面遷移を意識した対応を行いました。
- 何をどう変更したか、追加したライブラリなど
- パートナーライセンス画面について、選択行の情報(selectedRow)をstateで保存し、ボタン押下で遷移した先の画面で使用できるよう対応
- 注文履歴画面について、stateの値の有無により、自アカウントとして情報を取得するか他アカウントとして情報を取得するか判定するよう対応
- ボタン表示について自アカウント、子アカウント+ステータスを意識した表示を行うよう対応
- ライセンス画面(第一~第四階層)から注文履歴画面を呼び出すよう対応
- 「LicenseOrderHistoryPage」について、仮画面用で不要となったので削除しました
- このPull Requestでの対象/対象外
- 「Issue」「Issue Cancel」のボタンについて、デザイナさん画面とことなる配置になっています。ボタンについてはPBI対象外で、親子で表示ボタンを変更できることを確認することが目的なので対象外とさせてください。(PBI1210の中で値対応します)
- 影響範囲(他の機能にも影響があるか)
- 無し
## レビューポイント
- 特筆する部分はなし
## UIの変更
- 無し
## 動作確認状況
- ローカルで確認済
## 補足
- 相談、参考資料などがあれば
|
2023-08-02 01:51:42 +00:00 |
|
oura.a
|
6fc9a2db64
|
Merged PR 252: 画面実装(パートナーライセンス一覧画面)(画面遷移・ページネーション周り)
## 概要
[Task2228: 画面実装(パートナーライセンス一覧画面)(画面遷移・ページネーション周り)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2228)
パートナー一覧画面の画面遷移・ページネーション周りを実装しました。
以下は対象外となります。
・「ViewDetails」を押下したときの挙動
・「Order History」を押下したときの画面の内容
## レビューポイント
なし
## 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/Task2228?csf=1&web=1&e=8Dup5q
## 動作確認状況
ローカルで動作確認済み
## 補足
なし
|
2023-08-01 02:34:29 +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 |
|
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 |
|
oura.a
|
b4026c1460
|
Merged PR 234: [dev動作確認指摘の修正]異常系メッセージ表示
## 概要
[Task2194: [dev動作確認指摘の修正]異常系メッセージ表示](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2194)
- 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず)
- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
- 影響範囲(他の機能にも影響があるか)
## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認、develop環境で確認など
## 補足
- 相談、参考資料などがあれば
|
2023-07-13 02:02:39 +00:00 |
|
oura.a
|
9528bb1ad6
|
Merged PR 226: テキストボックス分割実装
## 概要
[Task2168: テキストボックス分割実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2168)
タスク 2168: テキストボックス分割実装
ライセンスキー入力のボックスを5分割に変更。
細かい画面レイアウトについては後々デザイナーさんに依頼することになるかと思いますので、レビュー対象外でお願いします。
## レビューポイント
入力時の挙動に過不足がないか。
## 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/Task2168?csf=1&web=1&e=oRgAOK
## 動作確認状況
- ローカルで動作確認済み
## 補足
なし
|
2023-07-13 00:02:40 +00:00 |
|
oura.a
|
883224c914
|
Merged PR 229: 再コミット_画面実装(カードライセンス取り込みPU)
## 概要
[Task2171: 再コミット_画面実装(カードライセンス取り込みPU)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2171)
- 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず)
- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
- 影響範囲(他の機能にも影響があるか)
## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認、develop環境で確認など
## 補足
- 相談、参考資料などがあれば
|
2023-07-11 07:02:33 +00:00 |
|
oura.a
|
3584a65682
|
Merged PR 228: Revert "Merged PR 218: 画面実装(カードライセンス取り込みPU)
Revert "Merged PR 218: 画面実装(カードライセンス取り込みPU)
## 概要
[Task2161: 画面実装(カードライセンス取り込みPU)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2161)
タスク 2161: 画面実装(カードライセンス取り込みPU)
カードライセンス取り込みポップアップを実装しました。
以下の項目については別タスクへ切り出しての対応とし、本タスクでは対象外とさせてください。
・テキストボックスへの4文字区切りの入力
・テキストボックスの文字数制限
・改行コード入力時の挙動(バーコードリーダー対応)
## レビューポイント
なし
## UIの変更
https://ndstokyo.sharepoint.com/:i:/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/Task2161/%E3%82%AB%E3%83%BC%E3%83%89%E3%83%A9%E3%82%A4%E3%82%BB%E3%83%B3%E3%82%B9%E5%8F%96%E3%82%8A%E8%BE%BC%E3%81%BFPU.PNG?csf=1&web=1&e=cRtMX3
## 動作確認状況
ローカルで動作確認済み
## 補足
なし"
Reverted commit `9a66ca02`.
|
2023-07-11 05:50:51 +00:00 |
|
oura.a
|
9a66ca027a
|
Merged PR 218: 画面実装(カードライセンス取り込みPU)
## 概要
[Task2161: 画面実装(カードライセンス取り込みPU)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2161)
タスク 2161: 画面実装(カードライセンス取り込みPU)
カードライセンス取り込みポップアップを実装しました。
以下の項目については別タスクへ切り出しての対応とし、本タスクでは対象外とさせてください。
・テキストボックスへの4文字区切りの入力
・テキストボックスの文字数制限
・改行コード入力時の挙動(バーコードリーダー対応)
## レビューポイント
なし
## UIの変更
https://ndstokyo.sharepoint.com/:i:/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/Task2161/%E3%82%AB%E3%83%BC%E3%83%89%E3%83%A9%E3%82%A4%E3%82%BB%E3%83%B3%E3%82%B9%E5%8F%96%E3%82%8A%E8%BE%BC%E3%81%BFPU.PNG?csf=1&web=1&e=cRtMX3
## 動作確認状況
ローカルで動作確認済み
## 補足
なし
|
2023-07-11 05:06:10 +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 |
|
maruyama.t
|
48b8cdc3e3
|
Merged PR 216: カードライセンス発行PU,処理成功時のメッセージを汎用メッセージから専用メッセージに変更する。
## 概要
[Task2130: カードライセンス発行PU,処理成功時のメッセージを汎用メッセージから専用メッセージに変更する。](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2130)
- 元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/Task2130?csf=1&web=1&e=Tu1y6N
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-07-06 07:03:58 +00:00 |
|
masaaki
|
4dd2446109
|
Merged PR 211: Create押下後に処理が成功するまでloadingを表示させる
## 概要
[Task2129: Create押下後に処理が成功するまでloadingを表示させる](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2129)
- ボタン押下時に処理中であることを示すイメージを実装しました
## レビューポイント
- licenseCardIssueSlice.tsでの、fulfilled時のloading解除タイミングについて。
fulfilled時、csvのblobイメージ作成→ダウンロード実施、という処理を実施しています。
同時実行を避ける意味で、ダウンロード実施の後(fulfilled処理の最後)でloading解除としていますが、タイミングとして問題ないか。
(とはいえ、blob作成中に解除する理由は浮かばないので、これで問題ないと考えています。何かあれば、くらいの感覚です)
## UIの変更
- 無し
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-07-05 05:44:15 +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 |
|
oura.a
|
5241fcc997
|
Merged PR 203: メッセージファイル最新化
## 概要
[Task2110: メッセージファイル最新化](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2110)
- 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず)
- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
- 影響範囲(他の機能にも影響があるか)
## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認、develop環境で確認など
## 補足
- 相談、参考資料などがあれば
|
2023-07-03 06:47:54 +00:00 |
|
oura.a
|
709db0d2ec
|
Merged PR 187: 画面実装(カードライセンス発行PU)
## 概要
[Task1991: 画面実装(カードライセンス発行PU)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1991)
カードライセンス発行ポップアップを新規作成。
上記ポップアップを起動するボタンを第1~4階層用のライセンスページに仮配置。
## レビューポイント
特筆するものはなし。
ライセンスページのポップアップ起動ボタンは仮配置なので、レビュー対象外でお願いします。
## UIの変更
https://ndstokyo.sharepoint.com/:i:/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/Task1991/%E3%82%AB%E3%83%BC%E3%83%89%E3%83%A9%E3%82%A4%E3%82%BB%E3%83%B3%E3%82%B9%E7%99%BA%E8%A1%8CPU.PNG?csf=1&web=1&e=NtuRgu
## 動作確認状況
ローカルで動作確認済み
## 補足
・csvファイルダウンロード確認のポップアップは、現在は表示しないつくりになっています。PBIの受け入れ条件には表示するように記載があるのですが、月曜日のDSで奥澤さんに確認予定です。
・ライセンス数入力のテキストボックスで「.」「-」「+」が入力できてしまう事象がありました。こちらですがライセンス注文ポップアップでも同様の仕様となっているため、本PBI内の別タスクで纏めての修正とさせてください。
|
2023-07-03 05:35:19 +00:00 |
|