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
|
a1a91207e4
|
Merged PR 308: IF実装（Dealer取得API）
## 概要
[Task2352: IF実装（Dealer取得API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2352)
- ディーラー取得APIのIFを実装
## レビューポイント
- レスポンスとして返却するデータの認識はあっているか
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-08-07 01:25:26 +00:00 |
|
Kentaro Fukunaga
|
e279957392
|
Merged PR 307: 音声ファイルZIPファイルをコンテナ配下に平置きするよう修正する
## 概要
[Task2346: 音声ファイルZIPファイルをコンテナ配下に平置きするよう修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2346)
- 音声ファイルZIPのアップロード/ダウンロードを、コンテナ直下で操作するよう修正しました
- Typistによる中間生成物アップロード対応のため
- ツール側も修正しました。（詳細は補足参照）
- タスク一覧に表示するファイル名から.zipを抜きました（顧客要望）
## レビューポイント
- 対応に抜け漏れ無いか
## 動作確認状況
- ローカルでアップロード/ダウンロードできることを確認しました
- タスク一覧も確認しました
## 補足
- ツール側の修正もあります。
- DL時にファイル名を抜き出しているところがありますが、そこを本修正に合わせて修正してます。
https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/_git/OMDSDictation/pullrequest/25?_a=overview
|
2023-08-07 00:51:07 +00:00 |
|
maruyama.t
|
34ad2e489d
|
Merged PR 304: API-IF実装
## 概要
[Task2338: API-IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2338)
- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）
新規のため、なし
## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカル環境で確認
（POSTMANによる正常時の戻り値nullと各Error）
## 補足
- 相談、参考資料などがあれば
|
2023-08-04 02:08:42 +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 |
|
水本 祐希
|
3ffa45e179
|
Merged PR 277: API実装（注文履歴取得API）
## 概要
[Task2261: API実装（注文履歴取得API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2261)
- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）
## レビューポイント
なし
## UIの変更
## 動作確認状況
-ローカルとポストマンにて確認
モック動作確認は１つのAPIから値を返す単純なもののため行わず。
## 補足
|
2023-08-01 10:10:58 +00:00 |
|
maruyama.t
|
b5183994e1
|
Merged PR 291: 画面実装（注文履歴画面）
## 概要
[Task2260: 画面実装（注文履歴画面）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2260)
- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）
## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認、develop環境で確認など
## 補足
- 相談、参考資料などがあれば
|
2023-08-01 01:55:17 +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 |
|
maruyama.t
|
42db870af0
|
Merged PR 267: API-IF実装（注文履歴）
## 概要
[Task2274: API-IF実装（注文履歴）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2274)
- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
注文履歴取得のAPI-IF実装とopenapi.jsonの生成
- このPull Requestでの対象/対象外
controller以下の処理については
[タスク 2261: 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%2013-2?workitem=2261)
で実装のため対象外
- 影響範囲（他の機能にも影響があるか）
新規機能のため影響なし
## レビューポイント
- エラーレスポンスに過不足ないか
## UIの変更
なし
## 動作確認状況
- ローカルで確認（Postmanでcontrollerの処理が呼べることを確認）
## 補足
- 相談、参考資料などがあれば
|
2023-07-26 08:50:29 +00:00 |
|
masaaki
|
70cb66e517
|
Merged PR 254: API実装（パートナーライセンス情報取得API）
## 概要
[Task2213: API実装（パートナーライセンス情報取得API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2213)
- パートナーライセンス情報取得APIについて実装を行いました
- このPull Request機能としては対象外。気づいたので直した部分
・accounts.service.mock.ts
→整形されていなかったのを修正、不要な参照を削除
・accounts.service.spec.ts
→整形されていなかったのを修正、不要な参照（userInfo）を削除
・license.entity.ts
→テスト実施のため必須項目の定義を追加
・dev-database-rg.json
→developからのマージだが、差分ファイルとして出てしまっている。
（誤ってコミットのstagingを解除してしまい、後からpull操作したことが影響していると思われる）
- 影響範囲（他の機能にも影響があるか）
→特になし
## レビューポイント
- MISOチームメンバについては、「このPull Request機能としては対象外。気づいたので直した部分」の確認をお願いします。整形や不要な参照はMISOで作りこんだものです。
- accounts.repository.service.tsについて、DRYの観点で一部処理をprivateなサブルーチンとしています（getAccountLicenseOrderStatus）。そもそもそういうことをやってよいか、分ける際のやりかた（引数など）について妥当か見ていただきたいです。
- repositoryからの戻り値型の名称についてわかりやすいかどうか。「PartnerLicenseInfoのうちリポジトリから取得した情報」として「PartnerLicenseInfoFromRepository」としていますが、Fromとかつけない方がよいのでは、とか呼び元を意識しすぎた名称になってる、とか気にしています。
## UIの変更
- 無し
## 動作確認状況
- ローカルで確認済
## 補足
- 相談、参考資料などがあれば
|
2023-07-25 02:08:20 +00:00 |
|
oura.a
|
bb8e16897d
|
Merged PR 246: API IF実装
## 概要
[Task2210: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2210)
パートナーライセンス情報一覧画面のIFを実装しました。
## レビューポイント
なし
## UIの変更
なし
## 動作確認状況
ローカルで確認
## 補足
なし
|
2023-07-19 01:59:13 +00:00 |
|
maruyama.t
|
9739942bdf
|
Merged PR 238: Revert 'Revert 'API実装（パートナーアカウント追加API）''
## 概要
[Task2157: API実装（パートナーアカウント追加API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2157)
- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
パートナーアカウント追加APIを実装しました。
- このPull Requestでの対象/対象外
認証メール送信後のフローは既存機能のため対象外
- 影響範囲（他の機能にも影響があるか）
既存のaccounts.service.spec.tsのテスト
## レビューポイント
- 特にレビューしてほしい箇所
エラー判定に過不足ないか
## UIの変更
なし
## 動作確認状況
- ローカルで確認
Azureに管理者ユーザが追加されたこと、認証メールが送信されてくることを確認。
（対象外だが、認証後に追加されたアカウントでログインできることを確認）
## 補足
- 相談、参考資料などがあれば
一度間違えてCompleteにしてしまったので、
Reverts !225
差分を戻すプルリクをCompleteにして出し直させていただいております。
Reverts !237
|
2023-07-18 05:01:17 +00:00 |
|
maruyama.t
|
f8a3b1798c
|
Merged PR 237: Revert 'API実装（パートナーアカウント追加API）'
## 概要
[Task2157: API実装（パートナーアカウント追加API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2157)
- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
パートナーアカウント追加APIを実装しました。
- このPull Requestでの対象/対象外
認証メール送信後のフローは既存機能のため対象外
- 影響範囲（他の機能にも影響があるか）
既存のaccounts.service.spec.tsのテスト
## レビューポイント
- 特にレビューしてほしい箇所
エラー判定に過不足ないか
## UIの変更
なし
## 動作確認状況
- ローカルで確認
Azureに管理者ユーザが追加されたこと、認証メールが送信されてくることを確認。
（対象外だが、認証後に追加されたアカウントでログインできることを確認）
## 補足
- 相談、参考資料などがあれば
Reverts !225
|
2023-07-14 04:19:24 +00:00 |
|
maruyama.t
|
3ddb010675
|
Merged PR 225: API実装（パートナーアカウント追加API）
## 概要
[Task2157: API実装（パートナーアカウント追加API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2157)
- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
パートナーアカウント追加APIを実装しました。
- このPull Requestでの対象/対象外
認証メール送信後のフローは既存機能のため対象外
- 影響範囲（他の機能にも影響があるか）
既存のaccounts.service.spec.tsのテスト
## レビューポイント
- 特にレビューしてほしい箇所
エラー判定に過不足ないか
## UIの変更
なし
## 動作確認状況
- ローカルで確認
Azureに管理者ユーザが追加されたこと、認証メールが送信されてくることを確認。
（対象外だが、認証後に追加されたアカウントでログインできることを確認）
## 補足
- 相談、参考資料などがあれば
|
2023-07-14 04:15:21 +00:00 |
|
masaaki
|
9c0f457e9f
|
Merged PR 220: [12-2]RoleGuradにTierに対するチェックを実装する
## 概要
[Task1951: [12-2]RoleGuradにTierに対するチェックを実装する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1951)
- 階層の指定について宣言的にできるように対応しました。
- 階層の定数について配列化しました。
（分かりやすさとroleguards側の実装しやすさのため）
- 使用していない宣言がいくつか見られたので、気づいた範囲で削除しました。
- 影響範囲（他の機能にも影響があるか）
- これまで処理内で階層のチェックを行っていた箇所について、宣言的にチェックするよう修正しました。
修正対象の洗い出しについては「補足」参照
- 階層のみチェックする場合を考慮し、既存のrolesに対するチェックを任意指定にしています。
これに伴い、rolesが指定されなかった場合を考慮して修正を行っています。
## レビューポイント
- 各コントローラを見ていただき、階層チェックのやり方について使いづらさがないか
## UIの変更
- 無し
## 動作確認状況
- ローカルで確認済
## 補足
- 修正対象の洗い出しは以下の通り実施しています。
https://ndstokyo.sharepoint.com/:u:/r/sites/Piranha/Shared%20Documents/General/OMDS/%E3%83%A9%E3%83%95%E3%82%B9%E3%82%B1%E3%83%83%E3%83%81/PBI1189_%5B%E9%9A%8E%E5%B1%A4%E5%91%A8%E3%82%8A%E6%95%B4%E7%90%86%5D%E7%AC%AC%E4%B8%80%EF%BD%9E%E7%AC%AC%E5%9B%9B%E9%9A%8E%E5%B1%A4%E3%81%A8%E3%81%97%E3%81%A6%E3%80%81%E3%83%91%E3%83%BC%E3%83%88%E3%83%8A%E3%83%BC%E5%90%91%E3%81%91Web%E3%82%B5%E3%83%BC%E3%83%93%E3%82%B9%E3%81%AB%E3%83%AD%E3%82%B0%E3%82%A4%E3%83%B3%E3%81%97%E3%81%9F%E3%81%84.drawio?csf=1&web=1&e=h3Sbf6
|
2023-07-12 02:06:16 +00:00 |
|
maruyama.t
|
c19adde702
|
Merged PR 221: API IF実装(パートナーアカウント追加API）
## 概要
[Task2153: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2153)
- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
パートナーアカウント追加APIのIFを追加しました
## レビューポイント
特になし
## UIの変更
無し
## 動作確認状況
ローカルで確認済
## 補足
- 相談、参考資料などがあれば
|
2023-07-11 02:29:08 +00:00 |
|
makabe.t
|
4d52da8d18
|
Merged PR 163: タイピストグループ取得API実装
## 概要
[Task1946: タイピストグループ取得API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1946)
- ユーザーグループを取得するAPIを実装しました。
- 別で対応予定のテストをコメントアウトしています
## レビューポイント
- 取得内容に問題ないか
- ライセンス取得のテストをコメントアウトしているが問題ないか
## UIの変更
なし
## 動作確認状況
- ローカルで確認
|
2023-06-22 00:36:37 +00:00 |
|
makabe.t
|
ef17e4ff0b
|
Merged PR 167: タイピストユーザー取得API実装
## 概要
[Task1931: タイピストユーザー取得API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1931)
- アカウント内のタイピストユーザーを取得する処理を実装しました。
## レビューポイント
- ADB2Cからのユーザー取得、マージは適切か
## UIの変更
なし
## 動作確認状況
- ローカルで確認
|
2023-06-20 06:16:19 +00:00 |
|
maruyama.t
|
f0e0d5e6b1
|
Merged PR 162: API IF修正（LicenseSummaryInfo）
## 概要
[Task1961: API IF修正（LicenseSummaryInfo）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1961)
- 何をどう変更したか、追加したライブラリなど
API実装時に一時的にLicenseSummaryInfo2と命名していた定義を、LicenseSummaryInfoに修正。
それに伴い、既存のLicenseSummaryInfoはレスポンスに直接記載するよう修正。
openapi.jsonを再生成し、画面側のパラメータの取得の記載を修正。
- このPull Requestでの対象/対象外
storageSizeとusedSizeの取得はPBI対象外
- 影響範囲（他の機能にも影響があるか）
なし
## レビューポイント
- クライアント側に対する修正漏れがないか。
## UIの変更
なし
## 動作確認状況
- ローカルでAPIの戻り値の確認と、画面にその値が表示されていることを確認。
## 補足
- 相談、参考資料などがあれば
|
2023-06-19 04:06:44 +00:00 |
|
maruyama.t
|
fd69541e1a
|
Merged PR 142: API実装（第五階層用ライセンス情報取得API）
## 概要
[Task1846: API実装（第五階層用ライセンス情報取得API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1846)
- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
accounts.controller.tsからaccountService.getLicenseSummaryを呼び出す。
一度のトランザクションで処理を行うよう、serviceとrepositoryをリファクタリング。
license.entity.tsにはライセンス系のテーブルで不足していたエンティティを追加。
- このPull Requestでの対象/対象外
Storage Sizeの値はPBI1203では対象外のため0固定
Used Sizeの値はPBI1203では対象外のため0固定
LicenseSummaryInfo2と定義している個所は、別タスクで修正します。
[タスク 1961: API IF修正（LicenseSummaryInfo）](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%2011-1?workitem=1961)
- 影響範囲（他の機能にも影響があるか）
新規追加のため、なし
## レビューポイント
- 特にレビューしてほしい箇所
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
PostmanでAPI実行。
各ライセンス数値が期待通りの結果であることを確認。
## 補足
テスト内容は、添付のテストデータを参照ください。
[テストデータ.xlsx](https://dev.azure.com/ODMSCloud/6023ff7b-d41c-4fa7-9c6f-f576ba48c07c/_apis/git/repositories/302da463-a2d7-40f9-b2bb-6e8edf324fa9/pullRequests/142/attachments/%E3%83%86%E3%82%B9%E3%83%88%E3%83%87%E3%83%BC%E3%82%BF.xlsx)
|
2023-06-15 08:56:03 +00:00 |
|
Kentaro Fukunaga
|
42bc458632
|
Merged PR 155: API IF実装
## 概要
[Task1930: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1930)
- Typist一覧取得API、TypistGroup一覧取得API、チェックアウト候補変更APIの3本のAPIIFを実装しました。
- 既存のTask系APIで使用していた Typist クラスの名前をAssigneeに変更しました
- npm run formatでフォーマットした内容も含まれています。
## レビューポイント
- IFはラフスケッチで合意した内容に沿っているか
- 既存のTask系APIのTypistクラスの名前変更に関して、修正抜け漏れはないか
- チェックアウト候補変更APIのリクエスト/レスポンスのクラス名は適切か
## UIの変更
- なし
## 動作確認状況
- ローカルでPostmanにてAPIIFの値が返ることを確認しました
## 補足
- 特になし
|
2023-06-14 07:55:22 +00:00 |
|
oura.a
|
201b6be260
|
Merged PR 150: API実装（自アカウント情報取得API）
## 概要
[Task1954: API実装（自アカウント情報取得API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1954)
・アクセストークンからユーザ情報を取得する処理を追加。
（本タスクで返しているのはアカウントIDのみ）
・本タスクのレビュー対象は以下となります。
・accounts.servicce
getMyAccountメソッド
・accounts.controller
@Get('me')
・accounts.service.spec
'アクセストークンからユーザ情報を取得する'
'ユーザ情報が取得できない場合エラーとなる'
・別タスクで作成したテストコードを取り込むためマージしたところ、controllerの別タスク箇所でエラーが出ていたので、一時的にコメントアウトしています。本タスクの挙動には影響ありません。
## レビューポイント
処理に過不足がないか。
エラーハンドリングが適切か。
## UIの変更
なし
## 動作確認状況
ローカルでユニットテスト実施済み。
ローカルで外部からAPIを実行し、動作確認済み。
## 補足
なし
|
2023-06-13 07:39:10 +00:00 |
|
masaaki
|
f126b0976e
|
Merged PR 146: API IF実装（accounts/me復活）
## 概要
[Task1953: API IF実装（accounts/me復活）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1953)
- accounts/meのIFを追加しました
※別タスク「タスク 1837: API I/F実装」でレビューまでしてもらったものの、階層をアクセストークンに含める方針としたため削除としたものを復活させています。
- prettierがエラーを出力するようになっていたので、以下URLを参考にsettings.jsonを更新しています
https://qiita.com/TellMin/items/a634149730b777e2e6d0
- このPull Requestでの対象外
・openapi.jsonのL952とL2094については本PBI対象外になります
## レビューポイント
- 特にありません
## UIの変更
- 無し
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-06-09 07:34:49 +00:00 |
|
saito.k
|
9441049201
|
Merged PR 134: API修正（IF）
## 概要
[Task1922: API修正（IF）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1922)
- タスク取得APIの引数を修正
- directionとparamNameを追加
- 定数を文字列配列からオブジェクトに変更
## レビューポイント
- 引数の型を自作のものに修正したが問題ないか。
- Pull Request 128: API実装（ソート条件変更）で議論中
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-06-07 07:45:30 +00:00 |
|
masaaki
|
cfca98e53b
|
Merged PR 133: API I/F実装
## 概要
[Task1837: API I/F実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1837)
- ラフスケッチに基づき、「アカウント情報取得API」「ライセンス集計取得API（ラフスケッチ時は「第五階層用ライセンス情報取得API」）」のI/F実装とopenapi化を行いました。
## レビューポイント
- 管理者のみのためUseGuardsを使用しています。実装内容について問題ないか特に確認いただきたいです
## UIの変更
- 無し
## 動作確認状況
- ローカルで確認済
## 補足
- 無し
|
2023-06-06 07:00:11 +00:00 |
|
makabe.t
|
e4f84f78ba
|
Merged PR 121: リフレッシュトークンの発行方法・発行内容が不適切な問題の修正
## 概要
[Task1904: リフレッシュトークンの発行方法・発行内容が不適切な問題の修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1904)
セキュリティ・権限管理の要であるリフレッシュトークンのロール設定に関する修正ですので、開発メンバー全員にご確認いただき、変更内容について認識を合わせていただきたいです。
- リフレッシュトークンの発行で使用するロールがクライアントから送られた値をそのまま利用する実装になっていましたので、APIのロールの取り扱いを修正しました
- ユーザー追加APIで想定している（author/typist/none）のみを受け付けるようにする
- 想定外のロールの場合は400エラー
- クライアントからも想定通りのロールを送信するようロールの定数値を修正
- リフレッシュトークンを発行する際にDBの値をそのまま使わず、想定値のどれかを判断して定数を設定する
- DBに登録されているロールが想定外の文字列の場合は500エラーとなる
## レビューポイント
- トークンのロール設定が安全にできるようになっていることをメンバー全員で合意
- APIのパラメータを受け付ける際にIsInで想定ロール文字列のみを受け付けるようにしているが問題はないか
- リフレッシュトークンの生成時に直接DBのロールを設定しないようにしたが、トークンのロール設定として問題はないか
- 意図しないトークンへの権限の付与などは起こりえないか。
- クライアントからadminなどのユーザー追加時に想定していない権限を付与しようとしてもAPIではじかれるか
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
## 補足
- ユーザー追加時に登録されるロールが変更となったのでマージ後に利用する場合は再度の登録をお願いします。
|
2023-06-02 04:55:28 +00:00 |
|
oura.a
|
24a784b02f
|
Merged PR 115: [MISOチーム] typeが「None」のステータスについてアカウント追加時（none）とユーザ追加時（None）で表記ゆれがある
## 概要
[Task1766: [MISOチーム] typeが「None」のステータスについてアカウント追加時（none）とユーザ追加時（None）で表記ゆれがある](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1766)
タスク 1766: [MISOチーム] typeが「None」のステータスについてアカウント追加時（none）とユーザ追加時（None）で表記ゆれがある
・アカウント追加時のtypeを「none」から「None」に修正。
## レビューポイント
なし
## UIの変更
なし
## 動作確認状況
ローカルでアカウント登録APIを実行し、DBへの登録内容が「None」に修正されていることを確認。
## 補足
なし
|
2023-05-25 01:37:03 +00:00 |
|
makabe.t
|
0099614a5f
|
Merged PR 58: API実装（アカウント登録/Azure AD B2C）
## 概要
[Task1550: API実装（アカウント登録/Azure AD B2C）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1550)
- アカウント登録APIでAzure ADB2Cにユーザを登録する処理を追加しました。
- GraphAPIで登録処理をしています。
## レビューポイント
- ADB2Cへ登録する情報は認識通りか
- ADB2CへのGraphAPI接続のためにADB2Cテナントにアプリを追加しているが認証として問題ないか。
- 環境変数にアプリの情報を設定しています
- ADB2C_TENANT_ID=xxxxxxxx
- ADB2C_CLIENT_ID=xxxxxxxx
- ADB2C_CLIENT_SECRET=xxxxxxxx
## UIの変更
無し
## 動作確認状況
- ローカルで確認
- ADB2C、DBに設定項目が追加されていることを確認
|
2023-04-05 09:22:50 +00:00 |
|
湯本 開
|
9f5252baf8
|
Merged PR 56: API実装（アカウント登録）
## 概要
[Task1496: API実装（アカウント登録）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1496)
- アカウント登録のAzureAD B2Cへのユーザー作成部分以外について実装
- migration SQLに足りない要素があった箇所を修正(default追加)
- 各種環境変数を追加
- 秘密鍵/公開鍵を取得する方法を環境変数に変更（KeyVaultからWebAppsの機能で環境変数へ流し込む想定）
## レビューポイント
- 実装方法として問題がありそうな箇所が存在しないか
- 可読性の低い箇所が存在しないか
- Moduleの分け方、つなげ方などは問題ないか
- ラフスケッチと違い、Account作成と管理者ユーザー作成を同一トランザクションで行うよう修正したが問題ないか
## 動作確認状況
- メール送信以外はローカルで確認、メール送信部分は未確認
|
2023-03-29 03:55:44 +00:00 |
|
湯本 開
|
dfd9abc1c3
|
Merged PR 54: API実装（I/F）
## 概要
[Task1494: API実装（I/F）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1494)
- POST /accounts のAPIを実装
- POST /users/confirm のAPIを実装
- 上記APIからopenapiを実装
## レビューポイント
- ラフスケッチ時から変更になった箇所があるが問題ないか
- ディーラーIDは省略可能かつIDを指定するべきなのでnumber?に型を変更
- 管理者ユーザー用に同意済み利用規約バージョンを受け付けるようにした
- reCAPTCHAを想定して事前にreCAPTCHA用トークンを受け付けるようにした
## 動作確認状況
- openapiが生成されることを確認
|
2023-03-23 07:56:18 +00:00 |
|
