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)
カードライセンス発行ポップアップを新規作成。
上記ポップアップを起動するボタンを第1~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/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 |
|
masaaki
|
4f69a57b84
|
Merged PR 182: cssについて「app.module.scss」と「GlobalStyle.css」をマージして、個別の参照をなくす
## 概要
[Task1895: cssについて「app.module.scss」と「GlobalStyle.css」をマージして、個別の参照をなくす](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1895)
- GlobalStyle.cssをなくしてデザイナさんのCSSをapp.module.scssにまとめるとともに、個別でcssを作成していた画面についてapp.module.scssを参照するように修正しました。
- 修正の過程で、最新のcss及び画像の取り込みを行いました。
- トップページのレイアウトについて最新化を行いました。
(適用前後でレイアウトにずれが発生したため、調査の過程で最新化を実施)
- 影響範囲は全画面レイアウトになります。適用前後の画面イメージを比較して確認しております。
## レビューポイント
- 各画面の変更前後のイメージについてスクリーンショットを格納しています。前後比較いただき問題ないか確認いただきたいです。
※変更後についてヘッダとシステム名の文字サイズが大きくなっていますが、最新の画面デザイン上そうなっています。
※トップページについては最新化を行ったため、デザイナさんの画面と比較いただければと思います。
## 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/Task1895?csf=1&web=1&e=heIWaD
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-06-27 07:19:41 +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 |
|
makabe.t
|
33fc741eee
|
Merged PR 156: タイピスト割り当てポップアップ実装&画面修正
## 概要
[Task1933: タイピスト割り当てポップアップ実装&画面修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1933)
- タイピスト割り当てポップアップを実装しました。
- ポップアップをタスクがUploaded時のみ開けるようにしています。
- タイピスト割り当てを変更・保存できるようにしています。
## レビューポイント
- 画面実装のデザインは適切か
- データの保持方法は適切か
## UIの変更
- [Task1933](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/Task1933?csf=1&web=1&e=4ghlqe)
## 動作確認状況
- ローカルで確認
- API連携は未確認
|
2023-06-23 04:57:54 +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
|
a2c5c436e1
|
Merged PR 151: パスワードリセット画面にて「Cancel」をクリックすると、「loading」画面で止まる問題を解決する
## 概要
[Task1921: パスワードリセット画面にて「Cancel」をクリックすると、「loading」画面で止まる問題を解決する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1921)
- 「補足」の「実装参考記事」を参考に、ログインリダイレクト時のハンドリング方法を修正
- ログインリダイレクト処理におけるAADB2Cからのエラーハンドリングを実装し、
エラーコードが「ユーザーによる操作キャンセル」だった場合、トップ画面に戻すよう修正
## レビューポイント
- ログインリダイレクト時のハンドリング方法は不適切ではないか
- とくに、React hookの使い方でアンチパターンになっているようなことはないか
## UIの変更
- なし
## 動作確認状況
- ローカルにて、ログインできることを確認
- ローカルにて、ユーザーフローキャンセル時にトップ画面に戻ることを確認(二要素認証、パスワードリセットともに)
## 補足
- 実装参考記事
- [その1](https://learn.microsoft.com/ja-jp/azure/active-directory/develop/msal-js-initializing-client-applications#initialize-msaljs-2x-apps)
- [その2](https://learn.microsoft.com/ja-jp/azure/active-directory/develop/msal-error-handling-js)
- [AADB2Cエラーコードリファレンス](https://learn.microsoft.com/ja-jp/azure/active-directory-b2c/error-codes)
|
2023-06-22 23:24:35 +00:00 |
|
saito.k
|
2155539365
|
Merged PR 175: タスク一覧画面でタスクステータスのフィルターをすべて外すとエラーとなる
## 概要
[Task2037: 対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2037)
- フィルターの選択をすべて未選択にするとエラーになる挙動を修正
- フィルターがすべて未選択時に画面側から空文字が渡されていた。
- APIはstetusに値が入っていると判断して、バリデーションを行っていたためエラーになっていた。
- フィルターがすべて未選択の場合は画面側からundefinedを渡すように修正した。
## レビューポイント
- 修正箇所は画面側でよいか
- 修正内容に問題はあるか
## 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/Bug2037?csf=1&web=1&e=pJJk6P
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-06-22 08:07:31 +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 |
|
oura.a
|
8c75f7a1b1
|
Merged PR 168: static appについて環境ごとに設定値を切り替える
## 概要
[Task2004: static appについて環境ごとに設定値を切り替える](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2004)
ステージング環境用のB2C情報を設定したenvファイルを作成。
作成したenvファイルはpipelineの
「Dictation Static App Service Deploy」→「Bash Script」→「Script」
を以下のように修正することでの反映を考えています。
■修正前
cd dictation_client && npm run build
■修正後
cd dictation_client && cp -f .env.staging .env && npm run build
※もしも本修正がdevelop環境にも影響する場合は、develop側も同様にenvファイルを作成しpipelineにコピー処理を追加する。
## レビューポイント
設定内容が妥当か。
上記のpipelineへの追加処理が妥当か。
## UIの変更
なし
## 動作確認状況
未
## 補足
なし
|
2023-06-20 05:18:30 +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 |
|