nik/OMDSCloud
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity
366 Commits 1 Branch 50 Tags
Commit Graph

366 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
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
maruyama.t
3f5f75a48f Merged PR 403: API実装（アカウント設定API） 
## 概要
[Task2603: API実装（アカウント設定API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2603)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
getDealerAccountという一階層上のアカウントを取得する共通的なAPIも実装しています。

- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）
なし

## レビューポイント
- 実行中にdealerアカウントに対して変更が走った場合でも対応できるよう、トランザクションをネストした実装にしています。
トランザクションをネストした場合は、内部のトランザクションが正常に完了し、その後外部のトランザクションも正常に完了すると、変更がコミットされます。
- 画面の仕様上、第五階層でないとdealerの変更は行わないが、API側でdelegationPermissionに対する階層（不整合チェック）をやっていないが、問題ないか。
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
なし

## 動作確認状況
- ローカルで確認
第五階層以外がアカウント情報を設定できる
アカウント情報を更新する（第五階層が実行/セカンダリ管理者ユーザがnull）
・プライマリ管理者ユーザを存在する値にして、更新される
・セカンダリ管理者ユーザをundefinedで入力し、nullで更新される
アカウント情報を更新する（第五階層以外が実行）
アカウント情報を更新する（ディーラーアカウントが未入力）
・parentAccountIdがnullで更新される
アカウント情報の更新に失敗する（ディーラー未存在）
アカウント情報の更新に失敗する（プライマリ管理者ユーザ未存在）
アカウント情報の更新に失敗する（プライマリ管理者ユーザがnull）
アカウント情報の更新に失敗する（セカンダリ管理者ユーザ未存在）
アカウント情報の更新に失敗する(プライマリ管理者ユーザ、セカンダリ管理者ユーザ両方が未入力)
以下POSTMANで確認
プライマリ管理者ユーザIDをundefinedで入力した場合はエラー
管理者権限のないアカウントで実行した場合、権限エラー
500エラー
## 補足
- 相談、参考資料などがあれば
 2023-09-19 07:12:58 +00:00
makabe.t
d1a8b887e5 Merged PR 417: ActiveWorktypeIDを未設定に戻せるように修正 
## 概要
[Task2677: ActiveWorktypeIDを未設定に戻せるように修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2677)

- ActiveWorktypeIDを設定後、未選択に戻す操作ができるように修正

## レビューポイント
- 修正箇所の共有

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-09-19 02:56: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
makabe.t
d843affe88 Merged PR 413: API実装（ActiveWorktypeID更新API） 
## 概要
[Task2622: API実装（ActiveWorktypeID更新API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2622)

- ActiveWorkTypeID更新APIとテストを実装しました。

## レビューポイント
- リポジトリの更新処理は適切か
- テストケースは適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-09-15 06:26:37 +00:00
湯本 開
a8983c418e Merged PR 414: 脆弱性試験ツール対応 
## 概要
[Task2665: 脆弱性試験ツール対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2665)

- Viteビルドを介さずデプロイしたい静的ファイルを配置する用のディレクトリを追加
- ビルド時にファイルコピーをする処理をnpm run build:* コマンドに追加
- 今回配置依頼のあった静的ファイルを配置

## レビューポイント
- 情報共有

## 動作確認状況
- ローカルで確認
 2023-09-15 05:16:01 +00:00
水本 祐希
78cbfd15e8 Merged PR 400: API修正（アカウント情報取得API） 
## 概要
[Task2601: API修正（アカウント情報取得API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2601)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
    - アカウント情報取得APIで返却する値を追加
    - テストしやすさを考慮し、getMyAccountInfoのパラメータと関数名を修正
    - ログ出力について規約に沿った形に修正

- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）

## レビューポイント
- 特にレビューしてほしい箇所
アクセストークンを使ったユニットテストがあれば教えてください。

- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載

## UIの変更
- Before/Afterのスクショなど
- スクショ置き場

## 動作確認状況
- ローカルで確認
アクセストークンからアカウント情報を取得するAPIであるため、ポストマンで確認しました。
- 確認事項
  - 追加したtier、country、parentAccountId、delegationPermission、primaryAdminUserId、secondryAdminUserIdが返却されることを確認。
  - 異常系
    - MySQLにてusersとaccountsがない場合のエラーメッセージが返却されるかを確認
　

## 補足
- 相談、参考資料などがあれば
 2023-09-15 02:17:54 +00:00
makabe.t
81d17fb57e Merged PR 408: API修正（WorktypeID一覧API） 
## 概要
[Task2621: API修正（WorktypeID一覧API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2621)

- Worktype一覧APIでActiveWorkTypeも取得するよう修正しました。
- API IFでプロパティがnullableになっているところをrequiredに修正しました。

## レビューポイント
- ActiveWorktypeの取得に問題はないか
- テストケースは適切か
- IFの修正は問題ないか

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-09-14 09:20:04 +00:00
水本 祐希
25de994013 Merged PR 411: パートナー一覧画面のDealer Management修正 
## 概要
[Task2635: パートナー一覧画面のDealer Management修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2635)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
APIの返却値によってDealerManagementのチェックの有効・無効を制御する

- この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/Task2539?csf=1&web=1&e=n647LC

## 動作確認状況
- ローカルで確認
MySQLWorkbenchのdelegation_permissionの値を操作
  - delegation_permission=0（false）の場合、チェック無効
  - delegation_permission=1（true）の場合、チェック有効

## 補足
- 相談、参考資料などがあれば
 2023-09-14 09:09:01 +00:00
oura.a
f2c442457c Merged PR 412: IF不具合修正（アカウント設定API） 
## 概要
[Task2663: IF不具合修正（アカウント設定API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2663)

アカウント設定APIのIF不具合を修正しました。

## レビューポイント
なし

## UIの変更
なし

## 動作確認状況
クライアント側のapi.tsに期待通りの変更が反映されることを確認。

## 補足
なし
 2023-09-14 06:45:58 +00:00
saito.k
40f526c81c エラー表示を他画面とそろえる 2023-09-14 11:19:33 +09:00
makabe.t
24d3117bdc Merged PR 405: DBマイグレーション 
## 概要
[Task2624: DBマイグレーション](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2624)

- AccountテーブルにActiveWorktypeIDを設定するカラムを追加しました。
  - 外部キー制約によってWorkTypeのIDと紐づけてWorkTypeに連動してカラムがNULLになるようにしています。

## レビューポイント
- カラムの設定は適切か
- 外部キー制約は適切に設定されているか

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
  - up/down
  - WorkTypeの削除によるSet NULL
 2023-09-13 08:40:22 +00:00
saito.k
8c8362150a デプロイで使用するdockerFileのnodeのバージョンを開発環境と合わせる 2023-09-13 15:56:35 +09: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
saito.k
95b48a766a Merged PR 409: nodeバージョンアップ 
## 概要
[Task2647: nodeバージョンアップ](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2647)

- タイトルの通り

## レビューポイント

## UIの変更
- Before/Afterのスクショなど
- スクショ置き場

## 動作確認状況
- ローカルで確認、develop環境で確認など

## 補足
- 相談、参考資料などがあれば
 2023-09-13 06:38:01 +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
3e12721a92 Merged PR 398: API実装（オプションアイテム更新） 
## 概要
[Task2593: API実装（オプションアイテム更新）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2593)

- オプションアイテム更新APIとテストを実装しました。
  - オプションアイテムのバリデータを追加しています。

## レビューポイント
- オプションアイテムのパラメータチェック用バリデータは適切か
- テストケースは適切か
- リポジトリの更新処理に問題はないか

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-09-12 08:52:12 +00:00
oura.a
9f7de83ae7 Merged PR 401: API-IF実装 
## 概要
[Task2600: API-IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2600)

アカウント設定APIのIFを実装しました。

## レビューポイント
なし

## UIの変更
なし

## 動作確認状況
ローカルのswaggerUIで確認済み

## 補足
なし
 2023-09-12 06:27:27 +00:00
makabe.t
bb1dd6bce1 Merged PR 393: DBマイグレーション 
## 概要
[Task2617: DBマイグレーション](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2617)

オプションアイテムテーブルのカラム長を変更するマイグレーションファイルを追加しました。
- 変更対象カラムは以下の通りです。
  - item_label : 50 ⇒ 16
  - initial_value : 50 ⇒ 20

## レビューポイント
- 修正内容は適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-09-12 05:56:50 +00:00
saito.k
7422ef38e1 Merged PR 399: API IF修正 
## 概要
[Task2631: API IF修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2631)

- OptionItem取得APIでidも返却するように修正

## レビューポイント
- OptionItem更新APIの引数で使用している型と分けたが問題ないか
  - 主に名前

## UIの変更
- Before/Afterのスクショなど
- スクショ置き場

## 動作確認状況
- ローカルで確認

## 補足
- 相談、参考資料などがあれば
 2023-09-12 01:42:08 +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
2dcb1c1f84 Merged PR 396: API実装（オプションアイテム取得） 
## 概要
[Task2592: API実装（オプションアイテム取得）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2592)

- オプションアイテム取得APIとテストを実装しました。

## レビューポイント
- リポジトリの取得ロジックは想定通りか
- テストケースは適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-09-11 08:31:03 +00:00
saito.k
d8d5789f5a Revert "redis接続確認" 
This reverts commit 28c5704b16fd161a96668e13e26a6d6c47770528.
 2023-09-10 11:27:02 +09:00
saito.k
28c5704b16 redis接続確認 2023-09-09 10:20:32 +09:00
oura.a
2812bc3d20 Merged PR 383: API実装（パートナー取得API） 
## 概要
[Task2540: API実装（パートナー取得API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2540)

パートナー取得APIを実装しました。

## レビューポイント
・データ取得方法が適切かどうか。
以下の優先順位を意識して作成したが適切か？また、意識できていない実装になっていないか？
①QueryBuilderを使用せずに処理する
②RDB、adb2cへのアクセス回数を最小限にする

## UIの変更
なし
## 動作確認状況
ローカルで動作確認済み、UT実施済み

## 補足
プライマリ、セカンダリ管理者IDがない場合のテストはUTでは実装せず、ローカルでの動作確認で正常に動作することを確認しました。
（プライマリ、セカンダリ管理者IDを指定してアカウントを作成するテストユーティリティを作成する必要があるが、あまり汎用的には思えず作成する手間が惜しかったため）
 2023-09-08 09:45:10 +00:00
maruyama.t
82fb224d67 Merged PR 390: [Sp17着手]ボタン押下時処理にpreventDefault()を行うものとおこなわないものが混在するのを整理する 
## 概要
[Task2547: [Sp17着手]ボタン押下時処理にpreventDefault()を行うものとおこなわないものが混在するのを整理する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2547)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
パートナー追加ポップアップで処理実行中にでブラウザを閉じようとしたときに確認ダイアログを表示していた箇所を削除（コピペ元からの削除漏れ）
ライセンス注文履歴画面で、ボタン押下時に不要なpreventDefaultを行っていた箇所を削除
- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）

## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載

## UIの変更
- Before/Afterのスクショなど
- スクショ置き場

## 動作確認状況
- ローカルで確認

## 補足
- 相談、参考資料などがあれば
 2023-09-08 05:27:52 +00:00
saito.k
da1ee8a9c2 Merged PR 391: [Sp17着手]セレクトボックスで長い文字列が入っても問題ないようにする 
## 概要
[Task2550: [Sp17着手]セレクトボックスで長い文字列が入っても問題ないようにする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2550)

- タイトルの通り

## レビューポイント
- 特になし

## 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/Task2550?csf=1&web=1&e=EcQEIB

## 動作確認状況
- ローカルで確認

## 補足
- 相談、参考資料などがあれば
 2023-09-08 05:01:24 +00:00
makabe.t
c71cab92cb Merged PR 392: API IF実装 
## 概要
[Task2591: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2591)

- オプションアイテムの取得/更新APIのIFを実装し、OpenAPIの定義を更新しました。

## レビューポイント
- パスは適切か
- パラメータの内容、バリデータは適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-09-08 01:38:44 +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
makabe.t
6cba76fa61 Merged PR 385: ワークタイプ更新API実装 
## 概要
[Task2569: ワークタイプ更新API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2569)

- worktypeID更新APIとテストを実装しました。

## レビューポイント
- リポジトリの更新処理に問題はないか
- テストケースは適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-09-07 00:53:30 +00:00
oura.a
6605658bac Merged PR 389: IF不具合修正 
## 概要
[Task2589: IF不具合修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2589)

パートナー一覧取得APIに以下の不具合があったため修正しました。
・RoleGuardが誤っている（tier3,tier4が不足）
・パラメータを文字列として受け取ってしまう
※openapi.jsonへ跳ねる修正はありません。

## レビューポイント
なし

## UIの変更
なし

## 動作確認状況
ローカルで動作確認済み（パラメータ受け取り）

## 補足
なし
 2023-09-06 08:45:44 +00:00
makabe.t
bde1ff60a9 Merged PR 384: API IF実装 
## 概要
[Task2568: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2568)

- worktype更新APIのIFを実装し、OpenAPI定義を更新しました。

## レビューポイント
- パスは適切か
- パラメータの制約は適切か

## UIの変更
- なし
## 動作確認状況
- ローカルで確認
 2023-09-05 08:09:41 +00:00
saito.k
b9a2d9b6b4 redis周りの修正をもとに戻す 2023-09-05 15:28:43 +09:00
saito.k
7c946686b7 Merged PR 289: ローカルでRedisコンテナに接続する 
## 概要
[Task2312: ローカルでRedisコンテナに接続する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2312)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）

## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載

## UIの変更
- Before/Afterのスクショなど
- スクショ置き場

## 動作確認状況
- ローカルで確認、develop環境で確認など

## 補足
- 相談、参考資料などがあれば
 2023-09-05 05:52:49 +00:00
maruyama.t
7524abbae6 Merged PR 378: API実装（ライセンス発行キャンセルAPI） 
## 概要
[Task2498: API実装（ライセンス発行キャンセルAPI）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2498)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
ライセンス発行をキャンセルするAPIを実装
下位のアカウント情報と、上位のアカウント情報をセットすると、パートナー関係であるかを返す関数を追加
既存のユニットテストのライセンス作成箇所で、注文ID、削除日時、削除注文IDを指定できるように修正
- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）
既存ユニットテストのライセンス作成部分

## レビューポイント
- 特にレビューしてほしい箇所
パートナー関係かどうかを返す箇所、共通的に使いやすいかどうか
14日より経過していた場合の箇所、ライセンスの有効期限の定数を使っているが分けたほうが良いか
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場

## 動作確認状況
- ローカルで確認
■正常系
ライセンス発行のキャンセルが完了できる（第一階層で実行）
ライセンス発行のキャンセルが完了できる（第二階層で実行）
キャンセルした発行の注文状態が発行待ちに戻る
発行されたライセンスは物理削除される
論理削除されていたライセンスは未割当で、削除前の状態に戻る
■異常系
第一、第二階層以外で実行した場合はエラー
キャンセル対象の発行が存在しない場合エラー
キャンセル対象の発行が14日より経過していた場合はエラー
キャンセル対象の発行のライセンスが使われていた場合はエラー
自身のパートナー以外の発行をキャンセルしようとした場合、エラー

## 補足
- 相談、参考資料などがあれば
 2023-09-05 05:17:47 +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
makabe.t
009f133c8b Merged PR 381: バグ対応&動作確認 
## 概要
[Task2551: バグ対応&動作確認](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2551)

- 管理者（None）ユーザーがタスク一覧画面でplaybackを押せないように修正しました。

## レビューポイント
- 対応内容、結果に問題はないか

## UIの変更
- [Task2551](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/Task2551?csf=1&web=1&e=7E3OvS)

## 動作確認状況
- ローカルで確認
 2023-09-05 00:03:07 +00:00
oura.a
17640eefa1 Merged PR 382: API IF実装 
## 概要
[Task2542: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2542)

パートナー取得APIのIFを実装しました。

## レビューポイント
なし

## UIの変更
なし

## 動作確認状況
ローカルでSwaggerUiを確認

## 補足
なし
 2023-09-04 08:37:01 +00:00
makabe.t
0b7d979fae Merged PR 375: API実装（ワークタイプID追加API） 
## 概要
[Task2516: API実装（ワークタイプID追加API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2516)

- ワークタイプ追加APIとテストを実装しました。
  - オプションアイテムも一緒に追加されるように実装しています。
  - ワークタイプの制限のためにカスタムバリデータを実装しています。

## レビューポイント
- 追加時のエラー処理は適切か
- バリデータは適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-09-04 07:08:19 +00:00
湯本 開
60269306c5 Merged PR 285: Staging用デプロイパイプラインを作成する 
## 概要
[Task2198: Staging用デプロイパイプラインを作成する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2198)

- Stating/Production用のデプロイパイプラインの構成ファイルを作成

## レビュー対象外
- depoyment slotを使ったデプロイには未対応（別PBIで対応予定）

## レビューポイント
- ざっくり実施される処理とその順序として問題はなさそうか
- 情報共有

## 動作確認状況
- Dockerイメージがビルド&プッシュ、静的ファイルがビルド&Pipeline用Blobストレージにアップロードされる所までは確認済
- 書式はPipeline上画面で確認済
 2023-09-03 12:05:18 +00:00
oura.a
f56f95123b Merged PR 376: [Sp17完了MISO]バリデータをクラスを使用した記述に統一する 
## 概要
[Task2502: [Sp17完了MISO]バリデータをクラスを使用した記述に統一する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2502)

バリデータの記述方法をクラスで外だしする形に統一しました。
また、未使用の引数を削除しました。

## レビューポイント
期待通りの修正内容であるか。
未使用の引数を削除してしまったが、問題ないか。

## UIの変更
なし

## 動作確認状況
ローカルで該当バリデーションを使用しているAPIを実行し、動作を確認済み

## 補足
なし
 2023-09-01 09:12:52 +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にしたが問題ないか
  - 必須にすると画面初期表示時に「０件です」がちらつくため

## 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
makabe.t
f43e0f779e Merged PR 373: DBマイグレーション 
## 概要
[Task2515: DBマイグレーション](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2515)

- オプションアイテムのテーブルを追加するマイグレーションファイルを追加しました。

## レビューポイント
- カラム項目は適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
  - migrate up/down
 2023-09-01 06:42:58 +00:00
makabe.t
2da40e278b Merged PR 371: API実装 
## 概要
[Task2505: API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2505)

- ワークタイプ一覧取得APIとテストを実装しました。
  - ワークタイプテーブルのリポジトリも実装しています。

## レビューポイント
- リポジトリの実装に問題はないか
- テスト実装に不足はないか

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-09-01 00:24:01 +00:00
水本 祐希
bb0bdce84c Merged PR 377: 修正の実施（再度） 
## 概要
[Task2548: 修正の実施（再度）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2548)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
isApproveTierの重複を解消しました。

- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）

## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載

## UIの変更
- Before/Afterのスクショなど
- スクショ置き場

## 動作確認状況
- ローカルで確認

## 補足
- 相談、参考資料などがあれば
 2023-08-31 09:28:29 +00:00
水本 祐希
b8a41d7b44 Merged PR 370: 修正の実施 
## 概要
[Task2537: 修正の実施](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2537)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
子アカウントの「Issue」ボタンは親アカウントのみ表示されるよう修正
（第1階層→第5階層までたどり、「order history」ボタンを押下時、「Issue」ボタンが表示されていた）

- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）
特になし

## レビューポイント
- 特にレビューしてほしい箇所
ボタン表示制御のコード

- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載

## UIの変更
- Before/Afterのスクショなど
- スクショ置き場

## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
 2023-08-31 08:55:57 +00:00
masaaki
d3aeaea777 Merged PR 306: auth/tokenのAPIでWAFのルールに引っかかることがある問題を解決する 
## 概要
[Task2272: auth/tokenのAPIでWAFのルールに引っかかることがある問題を解決する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2272)

- WAFのルールについて、idTokenに対するREQUEST-942-APPLICATION-ATTACK-SQLIグループのルールを一律除外するよう設定しました

## レビューポイント
- 特になし

## UIの変更
- 特になし

## 動作確認状況
- 無し

## 補足
- 相談、参考資料などがあれば
 2023-08-31 08:35:27 +00:00
makabe.t
e77d8d8af0 Merged PR 374: API IF実装 
## 概要
[Task2521: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2521)

- Worktype追加API IFを実装しOpenAPI定義を更新しました。

## レビューポイント
- パスは認識通りか
- パラメータは適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-08-31 07:42:38 +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