水本 祐希
|
d7bb56af54
|
Merged PR 580: [Sp20]既存APIのログを強化（外部連携API以外）
## 概要
[Task2295: [Sp20]既存APIのログを強化（外部連携API以外）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2295)
- 何をどう変更したか、追加したライブラリなど
誰が操作したのかを追えるようにログを強化
## レビューポイント
- 特にレビューしてほしい箇所
特になし
## 動作確認状況
- ユニットテスト
|
2023-11-17 02:54:18 +00:00 |
|
makabe.t
|
effbfe9d46
|
Merged PR 575: タイピスト割り当てで、未認証のユーザーは割り当てられないようにする
## 概要
[Task2818: タイピスト割り当てで、未認証のユーザーは割り当てられないようにする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2818)
- 以下のAPIについて未認証のタイピストを割り当て内容にユーザー取得処理とテストを修正しました。
- Typist一覧取得
- Workflow追加
- Workflow編集
- ユーザーグループ追加
- ユーザーグループ編集
- 割り当て候補変更
## レビューポイント
- 対応箇所は適切か
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-11-14 02:30:46 +00:00 |
|
makabe.t
|
5c0d5f6476
|
Merged PR 574: 未認証のAuthorIDは選択できないようにする
## 概要
[Task2819: 未認証のAuthorIDは選択できないようにする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2819)
- Author取得APIではメール認証済みのユーザーのみ取得するように修正しました。
- Workflowの追加/編集APIでメール未認証のAuthorを指定して実行するとエラーとなるように修正しました。
- 修正APIに対応したテストケースを追加しました。
## レビューポイント
- 対応箇所は適切でしょうか？
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-11-14 02:10:32 +00:00 |
|
makabe.t
|
8a2ca2b786
|
Merged PR 519: 代行操作で使用するAPIのガード修正＋ログ修正
## 概要
[Task2907: 代行操作で使用するAPIのガード修正＋ログ修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2907)
- 代行操作するAPIについてガードに代行操作許可の設定を追加しました。
- ガードの引数で設定したAPIだけが代行操作許可となるように設定しています。
- APIのバリデーションで意図せぬエラーが発生する問題に対処しました。
※ログは未適用なのでガードの設定だけ確認をお願いします。
## レビューポイント
- ガードを設定する対象は適切か
- ガード内での代行操作許可のロジックはホワイトリスト形式の実装として適切か
- アクセストークンが代行操作用の（代行操作ユーザーIDがある）場合には、ガードで代行操作が許可されているかをチェックするようにしています。
※ログの代行操作への適用は未実装です。
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-11-07 08:41:47 +00:00 |
|
saito.k
|
8ace80de74
|
Merged PR 517: API IF実装（代行操作用トークン生成）
## 概要
[Task2904: API IF](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2904)
- 以下のAPIのIF実装
- POST /api/auth/delegation-token
- POST /api/auth/delegation-access-token
- OpenApiを生成
- 不要なas変換を削除
## レビューポイント
- リクエスト・レスポンスの型は認識通りか
- 代行操作用トークン生成APIの引数のパラメータ名はよさそうか（delegatedAccountId）
- アクセストークン再生成APIについてはガードを付けず、service内で引数に受け取ったリフレッシュトークンを検証し、正しいロール、階層かチェックする方向でよさそうか（通常のアクセストークン再生成と同様に）
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-10-26 01:37:35 +00:00 |
|
saito.k
|
f553bfc95b
|
Merged PR 501: strictNullChecks修正①（accounts,auth,Repositoiesのaccounts,common）
## 概要
[Task2835: 修正①（accounts,auth,Repositoiesのaccounts,common）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2835)
- features
- accounts
- auth
- common
- repositories
- accounts
- 各entity
- Nullableの項目の`@Column`デコレータに`type`を追加しないとTypeORMがエラーになりテストが通らないので追加
- https://qiita.com/maruware/items/08c9ad594e14e4ea1497#%E5%95%8F%E9%A1%8C
## レビューポイント
- コメントとして記載
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認、develop環境で確認など
## 補足
- レビュー完了後、TODOコメント(strictNullChecks対応)は削除します
|
2023-10-19 07:13:56 +00:00 |
|
saito.k
|
d2c2223acf
|
Merged PR 497: strictNullChecks修正③（task,users ,Repositoriesのusers）
## 概要
[Task2837: 修正③（task,users ,Repositoriesのusers）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2837)
- feature
- tasks
- users
- Repositories
- users
## レビューポイント
- 該当箇所にコメントで記載
## 動作確認状況
- ローカルでテストが通ることを確認
## 補足
- 相談、参考資料などがあれば
|
2023-10-16 10:25:45 +00:00 |
|
saito.k
|
d258d569f7
|
Merged PR 483: strictNullCheck修正④（gateways ,notification）
## 概要
[Task2838: 修正④（gateways ,notification）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2838)
- strictNullCheckの対応
- gateways配下
- feartures
- notification
## レビューポイント
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認、develop環境で確認など
## 補足
- 相談、参考資料などがあれば
|
2023-10-16 01:34:28 +00:00 |
|
saito.k
|
370d143c2c
|
Merged PR 473: strictNullCheckの対応を部分的に行う
## 概要
[Task2795: 部分的に修正を行う](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2795)
- strictNullChecks対応
- features
- template
- workflow
- gateways
- adb2c
- repositories
- template
- workflow
## レビューポイント
- entityの修正内容
- nullを追加する項目はあってるか
- adb2cの環境変数を取得している箇所
- getOrThrowで値が取得できなければエラーになる関数があったので使用しています。
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルでテストが通ることを確認
## 補足
- 相談、参考資料などがあれば
|
2023-10-13 04:07:18 +00:00 |
|
makabe.t
|
d48afdbffd
|
Merged PR 468: API実装（ワークフロー削除API）
## 概要
[Task2785: API実装（ワークフロー削除API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2785)
- ワークフロー削除APIとテストを実装しました。
## レビューポイント
- リポジトリの削除ロジックは適切か
- テストケースは適切か
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-10-11 09:14:41 +00:00 |
|
saito.k
|
8e317f5aca
|
Merged PR 478: POデモ修正対応
## 概要
[Task2821: POデモ修正対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2821)
- リテラル修正
- 追加Popupのタイトル
- AuthorIDの初期値
- ワークフロー追加・更新の重複チェックの条件指定を修正
- undefined,nullのままだと条件指定しないという挙動になるためIsNull（）を使用
- 該当ケースをテストに追加
## レビューポイント
- 特になし
## UIの変更
- タスクのなか
## 動作確認状況
- ローカルで確認、develop環境で確認など
## 補足
- 相談、参考資料などがあれば
|
2023-10-10 07:22:00 +00:00 |
|
makabe.t
|
f70e266e85
|
Merged PR 465: API実装（ワークフロー更新API）
## 概要
[Task2776: API実装（ワークフロー更新API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2776)
- ワークフロー編集APIとテストを実装しました。
## レビューポイント
- リポジトリでのチェック処理は適切か
- テストケースは適切か
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-10-06 00:10:47 +00:00 |
|
makabe.t
|
983726eaf3
|
Merged PR 467: API IF実装
## 概要
[Task2784: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2784)
- ワークフロー削除APIのIFを実装しOpenAPI定義を更新しました。
- アプリで使用しない環境変数を削除し、チェック対象から外しました。
- `KEY_VAULT_NAME`
## レビューポイント
- APIのパスは適切か
- 対応する環境変数は足りているか
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-10-05 09:41:20 +00:00 |
|
makabe.t
|
c4c2038e6e
|
Merged PR 458: ワークフロー追加API実装
## 概要
[Task2739: ワークフロー追加API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2739)
- ワークフロー追加APIとテストを実装しました。
## レビューポイント
- リポジトリのチェックロジックは適切か
- テストケースは適切か
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-10-05 00:20:42 +00:00 |
|
makabe.t
|
6baeb0b049
|
Merged PR 463: API IF実装
## 概要
[Task2775: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2775)
- ワークフロー編集APIのIFを実装し、OpenAPIを更新しました。
## レビューポイント
- パスは想定通りか
- パラメータは適切か
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-10-05 00:11:41 +00:00 |
|
makabe.t
|
1cc7a0141d
|
Merged PR 453: ワークフロー一覧取得API実装
## 概要
[Task2736: ワークフロー一覧取得API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2736)
- ワークフロー一覧取得APIとテストを実装しました
## レビューポイント
- リポジトリの取得処理は適切か（リレーションなど）
- ADB2Cからの取得処理は適切か
- サービスでのワークフローの整形処理は適切か
- テストケースは適切か
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-10-03 01:14:18 +00:00 |
|
makabe.t
|
879169c3c7
|
Merged PR 450: ワークフローAPI IF実装
## 概要
[Task2737: ワークフローAPI IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2737)
- Workflow関連で以下のAPIIFを実装し、OpenAPIを更新しました。
- ワークフロー一覧取得API
- ワークフロー追加API
- Author一覧取得API
## レビューポイント
- パラメータは想定通りか
- バリデーションは適切か
- パスは適切か
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-09-28 10:06:51 +00:00 |
|
