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

14 Commits

Author SHA1 Message Date
saito.k
df55be5e19 Merged PR 917: 画面遷移してもインターバルが初期化されないように修正 
## 概要
[Task4248: 画面遷移してもインターバルが初期化されないように修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/4248)

- UpdateTokenTimerを修正
  - F5キーリロードの時もトークンをチェックする
- UpdateTokenTimerを各画面ごとではなくApp.tsxに配置する
  - 画面遷移のたびにSetIntervalが初期化されるのを防ぐ
- 各画面のUpdateTokenTimerを削除
  - App.tsxで管理するため

## レビューポイント
- いかの問題が解消されるかを可能であれば確認してほしい。
  - Dictation画面でトークンのチェック～自動更新ができること
  - 画面遷移をしてもトークンチェックのインターバルが初期化されないこと
  - F5キーリロードをした時にトークンチェックが走ること

## 動作確認状況
- ローカルで確認
- 行った修正がデグレを発生させていないことを確認できるか
  - 具体的にどのような確認をしたか
    - 各画面で取得系の処理が正しく動作することを確認

## 補足
- 相談、参考資料などがあれば
 2024-07-02 06:43:41 +00:00
saito.k
deb3431d74 Merged PR 531: 画面実装（代行操作） 
## 概要
[Task2909: 画面実装（代行操作）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2909)

- ディーラーユーザーが代行操作用トークンを取得して、第五階層ユーザーの代行操作ができる実装をしました。
  - 代行後に各対象タブが代行操作デザインで表示され、代行対象として操作できるようになっています。
    - 各APIの呼び出しについて代行操作用トークンがあればそちらを使うように実装しています。
 - 代行操作のタブ、ページ移動時に代行操作を維持するために遷移をリンクから`useNavigate`に変更しました。

## レビューポイント
- 代行操作用トークンの取り扱いについて
  - APIからのトークン取得後、`store.auth`に代行操作用トークンを保存し、利用時には関数を使って間接的に呼でいますが構成として不自然な点はないでしょうか？
    - トークン取得関数では代行操作用トークンがあればそれを、なければ通常のトークンを取得するようにしています。
      - これはAPIのトークンを設定する際にトークンを取得側では代行操作中か意識せずに一つの関数を呼べばいいようにするためです。
  - 代行操作用トークンの保存は通常のトークン保存と同様に`operation`から`Slice`に設定したSet関数を呼ぶことでstateに保存していますが、使い方として気になることはないでしょうか？

※代行操作中に表示するタブの制限と代行操作トークンの更新処理は別タスクでの実施予定です。

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

## 動作確認状況
- ローカルで確認
 2023-11-06 08:28:40 +00:00
maruyama.t
e9541e22a2 Merged PR 523: 画面修正（ヘッダー右上にユーザー名を表示） 
## 概要
[Task2920: 画面修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2920)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
ログイン時に、API（getMyUser）を呼び出し、フラグをtrueにして結果をstateに保存する。
フラグで管理する目的は、ページ遷移時に再度API呼び出しを行わないようにするため。

- このPull Requestでの対象/対象外
サインアウト部分は対象外→別タスクにて対応

- 影響範囲（他の機能にも影響があるか）
既存の各ページのヘッダー部分

## レビューポイント
- component/header配下にSliceを追加すると循環参照になってしまうのを避けるため、feature/login配下にSlice及び各処理を追加した、妥当か。

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

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

## 補足
- 相談、参考資料などがあれば
 2023-10-31 09:02:19 +00:00
saito.k
375e3a5a3b Merged PR 524: 画面実装（代行操作中表示コンポーネント＋代行操作中に表示する画面のデザイン） 
## 概要
[Task2908: 画面実装（代行操作中表示コンポーネント＋代行操作中に表示する画面のデザイン）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2908)

- 代行操作中に表示する画面に対して、代行操作中のデザインを反映
- 代行操作中に画面上部に表示するバーのコンポーネントを作成
- 代行操作用のトークンをauthStateに追加

## レビューポイント
- 代行操作バーのコンポーネントに表示する会社名をpropsとしたが良いか
  - このコンポーネントを表示する画面で会社名を取得して、会社名とトークンがあれば表示するという風にしたいから
- 代行操作用トークンはauthStateでよいと思っているが認識会うか

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

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

## 補足
- 相談、参考資料などがあれば
 2023-10-26 06:35:59 +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
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
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
saito.k
7a453c80f8 Merged PR 372: 画面実装（WorktypeID設定画面） 
## 概要
[Task2506: 画面実装（WorktypeID設定画面）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2506)

- WorktypeID設定画面を実装
  - WorktypeID一覧を表示する
  - 各種ボタンを表示する
  - ActiveWorktypeIDのセレクトボックスを表示する

- 言語切り替えの対応で同一の文言をcommonにまとめ、該当箇所を修正

## レビューポイント
- stateのdomain配下にあるAPIからの戻り値をOptionalにしたが問題ないか
  - 必須にすると画面初期表示時に「０件です」がちらつくため

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

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

## 補足
- Returnボタン以外のボタンの挙動はレビュー対象外
- Active WorkTypeIDの挙動はレビュー対象外
 2023-09-01 07:27:36 +00:00
makabe.t
c85ecbfc90 Merged PR 341: 画面実装（ルーティングルール一覧画面） 
## 概要
[Task2410: 画面実装（ルーティングルール一覧画面）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2410)

- ワークフロー画面と遷移先のタイピストグループ設定画面を実装しました。
  - ワークフロー画面はタブ制御とリンク以外は仮です。
    - 管理者ユーザーでログインした時だけタブが見えるようにしています。
  - タイピストグループ設定画面は空の仮画面です。
- 画面コンポーネントをルーティングしました。
  - /workflow
  - /workflow/typist-group

## レビューポイント
- タブ制御は適切か
- URL設計は適切か

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

## 動作確認状況
- ローカルで確認
 2023-08-23 02:32:36 +00:00