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

357 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
水本 祐希
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
oura.a
cb7ba77bc3 Merged PR 362: API実装（ライセンス注文キャンセルAPI） 
## 概要
[Task2484: API実装（ライセンス注文キャンセルAPI）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2484)

ライセンス注文キャンセルAPIを実装しました。

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

## UIの変更
なし

## 動作確認状況
ローカルでUT、動作確認実施済み

## 補足
なし
 2023-08-31 00:25:13 +00:00
makabe.t
6b91745b2b Merged PR 368: DBマイグレーション 
## 概要
[Task2503: DBマイグレーション](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2503)

- Worktypeテーブルを追加するマイグレーションファイルを追加しました

## レビューポイント
- カラム名は適切か
- カラムの制約は適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
  - up/downできることを確認
 2023-08-30 07:39:19 +00:00
makabe.t
a9d5d926ba Merged PR 369: API IF実装 
## 概要
[Task2504: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2504)

- ワークタイプ一覧取得APIのIFを実装し、openapi.jsonを更新しました

## レビューポイント
- パスは適切か
- プロパティに不足はないか
  - 返却値にidを追加しています

## UIの変更
- なし
## 動作確認状況
- ローカルで確認
 2023-08-30 06:53:58 +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
makabe.t
bba69651fe Merged PR 364: API実装（TypistGroup更新API） 
## 概要
[Task2460: API実装（TypistGroup更新API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2460)

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

## レビューポイント
- DBの更新ロジックに問題はないか
- テストケースは適切か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-08-29 08:10:16 +00:00
makabe.t
5b1c3a0e99 Merged PR 361: API実装（TypistGroup取得API） 
## 概要
[Task2461: API実装（TypistGroup取得API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2461)

- TypistGroup取得APIとテストを実装しました。

## レビューポイント
- DBからの取得ロジックに問題はないか
- テストケースは十分か

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-08-29 06:25:15 +00:00
masaaki
1c4026eff9 Merged PR 366: API IF実装 
## 概要
[Task2496: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2496)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
  - ライセンス発行キャンセルAPIのIFと、使用するリクエスト/レスポンスのtypeを作成しました
- このPull Requestでの対象/対象外
  - 対象外：openapi.jsonの「/licenses/orders/cancel」について、descriptionが追加されていますが本タスクでの追加ではありません。
- 影響範囲（他の機能にも影響があるか）
  - 特になし

## レビューポイント
- 特筆するポイントは無し

## UIの変更
- 無し

## 動作確認状況
- swagger uiよりリクエスト投入できることを確認

## 補足
- 無し
 2023-08-29 02:23:34 +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
湯本 開
89751396bf Merged PR 354: テスト用関数を作成する(Sprint16) 
## 概要
[Task2383: テスト用関数を作成する(Sprint16)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2383)

- WIPレビューで指摘された関数名等を修正
- 汎用関数で問題があった部分を修正
  - データ構築ミスしていた箇所を修正
  - DB的にnullableな所をテスト用関数でも省略できるよう修正
- 各種Account、User作成関数を汎用関数に置き換え

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

## 動作確認状況
- npm run test 成功
 2023-08-28 10:40:55 +00:00