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
maruyama.t
f265d0ff45 Merged PR 812: 「oderLicense」や「CardLicense」を開いて閉じるとパンくずリストがおかしくなる 
## 概要
[Task3771: 対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3771)

ポップアップを×で閉じた際に、パンくずリストを保持したまま、`dispatch(getMyAccountAsync()`しており画面を初期化していた。
画面を更新するのかしないのかあいまいな挙動になっていた。

対応としては、他のポップアップと同様、×で閉じた場合は画面は更新しないように修正。
追加でパンくずリストを初期化する処理を作成し、ポップアップ側でAPIを呼び出した場合のみそれを実行する。

## レビューポイント
- 特筆する点はなし

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

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

## 補足
- 相談、参考資料などがあれば
 2024-03-07 10:41:30 +00:00
makabe.t
da40e8f09c Merged PR 800: 画面実装（一括追加ボタン＆ポップアップ画面） 
## 概要
[Task3753: 画面実装（一括追加ボタン＆ポップアップ画面）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3753)

- ユーザー一括登録画面を実装しました。
  - 一括登録ポップアップ
    - テンプレートCSVダウンロード
    - ファイルインポート
      - エラー行表示

## レビューポイント
- 行エラーの条件、内容は認識通りでしょうか？
- 画面の表示内容は認識通りでしょうか？
- CSV変換時にworkerを有効にしているとエラーとなりうまくいかないのでOFFにしてしまいましたが問題ないでしょうか？
  - @<湯本 開> さん

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

## 動作確認状況
- ローカルで確認
 -  ファイルチェックするところまで
 2024-03-06 01:58:32 +00:00
湯本 開
363f12f86f Merged PR 774: 画面実装（csv読み込み部分切り出し） 
## 概要
[タスク 3754: 画面実装（csv読み込み部分切り出し）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3754)

- CSVファイルの内容を入力として、JSONに変換する処理＆テストを実装
- 画面実装時の想定としては、以下の流れと想定しており、本Taskの範囲は1のみ
  1. csv->jsonへパースが出来るか？(csvの形式として合っていて読み込み可能か？)
  2. json内の各パラメータは問題ないか？（データの制限や組み合わせは問題ないか？）
- 使い方を示す & 動作確認のために、client側でもテストを実施できるよう修正（※pipelineでは実行されない）

## レビューポイント
- テストケースを見て、使い方は分かるか
- CSVの形式自体が想定とズレていた場合は入力を弾く必要がある想定だが、間違っていないか
- 利用ライブラリはメジャーかつ便利そうなものを選定したが、問題なさそうか

## 動作確認状況
- npm run testを通過
 2024-02-28 09:03:27 +00:00
SAITO-PC-3\saito.k
ddd4d31f25 Merge branch 'develop' into ccb 
# Conflicts:
#	dictation_server/src/features/users/users.service.spec.ts
 2024-02-27 09:26:46 +09:00
Kentaro Fukunaga
5305984b1a Merged PR 764: 第五階層ライセンス情報画面実装 
## 概要
[Task3709: 第五階層ライセンス情報画面実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3709)

- ストレージ使用可否切り替えの画面実装をしました
- 動作確認中に、既存実装でライセンスオーダーするときとカードライセンスアクティベートするときの操作不能化処理に漏れがあったのを修正しました

## レビューポイント
- Redux周りの実装でお作法に違反しているところがないか。もしくは改善点ないか。
- ライセンス情報表示のAPI結果待ち部分のローディング処理で、最低限の改善にしたが現時点ではこれでよいか？（いつ修正するかも未定だけど、実害はないためひとまずこんな感じで。。。）
    - `licenseSummarySlice.ts` のコメント部分が該当箇所です

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

## 動作確認状況
- ローカルで動作確認しました。
 2024-02-27 00:01:02 +00:00
saito.k
d0628caa05 Merged PR 767: パートナー追加成功時に一覧の更新が行われていない 
## 概要
[Task3769: パートナー追加成功時に一覧の更新が行われていない](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3769)

- パートナー追加成功時に一覧の更新を行う

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

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

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

## 補足
- 相談、参考資料などがあれば
 2024-02-26 10:42:26 +00:00
SAITO-PC-3\saito.k
562db3def9 Merge branch 'develop' into ccb 2024-02-19 21:25:54 +09:00
saito.k
f8ff19a3fa Merged PR 758: リテラル反映・Typist表示をTranscriptioninstにする 
## 概要
[Task3692: リテラル反映・Typist表示をTranscriptioninstにする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3692)

- フッターのコピーライトを修正
  - 文字列がべた書きだったのを翻訳情報を入れるように修正
- User一覧画面でRoleがTypistの時の表記をTranscriptioninstに修正
- OptionItemの項目名を入れ替え
  - initialValue　⇔　defaultValue
- アカウント画面のSaveボタン下にある代理操作に対する注釈を削除
- 翻訳情報反映

## レビューポイント
- OptionItemの項目名を入れ替えたが、表示するプロパティ名も名前を修正したほうが良いか？(initialValueの列にdefaultValueTypeを表示しているため混乱するのではないかという懸念)
  - APIのレスポンスのプロパティ名から変えないと揃えられないため、表示する項目名の並びだけを変えた
    - なんで入れ替えたのかコメントしたし、これで良いか

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

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

## 補足
- 相談、参考資料などがあれば
 2024-02-19 10:29:38 +00:00
SAITO-PC-3\saito.k
cdef84e269 Merge branch 'develop' into ccb 
# Conflicts:
#	dictation_server/src/features/accounts/accounts.service.spec.ts
 2024-02-19 12:00:01 +09:00
saito.k
4caa23ba3c Merged PR 755: [FB対応]File Property表示について 
## 概要
[Task3735: 対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3735)

- FilePropertyの表示順を変更する
  - カテゴリ（General、Job）はそのままで、OMDS様の希望する並び順にする

## レビューポイント
- 並び順の変更にミスはないか

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

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

## 補足
- 相談、参考資料などがあれば
 2024-02-19 02:17:37 +00:00
saito.k
b7db9c5fad Merged PR 752: [FB対応]Edgeで表示言語が切り替わらないことがある 
## 概要
[Task3728: 原因調査＆調査結果をPBIに記載する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3728)

- cookieから言語情報取得時にほかのcookieが存在していると、うまく取得できなくなってしまう
  - https://into-the-program.com/javascript-cant-get-value-cookie-name-array/
  - この方法でほかのcookieがある場合でも半角スペースを排除して正しく取得できるようにした。

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

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

## 補足
- 相談、参考資料などがあれば
 2024-02-19 01:49:23 +00:00
saito.k
b4df229de4 Merged PR 751: [FB対応]WorkTypeIDを大文字固定にしたい 
## 概要
[Task3733: 対応する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3733)

- 大文字のみの入力に強制する
  - WorkTypeID
  - OptionItem

- APIの大文字固定のパラメータにバリデーションを付与する
  - AuthorID
  - WorkTypeID
  - OptionItem

## レビューポイント
- 大文字に変換する処理をreducerに寄せたが問題ないか
- バリデータを付けるパラメータに漏れはないか

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

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

## 補足
- 相談、参考資料などがあれば
 2024-02-19 01:09:39 +00:00
makabe.t
fc7d271a29 Merged PR 749: 第五階層ライセンス情報画面実装 
## 概要
[Task3654: 第五階層ライセンス情報画面実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3654)

- 第五階層ライセンス情報画面でストレージ情報（Storage Available、Storage Used）を表示するように画面を修正しました。
  - 表示する値はGB単位となるようにし、小数点以下第三位までを表示するようにしています。

## レビューポイント
- 表示する値のフォーマットは適切でしょうか？
  - 小数点以下第三位までを固定で表示するようにしていますが問題ないでしょうか？

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

## 動作確認状況
- ローカルで確認
 2024-02-16 05:15:45 +00:00
SAITO-PC-3\saito.k
3877a4670d Merge branch 'develop' into ccb 
# Conflicts:
#	dictation_client/src/pages/UserListPage/index.tsx
 2024-02-14 11:33:11 +09:00
saito.k
25b7936bf4 Merged PR 747: デザイン反映 
## 概要
[Task3670: デザイン反映](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3670)

- ユーザー一覧画面のヘッダーのスタイル修正
  - ブラウザのサイズに合わせて折り返すようにした
  - ユーザー一覧のテーブルを横スクロールできるように修正
- ライセンス履歴画面にパンくずリストを表示
- OptionItem更新ポップアップのスタイル修正
  - 端が切れてしまっていたため枠に収まるようにした
- Typistを「Selected」と「Pool」で選択するUIのスタイル修正
  - 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/Task3670?csf=1&web=1&e=zIcngJ
## 動作確認状況
- ローカルで確認

## 補足
- 相談、参考資料などがあれば
 2024-02-14 01:50:38 +00:00
makabe.t
bbbd3e757b Merged PR 743: テンプレートファイル削除画面修正 
## 概要
[Task3600: テンプレートファイル削除画面修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3600)

- テンプレートファイル削除の画面を実装しました。

## レビューポイント
- エラー処理は適切でしょうか？

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

## 動作確認状況
- ローカルで確認
 2024-02-13 02:32:17 +00:00
saito.k
9cc9a3bd94 Merged PR 744: タスク一覧画面修正 
## 概要
[Task3672: タスク一覧画面修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3672)

- タスク一覧の表示項目制御用のチェックボックスの操作可能範囲を修正
  - 「Comment」のチェックボックスをリテラルまで押せるようにした
- タスク一覧のPriorityがHighの時の表示を変更した
  - 行を赤く表示→文字を赤く表示

## レビューポイント
- 修正する箇所の認識は合っているか

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

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

## 補足
- 相談、参考資料などがあれば
 2024-02-13 02:12:00 +00:00
x.yumoto.k
cd277f3f9a Merge branch 'develop' into ccb 2024-02-09 17:37:38 +09:00
湯本 開
7ca4249f04 Merged PR 742: ログアウトせずに認証切れまで待った後にTOPページからログインしようとするとエラーが発生する 
## 概要
[Task3680: ログアウトせずに認証切れまで待った後にTOPページからログインしようとするとエラーが発生する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3680)

- 現象
  - ブラウザバック対策のコードで、アクセストークンの寿命を意識せずにログイン後画面に遷移していたため、リフレッシュトークンとアクセストークンの再発行がSkipされた上で寿命が切れたトークンを使うような状態でログイン後画面に遷移してしまっていた
- 対応
  - /login, /authで、アクセストークンチェック→IdTokenを使ったログイン中プロセスかチェック の順番を変更
    - 有効なアクセストークンがあったとしても、再ログインをしてるのであればリフレッシュトークン等の再発行を実施する形に変更

## レビューポイント
- Bugの現象が解消する以外の挙動の変更が発生してしまわないか
  - 特にブラウザバック等
- 実装方法で問題がありそうな部分はないか
  - 不要なTokenのクリア等はないか

## UIの変更
- なし

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

## 補足
- **可能であればローカルで挙動を確認していただきたいです**
  - .env.local の `ACCESS_TOKEN_LIFETIME_WEB` を60(=1分)とかにすれば確認はしやすいはず
 2024-02-09 08:12:26 +00:00
makabe.t
270122b135 Merged PR 741: アカウント情報画面修正（保存日数の表示） 
## 概要
[Task3691: アカウント情報画面修正（保存日数の表示）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3691)

- アカウント情報画面にファイルの保存日数を表示するよう修正しました。
  - ファイル削除が設定されている場合は日数、されていない場合はハイフン表示となるようにしています。

## レビューポイント
- 画面イメージは想定通りでしょうか？
- 表示する保存日数は想定通りの値でしょうか？

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

## 動作確認状況
- ローカルで確認
 2024-02-09 01:05:15 +00:00
Kentaro Fukunaga
548c7a05e9 Merged PR 729: 画面実装（ファイル削除設定ポップアップ画面） 
## 概要
[Task3547: 画面実装（ファイル削除設定ポップアップ画面）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3547)

- ファイル削除設定ポップアップの新規実装をしました。

## レビューポイント
- reduxとの連携部分の設計でNGな部分ないか？
- 処理の抜け漏れないか？

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

## 動作確認状況
- ローカルで一通りの動作確認しました。

## 補足
- とくになし
 2024-02-06 12:58:55 +00:00
SAITO-PC-3\saito.k
1e8bc39c7f Merge branch 'develop' into ccb 
# Conflicts:
#	dictation_client/src/translation/de.json
#	dictation_client/src/translation/en.json
#	dictation_client/src/translation/es.json
#	dictation_client/src/translation/fr.json
 2024-02-05 21:15:24 +09:00
saito.k
84b0da1f95 Merged PR 723: [FB対応]タイピストグループ重複時のエラーとする 
## 概要
[Task3613: 対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3613)

- タイピストグループ名が重複した際のエラーを追加
  - タイピストグループ追加API
  - タイピストグループ更新API
- タイピストグループ設定画面に表示するエラーメッセージを追加

## レビューポイント
- 行ロックするべきかどうか
  - ギリギリのタイミングで同名のタイピストグループが作成される場合は防げないのでDBでユニーク制約を設定する？
    - insertにロックはかけられないから

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

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

## 補足
- 相談、参考資料などがあれば
 2024-02-05 11:46:29 +00:00
makabe
ce3deecd1b Merge branch 'develop' into ccb 2024-02-05 19:39:37 +09:00
makabe
8e159a1c2a Merge branch 'develop' into ccb 2024-02-05 16:36:09 +09:00
saito.k
48d2e625db Merged PR 730: [FB対応]タスク一覧画面のデザイン修正 
## 概要
[Task3614: 対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3614)
- デザイン修正
  - ソートが昇順の時は▲
  - ソートが降順の時は▼
- Priorityが「High」の時にその行を赤くする

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

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

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

## 補足
- 相談、参考資料などがあれば
 2024-02-05 07:08:00 +00:00
saito.k
1daeedbfdb Merged PR 722: アカウント画面に注釈を追加する＆翻訳反映 
## 概要
[Task3592: アカウント画面に注釈を追加する＆翻訳反映](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3592)

- ディーラーマネジメントの下、保存ボタンの下に注釈追加
  - タスクの中のBacklogに詳細があります
- 翻訳反映

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

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

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

## 補足
- 相談、参考資料などがあれば
 2024-02-05 06:11:41 +00:00
saito.k
4be13e002d Merged PR 710: 画面実装（削除操作） 
## 概要
[Task3488: 画面実装（削除操作）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3488)

- ユーザー削除の画面実装
  - 確認ダイアログ
  - 削除API呼び出し
  - エラーハンドリング
  - 成功時のメッセージ
  - 成功時のユーザー一覧更新

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

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

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

## 補足
- API呼び出しのエラーハンドリング部分はエラーコードが採番されたら追従します
 2024-02-05 00:39:53 +00:00
Kentaro Fukunaga
44759b1aac Merged PR 725: タイピストグループ削除　画面実装 
## 概要
[Task3536: 画面実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3536)

- タイピストグループ画面に削除ボタン配置
- 削除ボタンクリック時に確認ダイアログ表示→OKで削除API実行＆ぐるぐる表示
- API実行結果によってメッセージの出し分け＆画面リスト更新
- 翻訳情報追加

## レビューポイント
- エラーコードに認識違いないか？
- 画面実装のお作法にそぐわないところがないか？
- なんか抜けてる実装などあれば

## UIの変更
大した変更ではないので、スクショ置き場に置く手間を省いてここに貼り付けます
![image (2).png](https://dev.azure.com/ODMSCloud/6023ff7b-d41c-4fa7-9c6f-f576ba48c07c/_apis/git/repositories/302da463-a2d7-40f9-b2bb-6e8edf324fa9/pullRequests/725/attachments/image%20%282%29.png)

## 動作確認状況
- 期待される動作を一通りローカルで確認しました。
    - API実行中はぐるぐる表示されること
    - API成功または削除済みの場合は成功メッセージ表示され、画面更新されること
    - API失敗時、理由によってエラーメッセージが表示分けされること
 2024-02-05 00:31:47 +00:00
SAITO-PC-3\saito.k
fbcafd2014 Merge branch 'develop' into ccb 
# Conflicts:
#	dictation_client/src/pages/DictationPage/index.tsx
 2024-02-02 12:01:46 +09:00
saito.k
06b5249e5a Merged PR 717: [FB対応]画面更新するとヘッダーのソートがJob numberに戻ってしまう 
## 概要
[Task3611: 対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3611)

- ソート条件を変更した際にローカルストレージに保持するように修正
  - 例　`direction:ASC,paramName:RECORDING_STARTED_DATE`
- ローカルストレージにソート条件が入っていれば、その条件でソートしたタスクを取得するように修正
- PlayBack時にローカルストレージにあるソート条件を削除するように修正
  - 削除することで、次回の画面初期表示時はPlayBackを押したときのソート条件を使用することができる。
- PlayBack時にユーザーがタイピストの時のみソート条件を保存していたがその制限は不要そうだったのでAuthorでもソート条件を更新するように修正。
  - AuthorがPlayBack押下時にソート条件を更新しても不都合はないため。

## レビューポイント
- ローカルストレージに保存する処理を入れる箇所に問題はないか

## 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/Task3611?csf=1&web=1&e=5uG6f4

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

## 補足
- 相談、参考資料などがあれば
 2024-02-02 02:00:47 +00:00
SAITO-PC-3\saito.k
08a37ff264 Merge branch 'develop' into ccb 
# Conflicts:
#	dictation_client/src/translation/de.json
#	dictation_client/src/translation/en.json
#	dictation_client/src/translation/es.json
#	dictation_client/src/translation/fr.json
 2024-01-29 18:30:34 +09:00
saito.k
f9d5082f39 Merged PR 706: 翻訳情報を反映する 
## 概要
[Task3529: 翻訳情報を反映する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3529)

- 翻訳情報を反映

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

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

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

## 補足
- 相談、参考資料などがあれば
 2024-01-29 08:38:26 +00:00
makabe
f00861702a Merge branch 'develop' into ccb 2024-01-29 15:52:55 +09:00
makabe.t
dbff86915e Merged PR 708: 対応 
## 概要
[Task3550: 対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3550)

- Worktypeの登録・編集時に16文字までとなるように制限を修正しました。
  - APIのバリデータ
  - 画面の最大文字数

## レビューポイント
- 対応内容の認識はあっていますでしょうか？

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2024-01-29 05:54:35 +00:00
makabe.t
271d85482d Merged PR 705: 対応 
## 概要
[Task3549: 対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3549)

- 第五階層のユーザーがライセンス注文する際にディーラーが設定されていない場合に出るエラーを専用のメッセージとなるように修正しました。
※文言はメッセージレビュー後に必要があれば修正します。

## レビューポイント
- エラーの仕分け方は適切でしょうか？

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

## 動作確認状況
- ローカルで確認
 2024-01-26 08:03:18 +00:00
SAITO-PC-3\saito.k
f2f8728319 ccbの競合解消漏れを対応 2024-01-24 11:15:27 +09:00
SAITO-PC-3\saito.k
ded446de93 Merge branch 'develop' into ccb 
# Conflicts:
#	dictation_client/src/pages/UserListPage/index.tsx
#	dictation_server/package.json
#	dictation_server/src/common/test/init.ts
#	dictation_server/src/features/auth/auth.service.spec.ts
#	dictation_server/src/features/files/files.service.spec.ts
#	dictation_server/src/features/licenses/licenses.service.spec.ts
#	dictation_server/src/features/tasks/tasks.service.spec.ts
#	dictation_server/src/features/terms/terms.service.spec.ts
#	dictation_server/src/features/users/users.service.spec.ts
#	dictation_server/src/features/workflows/workflows.service.spec.ts
 2024-01-24 10:46:48 +09:00
makabe.t
1524ec2473 Merged PR 701: パイプラインエラー対応 
## 概要
[Task3531: パイプラインエラー対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3531)

- パイプラインエラー解消

## レビューポイント
- 共有
 2024-01-24 00:48:21 +00:00
saito.k
a59cb0fffb Merged PR 700: [テストFB対応]User画面でライセンスのStatusの表示を変更したい 
## 概要
[Task3506: テスト対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3506)

- ライセンスが割り当たっている状態の表示をLicense Assignedにする
- ライセンスの期限切れ状態の表示をNo Licenseとする
- ヘッダーのOMDSCloudの表記を削除

## レビューポイント
- 修正内容に不足はないか
- 修正の認識ずれはないか
- ほかに修正が影響している箇所はないか

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

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

## 補足
- 相談、参考資料などがあれば
 2024-01-23 11:02:50 +00:00
maruyama.t
1ad3cb70c6 Merged PR 687: アカウント登録画面修正 
## 概要
[Task3437: アカウント登録画面修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3437)

- アカウント登録時にPrivacyNoticeに同意できるよう修正（サーバに処理自体は入っていたが、画面修正だけが漏れていた）
- アカウント登録画面と利用規約同意画面で文言が統一されていなかったのを修正。
- 利用規約同意画面で、規約に同意する文言のlabel→label htmlForに修正
（文言にマウスオーバーするとカーソルが矢印から指に代わるが、チェックボックスを押さないかぎりチェックが付かなかったのを、文言を押してもチェックが付くように修正）

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

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

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

## 補足
- 相談、参考資料などがあれば
 2024-01-17 06:06:08 +00:00
makabe.t
7075c2a9eb Merged PR 686: いろいろ消す＆表示変える 
## 概要
[Task3328: いろいろ消す＆表示変える](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3328)

- CCB後回し分の部分について表示を削除、修正しました。
  - ユーザー一覧画面の「Delete user」を消す
  - タスク一覧画面の「Delete dictation」を削す
  - テンプレートファイル一覧画面の「Delete」を消す
  - アカウント情報画面の「File Delete Setting」を消す
  - パートナー一覧画面の「Delete Account」を消す
  - ライセンスページの「Storage Available」と「Storage Used」の値をハイフン表示に修正
※変更部分はまた使う予定なのでコメントアウトで対応しています。

## レビューポイント
- 対応箇所は認識通りでしょうか？
- 対応内容は認識通りでしょうか？

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

## 動作確認状況
- ローカルで確認
 2024-01-16 10:37:12 +00:00
maruyama.t
6b1020be8f Merged PR 690: dev動作確認&確認後コンソールログのloginResultを削除する 
## 概要
[Task3478: dev動作確認&確認後コンソールログのloginResultを削除する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3478)

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

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

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

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

## 補足
- 相談、参考資料などがあれば
 2024-01-16 08:32:25 +00:00
makabe.t
d08c6c99af Merged PR 681: タスク削除API実装 
## 概要
[Task3457: タスク削除API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3457)

- タスク削除APIとUTを実装しました。

## レビューポイント
- テストケースは適切でしょうか？
- リポジトリでの削除処理は適切でしょうか？
- エラー時のコード使い分けは適切でしょうか？

## UIの変更
- なし
## 動作確認状況
- ローカルで確認
 2024-01-16 07:55:38 +00:00
maruyama.t
4c527b1959 Merged PR 685: authページでinstanceの取得に失敗した場合でもログインページに遷移させる 
## 概要
[Task3476: authページでinstanceの取得に失敗した場合でもログインページに遷移させる](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3476)

以下暫定対応です。
`const loginResult = await instance.handleRedirectPromise();`

推測としては、この処理でloginResultがnullになってしまうことがあり、
結果、ページ遷移などが発生しないままになってまうため、loading表示で固まる。
そのため、いかなる場合でも最終的にはログインページに遷移させるよう修正した。
これによって、loginResultがnullの場合は、ログインページに遷移後トップページに遷移し、ユーザは再度ログイン処理を行うことができる。

一旦この状態でテストは継続していただいて、根本原因調査は裏で行う。
## レビューポイント
- console.log({ loginResult }); はDEV環境での動作確認後に削除いたします。

## UIの変更
なし

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

## 補足
- 相談、参考資料などがあれば
 2024-01-16 06:55:11 +00:00
makabe.t
8793606070 Merged PR 682: タスク一覧画面修正 
## 概要
[Task3458: タスク一覧画面修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3458)

- タスク一覧画面のタスク削除ボタンからタスクを削除する処理を実装しました。
  - タスクがInProgress、ユーザーがTypistの場合にはボタンを非活性となるようにしています。

## レビューポイント
- エラーごとの処理内容は適切でしょうか？
- ボタンの活性制御は適切でしょうか？

## UIの変更
- なし
## 動作確認状況
- ローカルで確認
 2024-01-16 00:17:45 +00:00
saito.k
d627bb8a09 Merged PR 684: タスク一覧の表示件数を増やす 
## 概要
[Task3447: タスク一覧の表示件数を増やす](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3447)

- タイトルの通り

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

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

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

## 補足
- 相談、参考資料などがあれば
 2024-01-15 08:54:14 +00:00
makabe.t
81c299dd99 Merged PR 680: タスク削除API IF実装 
## 概要
[Task3456: タスク削除API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3456)

- タスク削除APIのIFを実装しopenapi.jsonを更新しました。

## レビューポイント
- パス、バリデータは想定通りでしょうか？

## UIの変更
- なし
## 動作確認状況
- ローカルで確認
 2024-01-11 08:47:28 +00:00
makabe.t
6111583678 Merged PR 673: ログイン画面へのブラウザバック時のローディング対応 
## 概要
[Task3380: ログイン画面へのブラウザバック時のローディング対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3380)

- ログイン済み後にブラウザバックした際にローディング画面からログイン後の画面に遷移する処理を追加しました。
  - loginPage
  - AuthPage
  - TermsPage

## レビューポイント
- 対応ページと対応する箇所は適切でしょうか？
  - 既存のuseEffectの中に処理を追加しています。

## UIの変更
- なし

## 動作確認状況
- ローカルで確認
 2024-01-11 08:22:31 +00:00
saito.k
c4efaf1a1a Merged PR 671: 「LicenseAlert」を消す対応 
## 概要
[Task3381: 対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3381)

- 以下の画面から「LicenseAlert」を削除
  - ユーザー追加Popup
  - ユーザー更新Popup
  - ユーザー一覧画面
- 以下のAPIの引数から「LicenseAlert」の設定をするパラメータを削除
  - ユーザー作成API（users/signup）
  - ユーザー更新API（users/update）
- ユーザー一覧取得APIのレスポンスから「LicenseAlert」を削除
- usersEntityから「LicenseAlert」を削除
- クライアントのAPI呼び出し部分を自動生成するopenapi-generator-cliのバージョンを7.1.0で固定
  - 最新バージョン（7.2.0）はaxiosのバージョン1.x用になるため
  - https://github.com/OpenAPITools/openapi-generator/releases/tag/v7.2.0

## レビューポイント
- openapi-generator-cliのバージョンを固定するのではなく、axiosのバージョンを上げたほうがよいか
  - axiosのバージョン1.xが出たのがおととしになるのでそろそろ上げてもいい？
  - このタイミングでメジャーバージョンを上げるのはどうか
    - 基本的な仕様について大きくは変わってない模様
    - 実際に上げてみて軽く動作確認してみたが問題なかった

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

## 補足
- 相談、参考資料などがあれば
 2024-01-11 06:58:01 +00:00