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

320 Commits

Author SHA1 Message Date
水本 祐希
2f5516ec81 Merged PR 503: features/accept/constants.tsを削除 
## 概要
[Task2863: features/accept/constants.tsを削除](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2863)

- features/acceptとfeatures/termsで同じconstantsの定義があったのでtermsに統一

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

## UIの変更
- なし

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

## 補足
- 相談、参考資料などがあれば
 2023-10-17 08:53:35 +00:00
水本 祐希
7196491cf0 Merged PR 472: 画面作成（利用規約同意画面） 
## 概要
[Task2802: 画面作成（利用規約同意画面）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2802)

- 何をどう変更したか、追加したライブラリなど
  - 利用規約同意画面の実装を行いました
- このPull Requestでの対象/対象外
  - api.tsおよびstyles
- 影響範囲（他の機能にも影響があるか）
  - ありません

## レビューポイント
- 特にレビューしてほしい箇所
  - URLの妥当性（動作確認のため別タスクで追加していますが、内容は本タスクで見てほしいです）
     違和感ないか確認お願いします。
         <Route path="/accept-to-use" element={<AcceptToUsePage />} />
  - 各処理のエラーハンドリングについて

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

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

## 補足
- 相談、参考資料などがあれば
 2023-10-17 07:15:49 +00:00
makabe.t
74bf434786 Merged PR 496: ワークタイプ削除画面処理実装 
## 概要
[Task2612: ワークタイプ削除画面処理実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2612)

- ワークタイプ削除の画面実装です。

## レビューポイント
- エラー処理に問題はないか

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

## 動作確認状況
- ローカルで確認
 2023-10-17 06:29:25 +00:00
oura.a
58674f7773 Merged PR 498: アカウント登録画面修正 
## 概要
[Task2809: アカウント登録画面修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2809)

アカウント作成時の管理者ユーザー作成時に、同意済みEULAバージョンを登録する処理を追加しました。

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

## UIの変更
なし

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

## 補足
なし
 2023-10-17 06:06:12 +00:00
masaaki
897bad289b Merged PR 480: 画面修正（ログイン画面） 
## 概要
[Task2801: 画面修正（ログイン画面）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2801)

- 以下の修正を実施しました
  - ログイン画面について、未同意バージョンがある場合、利用規約同意画面に遷移する処理を実装
  - 利用規約同意画面（ADB2C以外の画面）からログイン画面に遷移した際も処理継続できるよう対応を実施
- このPull Requestでの対象/対象外
  - AcceptToUsePageについては、遷移確認用のダミーページなので対象外でお願いします。
- 影響範囲（他の機能にも影響があるか）
  - ありません。

## レビューポイント
- 特にレビューしてほしい箇所
  1. 既存のLoginPageを以下のように分割しています。
      実装内容のイメージあっているか確認お願いします。
      - LoginPage→AADB2Cからのリダイレクトを元にLocalStorageアクセス用のキーを生成
      - TokenSettingPage→LocalStorageアクセス用のキーを使用してidTokenを取得し各種token生成を実施
  1. TokenSettingPage/index.tsxにて、型ガード（isErrorObject）を作成し使用しています。
      使い方やガードの実装が妥当か確認お願いします。

## UIの変更
- 無し

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

## 補足
- 相談、参考資料などがあれば
 2023-10-16 06:52:08 +00:00
makabe.t
685a8f6c3e Merged PR 487: 削除済みエラーを成功扱いとするように画面修正 
## 概要
[Task2841: 削除済みエラーを成功扱いとするように画面修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2841)

- ワークフロー画面からワークフローを削除した際に、対象がすでに削除済みの場合も成功扱いになるように修正しました。

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

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

## 動作確認状況
- ローカルで確認
 2023-10-13 05:23:37 +00:00
oura.a
cafacb761d Merged PR 494: App.ts修正 
## 概要
[Task2845: App.ts修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2845)

App.tsxのエラーコード判定部分を修正しました。

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

## UIの変更
なし

## 動作確認状況
ローカルでビルドエラーが出ないことを確認。
ローカルで無理やりエラーを投げ、以下を確認。
E010209を入れて401エラーを投げる
　→ログアウトされない
E010208を入れて401エラーを投げる
　→ログアウトされる

## 補足
なし
 2023-10-13 05:23:21 +00:00
oura.a
274292c389 Merged PR 477: API修正（トークン生成API） 
## 概要
[Task2800: API修正（トークン生成API）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2800)

既存のトークン生成APIに、利用規約バージョンのチェック処理を追加しました。
また、チェック処理で同意済みバージョンが最新でないときのエラー時にログアウトしないような処理を追加しました。

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

## UIの変更
なし

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

## 補足
なし
 2023-10-12 08:59:36 +00:00
makabe.t
2d569aee6d Merged PR 479: ワークフロー更新ポップアップ実装 
## 概要
[Task2777: ワークフロー更新ポップアップ実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2777)

- ワークフロー編集ポップアップを実装しました。

## レビューポイント
- 表示内容は適切か
- 選択ワークフローの値取得処理は適切か

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

## 動作確認状況
- ローカルで確認
 2023-10-12 07:42:53 +00:00
oura.a
a5b07596c1 Merged PR 486: IF不具合修正 
## 概要
[Task2840: api.ts最新化](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2840)

IFの修正を行いました。
・/api/termsがPOSTになっていたのでGETに修正
・/api/termsのtypesの記載が足りていなかったので修正

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

## UIの変更
なし

## 動作確認状況
swaggerUIで確認

## 補足
なし
 2023-10-12 05:37:25 +00:00
oura.a
5b0058b707 Merged PR 482: ビルドエラー修正 
## 概要
[Task2839: ビルドエラー修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2839)

ビルドエラーを修正しました。

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

## UIの変更
なし

## 動作確認状況
ローカルでビルドが通ることを確認。

## 補足
なし
 2023-10-11 08:44:07 +00:00
makabe.t
f4293213be Merged PR 475: 画面実装 
## 概要
[Task2786: 画面実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2786)

- ワークフロー一覧画面からワークフローを削除する処理を追加しました。
- アカウント画面の実装について想定外の変更となっていた、部品の不要なクラスを削除しました。
  - クラスに当たっていた`required`はクラスではなくタグの属性だったため削除して属性に設定しています。

## レビューポイント
- 削除処理は適切か。
- アカウント画面の対応内容に問題はないか
  - `select`の`required`は不要だと思いますがいかがでしょうか？

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

## 動作確認状況
- ローカルで確認
  - 実際にAPIで削除しての動作は未検証です。
 2023-10-11 08:35:51 +00:00
saito.k
3f018d1100 ほかブランチのマージで消えた実装をもとに戻す 2023-10-10 16:52:43 +09:00
saito.k
8e317f5aca Merged PR 478: POデモ修正対応 
## 概要
[Task2821: POデモ修正対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2821)

- リテラル修正
  - 追加Popupのタイトル
  - AuthorIDの初期値
- ワークフロー追加・更新の重複チェックの条件指定を修正
  - undefined,nullのままだと条件指定しないという挙動になるためIsNull（）を使用
- 該当ケースをテストに追加

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

## UIの変更
- タスクのなか

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

## 補足
- 相談、参考資料などがあれば
 2023-10-10 07:22:00 +00:00
水本 祐希
3af64fbf8e Merged PR 464: データ取得失敗時に各一覧表示画面の表示がそろっていない 
## 概要
[Task2720: データ取得失敗時に各一覧表示画面の表示がそろっていない](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2720)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
  - 各一覧表示画面で値０件およびデータ取得エラー時に表示する挙動を統一
 ※Accountに関しては自アカウント情報の表示のため値０件はないとみて、特に本タスクの実装
は必要ないと判断しました。
→対応としてはデータ取得エラー時のスナックバー表示のみ
※Userも自アカウント情報が表示されるため上記と同じ実装にする予定でしたが、スナックバーと０件表示ができました。

![image (4).png](https://dev.azure.com/ODMSCloud/6023ff7b-d41c-4fa7-9c6f-f576ba48c07c/_apis/git/repositories/302da463-a2d7-40f9-b2bb-6e8edf324fa9/pullRequests/464/attachments/image%20%284%29.png)

・〇はそれぞれの検証条件をクリアできているという意味です。
・「データ取得エラーとする」は通常ではAccount、License、Userは値が０件となることはなく、データベースエラーの場合のみと考えたので、値が0件＝データ取得エラーとしました。
・「スナックバー表示のみ」はデータ取得エラーが発生した場合、スナックバー表示のみで対応するという意味です。
・「表示できない」はデータベースを切った状態だと、子アカウントのorderhistoryが表示されないため、表示できないと記載しました。

- dictationSlice.tsでbuilder.addCase(getSortColumnAsync.rejected, (state) => {
      state.apps.isLoading = false;
を実装したのは、修正前はデータベースを切った状態だとロードのぐるぐるが消えずにいました。原因はgetSortColumnAsyncにrejectの場合、isLordingをfalseにする実装がなかったためです。
以上のことから上記実装を追加しました。

- この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/Task2720?csf=1&web=1&e=MBNgO8

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

## 補足
- 相談、参考資料などがあれば
 2023-10-10 06:40:15 +00:00
saito.k
7682c41ba5 Merged PR 462: ワークフロー追加ポップアップ実装 
## 概要
[Task2740: ワークフロー追加ポップアップ実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2740)

- ワークフロー追加Popupを実装
- 多言語対応

## レビューポイント
- タイピスト・タイピストグループの変更処理
  - 選択・除外で処理を分けたが一緒にしたほうが良い？
- 各パラメータの変更処理
  - 値のチェックは足りているか

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

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

## 補足
- 相談、参考資料などがあれば
 2023-10-10 00:57:11 +00:00
saito.k
088e6afc85 Merged PR 454: ワークフロー一覧画面 
## 概要
[Task2735: ワークフロー一覧画面](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2735)

- ワークフロー一覧画面のデザイン反映
- 多言語対応
- 一覧取得API呼び出し

## レビューポイント
- デザイン反映に問題はないか
- フォルダ構成はこれでよいか
  - workflow配下に直置き

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

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

## 補足
- 相談、参考資料などがあれば
 2023-10-03 02:15:57 +00:00
水本 祐希
5cfe069b58 Merged PR 444: 画面実装（アカウント削除成功ページ） 
## 概要
[Task2715: 画面実装（アカウント削除成功ページ）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2715)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
・アカウント削除成功後に、アカウント削除成功ページに遷移するよう実装
・ログオフ状態にする

- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）

## レビューポイント
- 特にレビューしてほしい箇所
・レイアウトやメッセージの文言
・ログオフ状態にするタイミング
→Back to TOP Pageを押下時にログオフ状態になるようにしています
・Delete Accountボタン（ポップアップ上の）押下時のページ遷移のコード
・その他、漏れがないか

- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載

## 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/Task2715?csf=1&web=1&e=0M85t6

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

## 補足
- 相談、参考資料などがあれば
 2023-10-02 01:19:18 +00:00
saito.k
3e90261ecc エラーラベルのデザインを修正 2023-09-29 14:41:05 +09:00
saito.k
f289bb3d19 エラーラベルの文言修正 2023-09-28 14:44:49 +09:00
saito.k
9a01eff837 Merged PR 449: [Sp18-2 or 19-1]翻訳情報を翻訳Excelに適用し、Webアプリにも適用する 
## 概要
[Task2166: [Sp18-2 or 19-1]翻訳情報を翻訳Excelに適用し、Webアプリにも適用する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2166)

- 翻訳情報を取り込む

## レビューポイント
- 一通り画面は確認しましたが、各画面をざっと見ていただいて問題なさそうか確認していただきたいです。
- 文字数が変わったことにより、項目名が改行されているところがあるが、一行で表示できた方が良いでしょうか。
  - 個人的には問題ないと思っています。

## UIの変更
- 変更点が多いため、手元で動かして確認していただきたいです。

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

## 補足
- 相談、参考資料などがあれば
 2023-09-28 01:23:36 +00:00
oura.a
ba7ef1b4c6 Merged PR 439: [PBI1197残]ドロップダウンの挙動について統一する 
## 概要
[Task2713: [PBI1197残]ドロップダウンの挙動について統一する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2713)

ドロップダウンの選択肢表示を修正しました。

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

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

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

## 補足
なし
 2023-09-27 08:40:26 +00:00
saito.k
de13753ef6 Merged PR 447: [Sp18-2 or 19-1]翻訳情報を翻訳Excelに適用し、Webアプリにも適用する 
## 概要
[Task2166: [Sp18-2 or 19-1]翻訳情報を翻訳Excelに適用し、Webアプリにも適用する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2166)

- 不要な文言を削除
- 削除した文言が使用されている箇所を修正

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-09-27 08:12:25 +00:00
saito.k
8265ca38c8 Merged PR 440: 画面実装（テンプレートファイルアップロードPopup） 
## 概要
[Task2656: 画面実装（テンプレートファイルアップロードPopup）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2656)

- テンプレートファイルアップロードのAPI呼び出し周りを実装
  - SASトークン付きURL取得
  - Blobストレージへファイルアップロード
  - アップロード完了
- server側
  - `helmet`の`connect-src`を修正
  - SASトークン付きURLが想定と違っていたため修正
  - DBに保存するURLが想定と違っていたため修正

## レビューポイント
- `connect-src`の`self`以外はローカル環境のみの設定でよさそう？
- Popupの挙動で不足している箇所はあるか
  - アップロードファイルでチェックすべき内容等

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

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

## 補足
- 相談、参考資料などがあれば
 2023-09-27 06:36:24 +00:00
水本 祐希
bf4dc1d717 Merged PR 443: 画面実装（アカウント削除確認ポップアップ）マージ用 
## 概要
[Task2738: 画面実装（アカウント削除確認ポップアップ）マージ用](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2738)

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

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-09-26 01:25:39 +00:00
水本 祐希
1a0edee5c9 Merged PR 415: 画面実装（アカウント削除確認ポップアップ） 
## 概要
[Task2669: 画面実装（アカウント削除確認ポップアップ）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2669)

- 元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/Task2669?csf=1&web=1&e=VbkLlR

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

## 補足
- 相談、参考資料などがあれば
 2023-09-26 00:52:46 +00:00
saito.k
9ca4ae61f8 Merged PR 434: 画面実装（テンプレートファイルアップロードPopupデザイン） 
## 概要
[Task2664: 画面実装（テンプレートファイルアップロードPopupデザイン）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2664)

- デザイン反映
- ファイルピッカーからファイル取得→storeに保存

## レビューポイント
- デザイン反映に不備はあるか
- 想定としてstoreに保持したfileを、Operationsでblobにアップロードする流れにしようとしているがよさそうか

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

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

## 補足
- 相談、参考資料などがあれば
 2023-09-25 07:53:45 +00:00
saito.k
3f4d4ec436 Merged PR 427: 画面実装（テンプレートファイル一覧画面） 
## 概要
[Task2651: 画面実装（テンプレートファイル一覧画面）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2651)

- テンプレートファイル一覧画面を実装

## レビューポイント
- 取得方法やstoreの構成は問題ないか

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

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

## 補足
- 相談、参考資料などがあれば
 2023-09-22 07:36:23 +00:00
oura.a
f120f4f7e5 Merged PR 436: [PBI1197残]画面レイアウトについて確認する 
## 概要
[Task2712: [PBI1197残]画面レイアウトについて確認する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2712)

アカウント情報画面について、以下のデザイン修正を実施しました。
・「Save changes」ボタンを右側に移動
・表内の値が長かった場合に三点リーダでの省略表示から、改行して全体を表示する

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

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

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

## 補足
なし
 2023-09-22 07:15:54 +00:00
makabe.t
77eeb0ea83 Merged PR 426: ActiveWorktypeIDの画面実装修正 
## 概要
[Task2706: ActiveWorktypeIDの画面実装修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2706)

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

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-09-22 06:05:19 +00:00
masaaki
9ac40c00ac Merged PR 431: static_contentsフォルダが空の場合の対応を行う 
## 概要
[Task2719: static_contentsフォルダが空の場合の対応を行う](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2719)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）
  - statis_contents配下が空の場合、gitに認識されなくなってしまうので、コピー処理自体を削除しました
  - 今後復活させたくなった時のため、wikiに手順を追加しておきました。
https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_wiki/wikis/OMDSDictation_wiki/268/%E9%9D%99%E7%9A%84%E3%82%B3%E3%83%B3%E3%83%86%E3%83%B3%E3%83%84%E3%81%AB%E5%AF%BE%E3%81%97%E3%81%A6%E9%9D%99%E7%9A%84%E3%83%95%E3%82%A1%E3%82%A4%E3%83%AB%E3%82%92%E9%85%8D%E7%BD%AE%E3%81%97%E3%81%9F%E3%81%84

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

## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
  - 特にありません

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

## 補足
- 相談、参考資料などがあれば
 2023-09-21 07:59:09 +00:00
masaaki
cda3652817 Merged PR 430: 脆弱性試験ツール対応を元に戻す 
## 概要
[Task2694: 脆弱性試験ツール対応を元に戻す](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2694)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
    - ドメイン確認用のファイルについて削除しました。
- このPull Requestでの対象/対象外
    - すべて対象
- 影響範囲（他の機能にも影響があるか）
    - 無し

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

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

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

## 補足
- 相談、参考資料などがあれば
    - ローカル環境でnpm run build:localを実施していた場合、
       /app/dictation_server/build配下のファイル「YOweYATRY5PBN1G9d….html」を手動で削除する必要があります。
      動作上の悪影響はありませんが、ゴミなので削除願います。
 2023-09-21 06:52:15 +00:00
oura.a
e7bd7b52fa Merged PR 425: 親アカウント名表示修正 
## 概要
[Task2687: 親アカウント名表示修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2687)

アカウント情報画面の親アカウント名表示対応を実施

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

## UIの変更
なし

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

## 補足
なし
 2023-09-20 02:34:02 +00:00
saito.k
75393d683d Merged PR 421: OptionItemポップアップサイズを大きくするデザイン依頼 
## 概要
[Task2678: OptionItemポップアップサイズを大きくするデザイン依頼](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2678)

- タイトルの通り

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-09-20 02:14:16 +00:00
saito.k
32db216d6a Merged PR 416: ログアウト時にlogoutRedirectを使うようにする 
## 概要
[Task1829: ログアウト時にlogoutRedirectを使うようにする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1829)

- タイトルの通り
  - 該当箇所をすべて修正

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

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

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

## 補足
- B2Cの設定も修正
   - 不要なアクセストークンは発行しないように修正
 2023-09-20 01:36:57 +00:00
oura.a
8fee108a78 Merged PR 422: 成功時のスナックバー追加 
## 概要
[Task2685: 成功時のスナックバー追加](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2685)

情報更新成功時の成功スナックバー実装が漏れていたため、追加しました。

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

## UIの変更
なし
## 動作確認状況
ローカルで確認

## 補足
なし
 2023-09-19 10:32:33 +00:00
oura.a
2705fcee0a Merged PR 406: 画面実装（アカウント情報画面） 
## 概要
[Task2605: 画面実装（アカウント情報画面）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2605)

アカウント情報画面を実装しました。
※以下の処理は本PBIの対象外となりますので、レビュー対象外でお願いします
・「File Delete Setting」ボタン押下時の処理
・「Delete Account」ボタン押下時の処理

## レビューポイント
もっと簡潔に記載できる箇所はないか

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

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

## 補足
なし
 2023-09-19 10:01:19 +00:00
makabe.t
ec5df50aa0 Merged PR 420: 結合動作確認不具合対応 
## 概要
[Task2684: 結合動作確認不具合対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2684)

- Active WorktypeIDの変更について結合動作確認で見つかった不具合に対応しました。
  - ActiveWorktypeIDの変更がダイアログ表示の前にセレクトボックスに反映される
  - 変更成功時に成功スナックバーを表示する

## レビューポイント
- セレクトボックスの変更時のイベントの取り扱いは適切か

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

## 動作確認状況
- ローカルで確認
 2023-09-19 09:29:04 +00:00
makabe.t
d1a8b887e5 Merged PR 417: ActiveWorktypeIDを未設定に戻せるように修正 
## 概要
[Task2677: ActiveWorktypeIDを未設定に戻せるように修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2677)

- ActiveWorktypeIDを設定後、未選択に戻す操作ができるように修正

## レビューポイント
- 修正箇所の共有

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-09-19 02:56:19 +00:00
makabe.t
b5ecd6de15 Merged PR 410: 画面実装（ActiveWorktypeID設定セレクトボックス） 
## 概要
[Task2623: 画面実装（ActiveWorktypeID設定セレクトボックス）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2623)

- WorktypeID設定画面でのActiveWorktypeID選択処理を実装しました。

## レビューポイント
- WorkTypeIDの変更時の処理に問題はないか
- 画面の表示に問題はないか

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

## 動作確認状況
- ローカルで確認
 2023-09-15 08:28:18 +00:00
湯本 開
a8983c418e Merged PR 414: 脆弱性試験ツール対応 
## 概要
[Task2665: 脆弱性試験ツール対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2665)

- Viteビルドを介さずデプロイしたい静的ファイルを配置する用のディレクトリを追加
- ビルド時にファイルコピーをする処理をnpm run build:* コマンドに追加
- 今回配置依頼のあった静的ファイルを配置

## レビューポイント
- 情報共有

## 動作確認状況
- ローカルで確認
 2023-09-15 05:16:01 +00:00
水本 祐希
25de994013 Merged PR 411: パートナー一覧画面のDealer Management修正 
## 概要
[Task2635: パートナー一覧画面のDealer Management修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2635)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
APIの返却値によってDealerManagementのチェックの有効・無効を制御する

- この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/Task2539?csf=1&web=1&e=n647LC

## 動作確認状況
- ローカルで確認
MySQLWorkbenchのdelegation_permissionの値を操作
  - delegation_permission=0（false）の場合、チェック無効
  - delegation_permission=1（true）の場合、チェック有効

## 補足
- 相談、参考資料などがあれば
 2023-09-14 09:09:01 +00:00
saito.k
40f526c81c エラー表示を他画面とそろえる 2023-09-14 11:19:33 +09:00
saito.k
bdbb0dfe5d Merged PR 394: 画面実装（オプションアイテム編集Popup） 
## 概要
[Task2594: 画面実装（オプションアイテム編集Popup）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2594)

- オプションアイテム編集Popupを実装
  - Popup表示時にWorktypeIDに紐づくOptionItemを取得
  - 入力値のバリデーション

## レビューポイント
- エラーの表示の仕方を変えてみたが、問題ないか
  -「 タスク 2630: 入力のエラーラベルの表示が不自然なので対応」の参考となる実装として
- 保存可能な文字種ルールの位置
- エラーラベルの位置

## 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/Task2594?csf=1&web=1&e=9Ygf6d

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

## 補足
- オプションアイテム更新の確認は未実施
 2023-09-13 06:39:55 +00:00
saito.k
95b48a766a Merged PR 409: nodeバージョンアップ 
## 概要
[Task2647: nodeバージョンアップ](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2647)

- タイトルの通り

## レビューポイント

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

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

## 補足
- 相談、参考資料などがあれば
 2023-09-13 06:38:01 +00:00
makabe.t
3f17ec615c Merged PR 402: API IF実装 
## 概要
[Task2620: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2620)

- ActiveWorktypeAPIのIFを実装しました。
  - クライアントはIF反映だけしています。

## レビューポイント
- IFはラフスケッチの認識通りか
- 制約に問題はないか。
- クライアントの対応に問題はないか

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2023-09-12 09:34:00 +00:00
水本 祐希
606ff6de9b Merged PR 379: 画面実装（パートナー一覧画面本実装） 
## 概要
[Task2539: 画面実装（パートナー一覧画面本実装）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2539)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
パートナー一覧画面でパートナーの一覧が表示されるように実装

- このPull Requestでの対象/対象外
・Add Accountボタンは前PBIのため対象外
・Dealer Managementボタンの挙動は対象外
・Delete Accountボタンの挙動は対象外

- 影響範囲（他の機能にも影響があるか）
特になし

## レビューポイント
- 特にレビューしてほしい箇所
・Dealer Management、Delete Accountボタンの表示制御
・ページネーション

- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載

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

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

## 補足
- 相談、参考資料などがあれば
 2023-09-12 01:14:26 +00:00
maruyama.t
82fb224d67 Merged PR 390: [Sp17着手]ボタン押下時処理にpreventDefault()を行うものとおこなわないものが混在するのを整理する 
## 概要
[Task2547: [Sp17着手]ボタン押下時処理にpreventDefault()を行うものとおこなわないものが混在するのを整理する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2547)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
パートナー追加ポップアップで処理実行中にでブラウザを閉じようとしたときに確認ダイアログを表示していた箇所を削除（コピペ元からの削除漏れ）
ライセンス注文履歴画面で、ボタン押下時に不要なpreventDefaultを行っていた箇所を削除
- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-09-08 05:27:52 +00:00
saito.k
da1ee8a9c2 Merged PR 391: [Sp17着手]セレクトボックスで長い文字列が入っても問題ないようにする 
## 概要
[Task2550: [Sp17着手]セレクトボックスで長い文字列が入っても問題ないようにする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2550)

- タイトルの通り

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

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

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

## 補足
- 相談、参考資料などがあれば
 2023-09-08 05:01:24 +00:00
makabe.t
f14c086980 Merged PR 386: ワークタイプ編集ポップアップ実装 
## 概要
[Task2570: ワークタイプ編集ポップアップ実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2570)

- ワークタイプ編集ポップアップを実装しました。
  - ローディング中処理をタスク追加にも追加しています。

## レビューポイント
- 画面の表示は想定通りか
- ローディング中処理の追加に問題はないか

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

## 動作確認状況
- ローカルで確認
 2023-09-07 08:58:36 +00:00