masaaki
|
43a65dbb4e
|
Merged PR 201: [Sp12-1着手]アカウント作成時、大文字英語+数字のパスワードを入力した場合、エラーとなる件を対応する
## 概要
[Task2007: [Sp12-1着手]アカウント作成時、大文字英語+数字のパスワードを入力した場合、エラーとなる件を対応する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2007)
- GraphAPIによるアカウント追加時、DisableStrongPasswordのオプションを付与しました。
- サポート問い合わせの結果、以下の挙動であることが分かっています(修正の妥当性の根拠)
・GraphAPIによるユーザ追加時、パスワードポリシはユーザフローに設定した内容ではなく、
Azure AD の既定のパスワードポリシーが評価される
・passwordPolicies属性にDisableStrongPasswordを設定することでAzre AD既定の
パスワードポリシーを無効にしてパスワードを設定することが可能
・DisableStrongPasswordを設定した場合、どのような値でもパスワードとして設定が可能
(画面のチェックが命になる)
・DisableStrongPasswordを設定しても、パスワードの有効期間については変わらない。
Azure AD の既定値(90日)が設定される。
・DisableStrongPasswordの有効範囲は、該当するリクエストのみ。
パスワードリセット時などGUIから操作する際は、ユーザフローのポリシーが評価される。
※サポート問合わせ番号:2306300060000188
- なお、ユーザ追加時のパスワード変更については、GraphAPIを使用していますが
Azure AD の既定のパスワードポリシーは評価されませんでした("TEST1969"というパスワードで登録可能でした)
## レビューポイント
- 修正内容。追加でサポート確認必要な点はないか。
## UIの変更
- 無し
## 動作確認状況
- ローカルで確認。
→アカウント追加時、および「Forgot your password?」からのパスワード変更時、OMDSのパスワードポリシーに準拠したパスワードが設定可能であることを確認。
## 補足
- 無し
|
2023-07-05 02:04:13 +00:00 |
|
maruyama.t
|
ceea4920f6
|
Merged PR 186: API実装(カードライセンス発行API)
## 概要
[Task1992: API実装(カードライセンス発行API)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1992)
- タスク 1992: API実装(カードライセンス発行API)
-カードライセンス発行APIを実装
## レビューポイント
- DB登録時の処理が適切かどうか
## UIの変更
なし
## 動作確認状況
ユニットテスト実施済み
ローカルでの動作確認実施済み
## 補足
なし
|
2023-07-04 08:58:28 +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 |
|
makabe.t
|
dadbc550c6
|
Merged PR 205: API実装(タスクチェックアウトAPI (Author))
## 概要
[Task2009: API実装(タスクチェックアウトAPI (Author))](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2009)
- タスクチェックアウトAPIのAuthorユーザーの動作を実装しました。
- 対応するタスクがあるかを確認だけして特に変更はしていません。
- 対応タスクを追加しました。
## レビューポイント
- 実装内容は認識通りか
- テスト内容に不足はないか
## UIの変更
なし
## 動作確認状況
- ローカルで確認
|
2023-07-04 02:31:39 +00:00 |
|
saito.k
|
8c5f5b61c1
|
Merged PR 199: [12-1着手]タイピスト割り当て変更APIのテスト実装
## 概要
[Task2010: [12-1着手]タイピスト割り当て変更APIのテスト実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2010)
- タイピスト割り当て変更APIのテスト実装
## レビューポイント
- テストケースは足りているか
- テスト名に不足・違和感はないか
- 各テストでチェックしている内容は妥当か
## UIの変更
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-07-04 00:52:28 +00:00 |
|
湯本 開
|
9270dc2c0d
|
Merged PR 200: CI/CDパイプラインで環境変数が取得できない問題を修正する
## 概要
[Task2073: CI/CDパイプラインで環境変数が取得できない問題を修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2073)
- replaceで先頭の改行しか置き換えされていなかった問題を修正
- 本来の動作からして先頭のみ置き換わるのが正常なので、正規表現を指定して全て置き換えに変更
- https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/String/replace
## レビュー対象外
- なぜ今までローカル環境で動作していたのか?
## レビューポイント
- 情報共有
- なぜローカルで動いてしまっていたのかの深掘りはしなくてもよいか
## 動作確認状況
- テストで動作確認
|
2023-07-03 05:32:41 +00:00 |
|
saito.k
|
3304ce75c9
|
Merged PR 194: [Sp12-1着手]カラムにCreated_atがないテーブルにcreated_atを追加する
## 概要
[Task2077: [Sp12-1着手]カラムにCreated_atがないテーブルにcreated_atを追加する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2077)
- 「created_at , created_by , updated_at , updated_by」が存在しないテーブルに対して追加するmigrationファイルを作成
## レビューポイント
- deleted_atについては現状は追加せず、削除系のPBIが来たらその時に考えるとしたいがそれでよいか
- updated_atを更新時に自動で現在日時を入れるように記述したが問題ないか。
- 問題なさそうであれば、ほかテーブルにもその設定を追加する。(別タスクで)
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認(up/down)
## 補足
- 相談、参考資料などがあれば
|
2023-07-03 05:27:42 +00:00 |
|
saito.k
|
1189e676b9
|
Merged PR 178: API実装(タスクチェックアウトAPI (Typist))
## 概要
[Task1996: API実装(タスクチェックアウトAPI (Typist))](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1996)
- タスクチェックアウトAPIのTypist用の処理を実装
- テスト実装
## レビューポイント
- DBアクセス処理に不足はないか
- テスト内容に不足はないか
- テストのチェック方法に問題はないか
- 特に今回の「started_at」はcheckoutした日時を入れるが、それがいつなのかを完全一致でチェックするのは大変なため、checkout前とcheckout後で値が異なっていることを確認するまでのチェックとした
- changeCheckoutPermissionsのPathパラメータにつけたコメントについて
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認
- テストが通ることを確認
## 補足
- 「タスク 1476: [Sp12-1]アクセストークンの寿命を2時間にする」も実施しています
|
2023-07-03 01:09:06 +00:00 |
|
Kentaro Fukunaga
|
f47a686bac
|
Merged PR 196: 音声ファイルアップロード時のアップロード先URLにユーザーIDを追加する
## 概要
[Task2101: 音声ファイルアップロード時のアップロード先URLにユーザーIDを追加する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2101)
- 音声ファイルアップロード先URLにユーザーIDを含めるのを、ODMSCloud側で実施するよう修正しました
- URLの構築はURLオブジェクトを利用するのが安全であるため、リファクタしました。
## レビューポイント
- 修正方針は妥当か
## 動作確認状況
- ローカルで確認
- 返却されたURLをもとに、ODMSCloudツールから音声ファイルアップロードできるところまで確認しました
|
2023-06-30 08:30:29 +00:00 |
|
makabe.t
|
e4bc4776b0
|
Merged PR 184: ローディング表現に関する実装等を行う
## 概要
[Task2015: [Task1895完了後][Sp11-2着手] ローディング表現に関する実装等を行う](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2015)
- 現在実装中のテーブル、ポップアップについてローディング表現を実装しました。
- ローディング中にローディング中を示すぐるぐるを表示
- ローディング中はボタンを非活性にする
## レビューポイント
- 対応箇所は適切か
- 表示内容に問題はないか
## UIの変更
- [Task2015](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/Task2015?csf=1&web=1&e=TmrOXa)
## 動作確認状況
- ローカルで確認
|
2023-06-30 05:51:14 +00:00 |
|
makabe.t
|
f11a9f229f
|
Merged PR 179: git hookの仕組みを実際に導入する&動作確認
## 概要
[Task1981: git hookの仕組みを実際に導入する&動作確認](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1981)
- 今回はGitHookはなしにして使わないコマンドを削除しました。
## レビューポイント
- なし
## UIの変更
なし
## 動作確認状況
- ローカルで確認
|
2023-06-30 05:13:45 +00:00 |
|
Kentaro Fukunaga
|
f36d298383
|
Merged PR 190: DBマイグレーション(PBI1133 音声ファイル/テンプレートファイルをダウンロードしたい)
## 概要
[Task2040: DBマイグレーション](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2040)
文字起こしテンプレートファイルテーブルを作成しました
## レビューポイント
- テーブル名は適切か?
- template_filesか悩んだ。
- 各カラムの型や制約に認識違いはないか?
## 動作確認状況
- ローカルで確認
- migrate up/migrate downでテーブルの追加と削除がされることを確認しました
|
2023-06-30 03:58:56 +00:00 |
|
saito.k
|
8c3e1db63b
|
Merged PR 185: API修正(チェックアウト候補変更)
## 概要
[Task2070: API修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2070)
- DBの検索条件を変更
- ユーザーグループ検索
- 同一アカウント内という条件を追加
- ユーザーグループIDは画面側から渡されるが、そのIDが同じアカウント内のIDかわからないため
- タイピストユーザー検索
- 同一アカウント内でロールが`typist`という条件を追加
- 画面側から渡されるユーザーIDが、同じアカウント内のIDかわからないため
- 画面側から渡されるユーザーIDが、Typistであるかわからないため
- タスク検索
- 同一アカウント内という条件を追加
- 渡される音声ファイルIDが同じアカウント内であるかわからないため
- 自身のロールが`author`だった場合、AuthorIDが一致するタスクという条件を追加(管理者ロールがある場合は条件無し)
## レビューポイント
- 不要な条件追加ではないか
- ロールの型変換処理に問題はないか
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-06-27 07:18:19 +00:00 |
|
湯本 開
|
5be4995d7d
|
Merged PR 158: [改善]ユニットテストの方針・実施方法を検討
## 概要
[Task1978: 検討し、Wikiにまとめる](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1978)
- テスト毎にSQLiteのインメモリモードでDBを作成→データ構築→テスト→DBを破棄をすることで、Service~DBを含んだロジックに対するテストを行う
- サンプルとしてTask一覧のテストを何件か実装
## レビューポイント
- 同様な形式で `features/xxx` 毎にテストを作成する時に問題となりそうなものはないか?
## レビュー対象外
- Wikiにまとめる方の作業は別途依頼予定
## 動作確認状況
- ローカルでテストが動作することを確認
|
2023-06-27 04:25:13 +00:00 |
|
masaaki
|
01a653015b
|
Merged PR 180: API IF実装
## 概要
[Task1989: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1989)
- カードライセンス発行APIのIFを追加しました
## レビューポイント
- 特になし
## UIの変更
- 無し
## 動作確認状況
- ローカルで確認済
## 補足
- 相談、参考資料などがあれば
|
2023-06-26 06:00:25 +00:00 |
|
makabe.t
|
4fb982bf8c
|
Merged PR 177: 外部連携APIの認証ヘッダの形式を修正
## 概要
[Task2054: 外部連携APIの認証ヘッダの形式を修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2054)
- 外部連携APIの認証ヘッダの設定がパラメータに出ているAPIがあったので元のbearerで設定するように修正しました。
- /files/audio/upload-finished
- /files/audio/upload-location
## レビューポイント
- 修正箇所は適切か
- 修正内容は適切か
- OpenAPIとしての内容は認識通りになっているか
## UIの変更
なし
## 動作確認状況
- ローカルで確認
- Swagger
- DevTool
|
2023-06-23 06:30:15 +00:00 |
|
saito.k
|
6a1226c62e
|
Merged PR 165: タイピスト割り当て変更API実装
## 概要
[Task1932: タイピスト割り当て変更API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1932)
- タイピスト割り当て変更APIを実装
- テスト実装
## レビューポイント
- IFのバリデーションを実装したがチェック内容はこれでよさそうか
- DBのデータ取得・更新処理は問題ないか
- DBへアクセスする回数は問題ない程度か
- パスパラメータのバリデーションは問題ないか
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認(swaggerUI,Postman)
## 補足
- 別途sqliteを用いたテストを実装する予定
|
2023-06-23 04:04:01 +00:00 |
|
Kentaro Fukunaga
|
97bb9c4190
|
Merged PR 174: アカウント登録時・ユーザー追加時のパスワードポリシーを、AADB2C標準に合わせる
## 概要
[Task1958: アカウント登録時・ユーザー追加時のパスワードポリシーを、AADB2C標準に合わせる](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1958)
- 利用可能な文字種を修正
- 開発規約上「推奨」となっていたルールを排除
- フロント側の潜在バグ(パスワードの長さチェック)を修正かつリファクタ
## レビューポイント
- 無駄な書き方になっているようなところはないか
## UIの変更
- 無し
## 動作確認状況
- ローカルでアカウント登録時、パスワードポリシー満たしていないものであれば次画面遷移できないことを確認
- ADB2Cユーザーフロー画面でパスワードリセット時に通過するルールと一致していることを確認しました。
## 補足
- 相談、参考資料などがあれば
|
2023-06-22 06:28:57 +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 |
|
湯本 開
|
d3446c10b3
|
Merged PR 173: 脆弱性対応
## 概要
[Task2018: 脆弱性対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2018)
- CI/CDで脆弱性の存在が出ていたものを修正
## レビューポイント
- 情報共有
## 動作確認状況
- テスト・ビルド成功
|
2023-06-20 11:39:08 +00:00 |
|
湯本 開
|
ee72a280c8
|
Merged PR 172: WORK_TYPEでのソートが失敗する問題を修正する
## 概要
[Task2016: WORK_TYPEでのソートが失敗する問題を修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2016)
- WorkTypeによるソート条件構築の実装が漏れていた箇所を修正
## レビューポイント
- 修正内容に問題はないか
## 動作確認状況
- テスト通過、ビルド成功
|
2023-06-20 09:26:02 +00:00 |
|
saito.k
|
756f6155d5
|
Merged PR 171: B2Cserviceのエラーの解消
<内容>
- 不要な処理を削除(削除漏れだった箇所)
|
2023-06-20 08:06:04 +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
|
22bd761d10
|
Merged PR 161: DBマイグレーション
## 概要
[Task1990: DBマイグレーション](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1990)
- 何をどう変更したか、追加したライブラリなど
以下を追加
カードライセンステーブル
カードライセンス発行テーブル
- 影響範囲(他の機能にも影響があるか)
新規追加テーブルのため、なし
## レビューポイント
- 各カラムの物理名が妥当であるか
- NOT NULL制約は正しいか
## UIの変更
テーブルの新規追加のため、なし
## 動作確認状況
- ローカルで確認
server側のターミナルで「sql-migrate up -config=db/dbconfig.yml -env=local」を実行
→DBへ反映されていることを確認。
「sql-migrate down -config=db/dbconfig.yml -env=local」を実行
→1世代巻き戻ることを確認。
## 補足
- インプットと資料としてラフスケッチをご参照ください
|
2023-06-19 04:10:32 +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 |
|
saito.k
|
d75c003b09
|
Merged PR 145: typist名をAzure AD B2Cから取得し表示できるようにする
## 概要
[Task1950: typist名をAzure AD B2Cから取得し表示できるようにする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1950)
- チェックアウト権限にあるTypistのユーザー名をB2Cから取得する
- チェックアウト権限に含まれているuser.externalIdを列挙
- ExternalIdでフィルターをかけてユーザー情報を取得
- B2Cへのリクエスト上限超過時のエラーを制御するために専用エラーを定義
- import文が常に絶対パスで指定されていて、それでテストがこけるので相対パスでインポートするようにvscodeを設定
## レビューポイント
- convert.tsの修正は問題ないか
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認
## 補足
- 取得方法についてはMSに問い合わせ中
|
2023-06-19 00:48:32 +00:00 |
|
湯本 開
|
bb926f9feb
|
Merged PR 164: ユニットテスト実装(タスク一覧取得 | typist)
## 概要
[Task1965: ユニットテスト実装(タスク一覧取得 | typist)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1965)
- Typistによるタスク一覧取得のユニットテスト部分を実装
## レビューポイント
- 現状、ユニットテストとして問題ないか
- レビュー対象外の部分は対象外で問題ないか
## レビュー対象外
- ライセンス周辺のテストが失敗している
## 動作確認状況
- ローカルで確認
|
2023-06-16 12:08:05 +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 |
|
湯本 開
|
2cdcae4924
|
Merged PR 152: API実装(タスク一覧取得 | typist)
## 概要
[Task1865: API実装(タスク一覧取得 | typist)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1865)
- Typist用のRepository呼び出しを追加
## レビュー対象外
- テスト実装(別Taskで対応予定)
## レビューポイント
- SQLの呼び出し回数が増え気味だが問題なさそうか
- Task一覧の取得条件は間違っていなさそうであるか
## 動作確認状況
- ローカルで確認(ユーザーグループ等は各1件程度のデータで確認)
|
2023-06-15 06:42:57 +00:00 |
|
Kentaro Fukunaga
|
4065268820
|
Merged PR 159: Task一覧取得の、文字起こし担当者の型を修正する
## 概要
[Task1987: Task一覧取得の、文字起こし担当者の型を修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1987)
- 型の修正および、既存コードでAssigeeを使うべきでないところをTypistを使うように修正しました
## レビューポイント
- 対応抜け漏れ無いか
## UIの変更
- なし
## 動作確認状況
- ローカルでビルド通ることを確認
- ローカルで既存テストに影響ないことを確認
|
2023-06-14 09:50:58 +00:00 |
|
makabe.t
|
418d568af2
|
Merged PR 153: Task一覧APIのResponseで省略可能でないといけないプロパティが必須になっている箇所を修正する
## 概要
[Task1956: Task一覧APIのResponseで省略可能でないといけないプロパティが必須になっている箇所を修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1956)
[Task1918: タスク一覧、タスク追加APIのコメント修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_workitems/edit/1918)
- Transcription開始日時と終了日時がNULLでも問題ないようにプロパティの型を修正しました。
- タスク一覧の音声ファイル録音時間のコメントも併せて修正しています。
## レビューポイント
- 修正内容は適切か
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-06-14 08:42:48 +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 |
|
saito.k
|
34b242684b
|
Merged PR 154: API実装(タスク一覧取得 | author)
## 概要
[Task1949: API実装(タスク一覧取得 | author)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1949)
- タスク一覧取得(Author用)
- AuthorIDとAccountIDを条件にタスクを取得する処理を実装
- テスト実装
- 成功ケースの時に、Repositoryのメソッドが正しい引数で呼ばれているか確認するテストを追加
## レビューポイント
- テスト実装の内容はこれでよいか
- DBから取得した値をレスポンス用の型に変換する処理はAdminと同様の認識だがあっているか
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認
- 実際にデータを取得して内容が正しいか確認
## 補足
- 相談、参考資料などがあれば
|
2023-06-14 03:57:30 +00:00 |
|
masaaki
|
3220ae0552
|
Merged PR 157: accounts.repository.service.tsの参照があやまっている
## 概要
[Task1960: accounts.repository.service.tsの参照があやまっている](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1960)
- constantsの参照についてsrcフォルダを参照しないように修正しました
## レビューポイント
- 特になし
## UIの変更
- 特になし
## 動作確認状況
- ビルドが成功することを確認
## 補足
- 相談、参考資料などがあれば
|
2023-06-13 09:45:30 +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 |
|
makabe.t
|
85d8221186
|
Merged PR 149: テスト実装(タスク一覧取得|admin)
## 概要
[Task1955: テスト実装(タスク一覧取得|admin)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1955)
- タスク一覧のテストを実装しました。
- adminの時のテストだけです
- テストとテストに必要なmockを実装しています。
## レビューポイント
- テスト項目として不足はないか
- mockの構成に問題はないか
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
- テスト実行
|
2023-06-13 04:23:21 +00:00 |
|
湯本 開
|
f5ce65add8
|
Merged PR 147: Task一覧APIのstatusの入力チェックを行うデコレータを実装する
## 概要
[Task1957: Task一覧APIのstatusの入力チェックを行うデコレータを実装する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1957)
- Statusに該当する文字列が `,` で連結されている場合のみバリデーション通過となるようなデコレータを実装
## レビューポイント
- 実装コストを考慮して汎用的な指定をできるようにしなかったが問題ないか
- 実装コードは妥当な実装になっているか
## 動作確認状況
- ローカルで確認
|
2023-06-12 05:26:26 +00:00 |
|
湯本 開
|
26098cc400
|
Merged PR 130: API実装(タスク一覧取得 | admin)
## 概要
[Task1831: API実装(タスク一覧取得 | admin)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1831)
- AdminがAPIを呼び出したときの処理である、アカウント内のTask情報すべてを取得するロジックを実装
## レビュー対象外
- Transcription開始日時と終了日時が必須プロパティになっている
- [Task一覧APIのResponseで省略可能でないといけないプロパティが必須になっている箇所を修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_workitems/edit/1956)で修正予定のため
- ユニットテストが未実装
- [テスト実装(タスク一覧取得 | admin)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_workitems/edit/1955)で実施予定のため
- API I/Fのstatusのバリデーションがされていない
- [Task一覧APIのstatusの入力チェックを行うデコレータを実装する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_workitems/edit/1957)で実施予定のため
## レビューポイント
1. SQLの発行方法は問題なさそうか
- ~.entityに依存関係を記述( `@OneToOne(...)` や `@OneToMany(...)` )し、TypeORMでの取得時の挙動に任せる方法でよさそうか
2. Permissionテーブル以下も含めて一括でTypeORMによるクエリビルダーに任せたかったが、他の上手くいっている構造と同じ指定をしてもSQL発行時に指定したカラム名を取ってこなくなるという問題が解決できなかったため、2回に分けて取ってくるようにしたが許容可能そうか?
3. 各テーブルでRepositoryを作ってEntityを定義し、他RepositoryからはそのRepository配下のディレクトリを参照するという形を取ってみたが、方針として問題ないか
- 各テーブルを個別に取得したい場合があるかも?という予想があったため
4. Serviceのつくりとして問題はなさそうか(roleによる呼び分けの実装方法など)
5. RepositoryDTO→ControllerDTOの型変換が複雑であったため、専用のconvert.tsというファイルに分離したが、方針として問題なさそうか
## 動作確認状況
- ローカルで確認
|
2023-06-12 03:50:48 +00:00 |
|
makabe.t
|
1082a48fe9
|
Merged PR 139: 認証・認可を宣言的に扱える仕組みを既存処理に適用する
## 概要
[Task1830: 認証・認可を宣言的に扱える仕組みを既存処理に適用する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1830)
- 既存処理について、認証認可をガードで実施するよう以下のAPIに適用しました。
- ユーザー一覧
- GET /users
- ユーザー追加
- POST /users/signup/
- 音声ファイルアップロード先取得
- GET /files/audio/upload-location/
- ライセンス注文
- POST /licenses/orders
ロールガードのテストで定数を使うように修正
## レビューポイント
- 対応APIの抜け漏れはないか
- 対応内容に問題はないか
- 付与権限
- トークン内容取得
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-06-09 08:39:44 +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
|
0907bd28af
|
Merged PR 138: SendGridService内のprivateキーの取得方法を修正する
## 概要
[Task1736: SendGridService内のprivateキーの取得方法を修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1736)
- CryptoServiceの削除
- トークン発行・認証に使用するKeyを環境変数から取得するように修正
- 既存テストの修正
## レビューポイント
- Keyの取得関数の配置場所は妥当か
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルでテストが通ることを確認
- keyを使う処理(ユーザー追加)を実行して、成功することを確認
## 補足
- 修正した箇所のロールチェックはレビュー対象外
- 「タスク 1830: 認証・認可を宣言的に扱える仕組みを既存処理に適用する」で調整してもらう想定
|
2023-06-09 05:35:39 +00:00 |
|
maruyama.t
|
c4aaee07b1
|
Merged PR 137: アクセストークン内に階層情報を含める
## 概要
[Task1925: アクセストークン内に階層情報を含める](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1925)
- アクセストークンとリフレッシュトークンにtierを追加
## レビューポイント
- tierのチェックは必要か
- DBの値をそのまま入れているが問題ないか
**以下は別タスクとして切り出す**
- RoleGuardsを拡張してtierもチェックできるようにする処理を追加し、階層ごとに許可される操作をI/Fの属性として宣言的にチェックできるように修正した箇所について、使いやすそうか。
例)
@UseGuards(RoleGuard.requireds({ roles: ['admin', 'author'] }))
の場合(階層の宣言はしていない場合)許可@UseGuards(RoleGuard.requireds({ roles: ['admin', 'author'], tier [2] }))の場合(階層の宣言をしている場合)ユーザのアカウントの階層を見て、2以上なら許可、2未満なら拒否
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
## 補足
- https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_wiki/wikis/OMDSDictation_wiki/202/%E3%82%A2%E3%82%AF%E3%82%BB%E3%82%B9%E3%83%88%E3%83%BC%E3%82%AF%E3%83%B3
wikiにアクセストークンとリフレッシュトークンについてのページを記載しました。
|
2023-06-08 08:28:10 +00:00 |
|
makabe.t
|
3dca5c604b
|
Merged PR 126: 画面実装(タスク一覧画面)
## 概要
[Task1834: 画面実装(タスク一覧画面)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1834)
- タスク一覧画面を実装しました。
- 表示項目、フィルターを制御できるようにしています
- 表示項目はlocalStorageに保存するようにしています
- ソート順の取得、変更、更新をする処理を追加
## レビューポイント
- デザインの反映は適切か
- フィルター、表示の反映ロジックは適切か
- ソートの反映を追加したので適切か確認をお願いします。
## UIの変更
- [Task1834](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/Task1834?csf=1&web=1&e=zBcQ6i)
## 動作確認状況
- ローカルで確認
- API呼び出しは未検証
|
2023-06-08 08:04:34 +00:00 |
|
saito.k
|
942ac30d8f
|
Merged PR 136: API実装(ソート条件取得API)
## 概要
[Task1923: API実装(ソート条件取得API)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1923)
- ソート条件取得APIを実装
- テスト追加
## レビューポイント
- ソート条件取得APIのレスポンスのデータは足りているか
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-06-07 09:21:20 +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 |
|
saito.k
|
d5e5e59f8c
|
Merged PR 128: API実装(ソート条件変更)
## 概要
[Task1835: API実装(ソート条件変更)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1835)
- ソート条件変更APIを実装
- トークンからB2CのIDを取得→userテーブルからユーザー情報を取得
- ユーザーIDでソート条件テーブルを検索→レコードを更新
- ソート条件テーブルにレコードを作成する
- アカウント作成時の処理にソート条件レコードを作成する処理を追加
- ユーザー追加時にも処理を追加
- テスト修正
## レビューポイント
- APIの引数をチェックする関数をControllerに配置してもよいか
- ソート条件のレコードを作成するタイミングに漏れはないか
- 実装漏れはないか
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-06-06 08:20:21 +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 |
|
maruyama.t
|
890b004a9c
|
Merged PR 124: [PBI1221指摘対応]ライセンス注文APIの中で画面から渡ってきたパラメータのバリデーションを行う
## 概要
[Task1901: [PBI1221指摘対応]ライセンス注文APIの中で画面から渡ってきたパラメータのバリデーションを行う](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1901)
- バリデーションチェック処理をコントローラに追加
- 未使用のimportを削除
- エラーテストで比較するメッセージが間違っていたので合わせて修正
## レビューポイント
- バリデーションのタイミングは適切か
- バリデーションの範囲に過不足がないか
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
## 補足
- licenses.controller.ts以外もセルフチェックを行いました。
|
2023-06-05 07:06:20 +00:00 |
|
saito.k
|
0ca766f309
|
Merged PR 127: Task1856: DBマイグレーション(タスク一覧取得)
## 概要
[Task1856: DBマイグレーション(タスク一覧取得)](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%2010-1?workitem=1856)
- 必要なテーブルを作成
- ユーザーグループテーブル
- ユーザーグループ所属テーブル
- ソート条件テーブル
- チェックアウト権限テーブル
## レビューポイント
- テーブル名は妥当か
- 各カラムの型や制約、コメントは妥当か
- ユーザーグループ名はNOT NULLでよい?
- ソート条件のカラム名はNOT NULLでよい?
- チェック制約を追加したが問題なさそうか
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認
- Migrate up /Migrate downでテーブル追加・削除を確認
- チェック制約で片方のカラムのみ値が入ることを確認(両方値がある or 両方NULLではNG)
## 補足
|
2023-06-02 08:13:33 +00:00 |
|
湯本 開
|
1ced5ef66d
|
Merged PR 123: API実装(I/F実装)
## 概要
[Task1836: API実装(I/F実装)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1836)
- タスクのソート条件更新APIのI/Fを実装
- タスク一覧APIのI/Fを修正
- TODO/XXXですぐ対処可能、または単純に消し忘れているものを対処
## レビューポイント
- API I/Fの修正は妥当な内容であるか
- 特にfilterはカンマ区切りでいいか、指定したものを除外という形式でいいか等
- TODO/XXXの対処は妥当な対処であるか
## 動作確認状況
- ローカルでswagger表示されることを確認
|
2023-06-02 06:12:35 +00:00 |
|