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

950 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
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
水本 祐希
6e3c9d5c81 Merged PR 244: Partner一覧仮画面 
## 概要
[Task2229: Partner一覧仮画面](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_workitems/edit/2229/)

- 何をどう変更したか、追加したライブラリなど
・Partner一覧の仮画面を作成しました。
　ボタン追加とアカウント階層の画面表示
- このPull Requestでの対象/対象外
・仮画面なので、レイアウトに関しては対象外です。
・46行目にonClickをコメントアウトしています。
　後々使用すると思い、わざと消していないですが、レビューで消すか否かを
　コメントいただければと思います。

## レビューポイント
- 特にレビューしてほしい箇所
・ヘッダのpartnerタブ押下により画面が表示できること
・第1～3階層で「Add Account」ボタンが表示されるか
・第4階層で「Add Account」ボタンが非表示かどうか
・第1～4階層で正しく階層が表示されるか
　例：第１階層でログイン→Tier:１と表示

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

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

## 補足
- 相談、参考資料などがあれば
 2023-07-19 00:43:09 +00:00
oura.a
fabcdc16f5 Merged PR 223: ヘッダ修正（partnerタブ追加） 
## 概要
[Task2154: ヘッダ修正（partnerタブ追加）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2154)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
・Partnerヘッダータグを追加
- この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/Task2154?csf=1&web=1&e=jv7CDb

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

## 補足
- 相談、参考資料などがあれば
 2023-07-18 06:13:23 +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
湯本 開
f658201d5a Merged PR 240: pipelineの構造をexportしてgit管理する 
## 概要
[Task2205: pipelineの構造をexportしてgit管理する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2205)

- PipelineをYAML形式でExportし、git管理に追加

## レビューポイント
- 配置するディレクトリ名などは適切であるか
- JSON形式だと未設定の項目の情報なども出て10倍弱のサイズになってしまうため、可読性の観点でJSONでなくYAMLを選択したが問題ないか
 2023-07-18 03:35:58 +00:00
oura.a
3df4dc557b Merged PR 236: [前PBI残作業]画面デザインを依頼する 
## 概要
[Task2203: [前PBI残作業]画面デザインを依頼する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2203)

タスク 2203: [前PBI残作業]画面デザインを依頼する
テキストボックスを5分割した際のデザインをデザイナーさんに依頼し、取り込みました。

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

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

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

## 補足
なし
 2023-07-18 00:59:57 +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
saito.k
8c8fa4dc6b エラーのステータスコードを修正 2023-07-13 16:50:47 +09:00
saito.k
14627ad7e9 Merged PR 231: タスクキャンセルAPI実装 
## 概要
[Task2120: タスクキャンセルAPI実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2120)

- タスクキャンセルAPIを実装
- テスト実装

## レビューポイント
- Adminの時とTypistの時の実行条件はあっているか
- テストケースは足りているか

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

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

## 補足
- 中断、チェックインAPIの実装が入っていますが、それに関しては別PRでレビューしていただいているので対象外とさせてください
 2023-07-13 06:55:12 +00:00
oura.a
b4026c1460 Merged PR 234: [dev動作確認指摘の修正]異常系メッセージ表示 
## 概要
[Task2194: [dev動作確認指摘の修正]異常系メッセージ表示](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2194)

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

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-07-13 02:02:39 +00:00
oura.a
9528bb1ad6 Merged PR 226: テキストボックス分割実装 
## 概要
[Task2168: テキストボックス分割実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2168)

タスク 2168: テキストボックス分割実装
ライセンスキー入力のボックスを5分割に変更。
細かい画面レイアウトについては後々デザイナーさんに依頼することになるかと思いますので、レビュー対象外でお願いします。

## レビューポイント
入力時の挙動に過不足がないか。

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

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

## 補足
なし
 2023-07-13 00:02:40 +00:00
saito.k
e4ba5229df Merged PR 224: タスク中断API実装 
## 概要
[Task2119: タスク中断API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2119)

- タスク中断APIの処理を実装
- テスト実装
- チェックアウト処理のエラーチェックを修正
  - エラーが発生したときに何が原因なのかログに出力するように修正

## レビューポイント
- チェックインと同様の処理部分を切り出さずにそのまま実装したが、スマートに切り出せる方法はありそうか。
  - チェックインとサスペンドの取得処理だけ切り出してもあまりうれしくない（この２つ以外のところで使えなさそう）
    - キャンセルでは使えるかもだけど
  - チェックアウトやその他のメソッドのタスク取得とまとめようとすると、チェック内容や検索条件に差異がありきれいに切り出すことができなさそう。
    - まとめようとすると、引数が膨大でチェック項目も複雑になる

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

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

## 補足
- コンフリクトが発生しているが、現在のdevelopを取り込むとエラーになるので解消し次第、取り込んで競合解決します
- チェックインAPIの実装も入っていますが、そこは別のPRでレビューを行っているので対象外となります
 2023-07-12 08:27:46 +00:00
saito.k
7be4da29bb Merged PR 219: タスクチェックインAPI実装 
## 概要
[Task2118: タスクチェックインAPI実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2118)

- チェックインAPIの処理を実装
- テスト実装

## レビューポイント
- 文字起こし担当であるかどうかをチェックする方法についてどちらが良いか
  - チェックアウト権限テーブルで、タスクに紐づく割り当て候補を確認する（チェックアウトした時点で個人指定のみとなっているはず）
  - タスク情報にあるtypist_user_idで確認する
- テストケースは足りているか

## UIの変更

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

## 補足
- 相談、参考資料などがあれば
 2023-07-12 02:57:53 +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
makabe.t
6f92313e9a Merged PR 232: Typistの音声ファイル取得で失敗する件の調査・修正 
## 概要
[Task2172: Typistの音声ファイル取得で失敗する件の調査・修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2172)

- 音声ファイルダウンロードURLの取得時にTypistの場合、音声ファイルが見つからない問題を修正しました。
  - ファイルパスを構築する際に実行ユーザーIDを材料にしていたのでTypistの場合が存在しないパスとなっていました。
  - ファイル所有者IDから構成するように修正しました。

## レビューポイント
- ファイルパスの作り方に問題はないか

## UIの変更
なし

## 動作確認状況
- ローカルで確認
 2023-07-12 00:06:14 +00:00
oura.a
883224c914 Merged PR 229: 再コミット_画面実装（カードライセンス取り込みPU） 
## 概要
[Task2171: 再コミット_画面実装（カードライセンス取り込みPU）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2171)

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

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-07-11 07:02:33 +00:00
oura.a
3584a65682 Merged PR 228: Revert "Merged PR 218: 画面実装（カードライセンス取り込みPU） 
Revert "Merged PR 218: 画面実装（カードライセンス取り込みPU）

## 概要
[Task2161: 画面実装（カードライセンス取り込みPU）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2161)

タスク 2161: 画面実装（カードライセンス取り込みPU）
カードライセンス取り込みポップアップを実装しました。
以下の項目については別タスクへ切り出しての対応とし、本タスクでは対象外とさせてください。
・テキストボックスへの4文字区切りの入力
・テキストボックスの文字数制限
・改行コード入力時の挙動（バーコードリーダー対応）

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

## UIの変更
https://ndstokyo.sharepoint.com/:i:/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/Task2161/%E3%82%AB%E3%83%BC%E3%83%89%E3%83%A9%E3%82%A4%E3%82%BB%E3%83%B3%E3%82%B9%E5%8F%96%E3%82%8A%E8%BE%BC%E3%81%BFPU.PNG?csf=1&web=1&e=cRtMX3

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

## 補足
なし"

Reverted commit `9a66ca02`.
 2023-07-11 05:50:51 +00:00
oura.a
9a66ca027a Merged PR 218: 画面実装（カードライセンス取り込みPU） 
## 概要
[Task2161: 画面実装（カードライセンス取り込みPU）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2161)

タスク 2161: 画面実装（カードライセンス取り込みPU）
カードライセンス取り込みポップアップを実装しました。
以下の項目については別タスクへ切り出しての対応とし、本タスクでは対象外とさせてください。
・テキストボックスへの4文字区切りの入力
・テキストボックスの文字数制限
・改行コード入力時の挙動（バーコードリーダー対応）

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

## UIの変更
https://ndstokyo.sharepoint.com/:i:/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/Task2161/%E3%82%AB%E3%83%BC%E3%83%89%E3%83%A9%E3%82%A4%E3%82%BB%E3%83%B3%E3%82%B9%E5%8F%96%E3%82%8A%E8%BE%BC%E3%81%BFPU.PNG?csf=1&web=1&e=cRtMX3

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

## 補足
なし
 2023-07-11 05:06:10 +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
saito.k
869880c204 Merged PR 210: API実装（テンプレートファイルDL元） 
## 概要
[Task2039: API実装（テンプレートファイルDL元）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2039)

- テンプレートファイルダウンロードURL取得API＆テストを実装しました。
  - 構成は音声ファイルDLと同様でBlobストレージアクセス部分は共通のメソッドを使用しています。
  - テンプレートファイルは手動でBlob、DBに追加して確認しています。

## レビューポイント
- 音声ファイルとほどんど同じ処理だが内容に問題はないか
- 共通部分の構成に問題はないか
- テスト項目に問題はないか

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-07-11 01:01:37 +00:00
maruyama.t
4bbd9b371d Merged PR 214: 第五階層の画面レイアウトを最新の状況にする 
## 概要
[Task2034: 第五階層の画面レイアウトを最新の状況にする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2034)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
クライアント側の画面表示内容の修正。
AccountLock→StorageAvailable
サーバ側の処理でAccountLockと定義している変数名を適切な名称に修正。
- 影響範囲（他の機能にも影響があるか）
openapi.json変更に伴い、api.tsがdev環境の最新版になっています。

- 本PBIの対象外
第五階層用ライセンス情報画面以外の機能
scssファイルの修正箇所は、次回デザイナさんの修正時に正式に展開される想定です。

## レビューポイント
- 特にレビューしてほしい箇所

## 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/Task2034?csf=1&web=1&e=gfSwl3
## 動作確認状況
ローカル環境で画面の表示内容が変更されていることを確認

## 補足
- 相談、参考資料などがあれば
 2023-07-10 04:03:03 +00:00
oura.a
365e4266e6 Merged PR 209: API実装（カードライセンス取り込みAPI） 
## 概要
[Task2096: API実装（カードライセンス取り込みAPI）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2096)

- タスク 2096: API実装（カードライセンス取り込みAPI）
カードライセンス取り込みAPIを実装しました。

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

## UIの変更
なし

## 動作確認状況
ユニットテスト実施済み
ローカルでの動作確認実施済み

## 補足
なし
 2023-07-07 09:02:44 +00:00
makabe.t
773c8894e7 Merged PR 208: API実装（音声ファイルDL元） 
## 概要
[Task2038: API実装（音声ファイルDL元）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2038)

- 音声ファイルのダウンロードURL取得API&テストを実装しました。
  - SASトークンの有効期限を2時間にしています。
- タスク作成APIでURLのSASトークンを取り除く処理を追加しています。

## レビューポイント
- URLの生成に問題はないか
- テストのためにmodule生成処理を追加したが問題ないか
- タスク作成APIでのURL処理は認識通りか

※対象外：テンプレートファイル関連
以下はコード整形による変更なので対象外
- licenses.repository.module.ts
- tasks.service.spec.ts

## UIの変更
なし

## 動作確認状況
- ローカルで確認
 2023-07-07 06:57:29 +00:00
maruyama.t
766c995ceb Merged PR 217: [PBI1211デモ指摘]カードライセンス発行PU，生成中は×を押下できないようにする。 
## 概要
[Task2151: [PBI1211デモ指摘]カードライセンス発行PU，生成中は×を押下できないようにする。](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2151)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
カードライセンス発行中（サーバの処理中）はポップアップを閉じれないよう修正しました。
また、ページから離れようとした場合にはダイアログ確認を行うよう修正しました。

- この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/Task2151?csf=1&web=1&e=Astuk1
## 動作確認状況
- ローカルで確認
（Chrome,Edgeで確認）

## 補足
- 相談、参考資料などがあれば
 2023-07-07 02:01:59 +00:00
maruyama.t
48b8cdc3e3 Merged PR 216: カードライセンス発行PU,処理成功時のメッセージを汎用メッセージから専用メッセージに変更する。 
## 概要
[Task2130: カードライセンス発行PU,処理成功時のメッセージを汎用メッセージから専用メッセージに変更する。](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2130)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
カードライセンス発行時のメッセージを修正
- この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/Task2130?csf=1&web=1&e=Tu1y6N
## 動作確認状況
- ローカルで確認

## 補足
- 相談、参考資料などがあれば
 2023-07-06 07:03:58 +00:00
masaaki
4dd2446109 Merged PR 211: Create押下後に処理が成功するまでloadingを表示させる 
## 概要
[Task2129: Create押下後に処理が成功するまでloadingを表示させる](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2129)

- ボタン押下時に処理中であることを示すイメージを実装しました

## レビューポイント
- licenseCardIssueSlice.tsでの、fulfilled時のloading解除タイミングについて。
　fulfilled時、csvのblobイメージ作成→ダウンロード実施、という処理を実施しています。
　同時実行を避ける意味で、ダウンロード実施の後（fulfilled処理の最後）でloading解除としていますが、タイミングとして問題ないか。
　（とはいえ、blob作成中に解除する理由は浮かばないので、これで問題ないと考えています。何かあれば、くらいの感覚です）

## UIの変更
- 無し

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

## 補足
- 相談、参考資料などがあれば
 2023-07-05 05:44:15 +00:00
oura.a
3f9ede7aed Merged PR 197: [備忘] 実装上、取り込みを「import」の英単語を使っている箇所を「activate」に変更する 
## 概要
[Task2003: [備忘] 実装上、取り込みを「import」の英単語を使っている箇所を「activate」に変更する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2003)

ライセンス取り込み部分の文言・内部の変数等を「import」→「activate」に変更

## レビューポイント
過不足がないか。

## UIの変更
「カードライセンス取り込みボタン」の文言が変更

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

## 補足
なし
 2023-07-05 04:12:40 +00:00
oura.a
2263412120 Merged PR 195: API IF実装 
## 概要
[Task2094: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2094)

-カードライセンス取り込みAPIのIFを実装。

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

## UIの変更
- なし

## 動作確認状況
- ローカルでビルドエラーが出ないことを確認

## 補足
- なし
 2023-07-05 02:36:27 +00:00
oura.a
24e52938c0 Merged PR 206: ライセンス数入力のテキストボックスに「.」「+」「-」が入力できてしまう件の修正 
## 概要
[Task2104: ライセンス数入力のテキストボックスに「.」「+」「-」が入力できてしまう件の修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2104)

replaceで消そうと思っていたのですが、うまくいかなかったため、
「+」「-」「.」のキー入力を取り消す方法で実装しました。

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

## UIの変更
なし

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

## 補足
なし
 2023-07-05 02:07:52 +00:00
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
masaaki
ced34e9d09 Merged PR 207: [Sp12-1着手]ログイン時、loading画面から遷移できない場合がある 
## 概要
[Task2060: [Sp12-1着手]ログイン時、loading画面から遷移できない場合がある](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2060)

- Client側のログイン時処理について、調査用のログを仕込みました。
  ・handleRedirectPromiseから取得したloginResultを出力
　　※基本的に、処理の分岐条件については、上記内容とlocalstorageの内容で追える見込みです
  ・exceptionの内容を出力

## レビューポイント
- 他に仕込んでおいた方がよいログがないかどうか

## UIの変更
- 無し

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

## 補足
- 本対応は事象解決のための修正ではなく、調査のための仕込みになります。
　事象が再発したら、以下を参照して調査再開する流れになります。
　・コンソールログ
　・ローカルストレージ
　・WAFのログ
 2023-07-04 10:14:02 +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
d6db89bc2c Merged PR 191: タスク一覧画面のデザインを修正する（ソートの▽表示とチェックボックスのカーソルなど） 
## 概要
[Task2045: [Sp12-1着手]タスク一覧画面のデザインを修正する（ソートの▽表示とチェックボックスのカーソルなど）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2045)

- aタグを使ったリンク・ボタン等でカーソルが指差しアイコンになるよう修正
- ソートの▽が選択とソート順によって変わるように修正
- チェックボックスのカーソルが常に指差しアイコンになる
- `.eslintignore`の修正がdevelopから漏れていたので反映

## レビューポイント
- 各修正内容は適切か
- ほかに必要なデザイン修正はないか
- `.eslintignore`は岩田の以下の修正内容がdevelopで上書きされてしまっていたので反映しています。
  - https://dev.azure.com/ODMSCloud/ODMS%20Cloud/_git/ODMS%20Cloud/pullrequest/192

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

## 動作確認状況
- ローカルで確認
 2023-07-04 02:32:00 +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
masaaki
279633ec4b Merged PR 204: [Sp12-1]対象国からタイが外れたので対応する 
## 概要
[Task1735: [Sp12-1]対象国からタイが外れたので対応する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1735)

- タイが登場する箇所について削除を行いました
- 影響範囲（他の機能にも影響があるか）
  - dictation_client、dictation_serverともに、「TH」「Thailand」の単語でgrepすることで影響調査を実施

## レビューポイント
- 影響範囲について、ほかの調査観点ないかどうか。

## UIの変更
- 無し

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

## 補足
- 無し
 2023-07-04 00:21:59 +00:00
oura.a
5241fcc997 Merged PR 203: メッセージファイル最新化 
## 概要
[Task2110: メッセージファイル最新化](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2110)

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

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-07-03 06:47:54 +00:00
oura.a
709db0d2ec Merged PR 187: 画面実装（カードライセンス発行PU） 
## 概要
[Task1991: 画面実装（カードライセンス発行PU）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1991)

カードライセンス発行ポップアップを新規作成。
上記ポップアップを起動するボタンを第１～４階層用のライセンスページに仮配置。

## レビューポイント
特筆するものはなし。
ライセンスページのポップアップ起動ボタンは仮配置なので、レビュー対象外でお願いします。

## UIの変更
https://ndstokyo.sharepoint.com/:i:/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/Task1991/%E3%82%AB%E3%83%BC%E3%83%89%E3%83%A9%E3%82%A4%E3%82%BB%E3%83%B3%E3%82%B9%E7%99%BA%E8%A1%8CPU.PNG?csf=1&web=1&e=NtuRgu

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

## 補足
・csvファイルダウンロード確認のポップアップは、現在は表示しないつくりになっています。PBIの受け入れ条件には表示するように記載があるのですが、月曜日のDSで奥澤さんに確認予定です。
・ライセンス数入力のテキストボックスで「.」「-」「+」が入力できてしまう事象がありました。こちらですがライセンス注文ポップアップでも同様の仕様となっているため、本PBI内の別タスクで纏めての修正とさせてください。
 2023-07-03 05:35:19 +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
539308f5b4 Merged PR 189: タスク一覧でタスク件数が0件のときページネーションが非活性にならない 
## 概要
[Task2074: [Sp12-1着手]タスク一覧でタスク件数が0件のときページネーションが非活性にならない](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2074)

- タスク一覧画面でタスク件数が0件の時にページネーションが非活性となるように活性条件を修正しました。

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

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

## 動作確認状況
- ローカルで確認
 2023-06-30 05:23:20 +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
masaaki
ebdfd78daf Merged PR 192: app.module.scss.d.tsをESlint対象から外す 
## 概要
[Task2100: app.module.scss.d.tsをESlint対象から外す](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2100)

- 6/29のDSで話題に出した件です
- デザイナさんのCSSから自動生成されるファイル（app.module.scss.d.ts）について、ESlintのエラーが出ていたのでファイル単位で除外設定しました

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

## UIの変更
- 特になし

## 動作確認状況
- ESLintエラーが消えること、ビルドが通ることを確認

## 補足
- 相談、参考資料などがあれば
 2023-06-29 05:17:11 +00:00
saito.k
f20fb5b362 Merged PR 188: 処理成功時、スナックバーを表示する 
## 概要
[Task2088: 処理成功時、スナックバーを表示する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2088)

- タイピスト割り当て変更処理の成功時、成功メッセージを表示する
- 他画面に同様の文言が存在していたため、共通のメッセージとして位置づけを変更
  - 同様の文言を表示していた箇所のメッセージ変数名を修正

## レビューポイント

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

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

## 補足
- 相談、参考資料などがあれば
 2023-06-28 08:47:23 +00:00