makabe.t
|
cbf7622909
|
Merged PR 675: File Propertyの「priority」をHighかNormalで表示する
## 概要
[Task3433: File Propertyの「priority」をHighかNormalで表示する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3433)
- FilePropertyポップアップのpriorityの項目の表示を数字からHigh/Normalとなるように修正しました。
## レビューポイント
- 表示内容は認識通りでしょうか?
## UIの変更
- [Task3433](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/Task3433?csf=1&web=1&e=5iWTtn)
## 動作確認状況
- ローカルで確認
|
2024-01-10 06:12:15 +00:00 |
|
masaaki
|
7652e5ef08
|
Merged PR 662: [OMDS様要望] タスク一覧画面の日時表示を変えたい(修正タスク)
## 概要
[Task3362: [OMDS様要望] タスク一覧画面の日時表示を変えたい(修正タスク)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3362)
- 以下の画面について、日時の表示形式をブラウザのロケールに合わせて変換し、時差を考慮した内容になるよう修正しました。
- タスク一覧画面
- 音声メタ情報ポップアップ
- 注文履歴画面(日付のみ表示)
- サーバからUTCでのYYYY/MM/DDの文字列として返却していたので、ISOStringとして返却して画面で表示内容を変換するよう処理を修正しました
- タスク一覧画面のFile Lengthについて、hh:mm:ss形式となるよう修正しました
## レビューポイント
- formatMillisecondsToHHMMSSについて、他で使うシーンがなかったのでdictation内の関数としたが、全体の関数とした方がよいか
- convertUtcToLocalについて不正な日付の文字列が渡された場合、変換せずに元の値を返却しているが、エラー対応として問題ないか
## 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/Task3362?csf=1&web=1&e=Nc7kPV
## 動作確認状況
- ローカルで確認、サーバ側はnpm run test実施済
## 補足
- 相談、参考資料などがあれば
|
2023-12-27 02:24:39 +00:00 |
|
水本 祐希
|
f4da949bfd
|
Merged PR 656: Revert 'タスク一覧のパフォーマンス調査'
## 概要
[Task3346: タスク一覧のパフォーマンス調査](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3346)
- 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず)
- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
- 影響範囲(他の機能にも影響があるか)
## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
## 動作確認状況
- ローカルで確認
Reverts !651
|
2023-12-22 07:09:18 +00:00 |
|
水本 祐希
|
01dda79d72
|
Merged PR 651: タスク一覧のパフォーマンス調査
## 概要
[Task3346: タスク一覧のパフォーマンス調査](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3346)
- 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず)
- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
- 影響範囲(他の機能にも影響があるか)
## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
## 動作確認状況
- ローカルで確認
|
2023-12-22 06:12:13 +00:00 |
|
saito.k
|
16f743c4c4
|
Merged PR 650: 最新の翻訳情報をWebアプリに適用する
## 概要
[Task3357: 最新の翻訳情報をWebアプリに適用する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3357)
- 最新の翻訳情報を反映
## レビューポイント
- 特になし
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認、develop環境で確認など
## 補足
- 以下の翻訳情報はまだもらえていないので、残っている
```
タスク一覧
dictationPage.message.backupFailedError
dictationPage.message.cancelFailedError
dictationPage.label.fileBackup
dictationPage.label.downloadForBackup
dictationPage.label.applications
共通
common.label.headerSupport
ヘルプページ全般
ライセンス情報
LicenseSummaryPage.label.licenseLabel
LicenseSummaryPage.label.storageLabel
|
2023-12-21 08:52:45 +00:00 |
|
湯本 開
|
1bc6618a6d
|
Merged PR 643: ライセンス引き戻し完了通知 [U-109] の実装
## 概要
[Task3306: ライセンス引き戻し完了通知 [U-109] の実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3306)
- ライセンス引き戻し完了のメール送信機能を追加しました。
- テストでメール送信しないようSendGridのメソッドを上書きする処理を追加しました。
## レビューポイント
- テンプレートの適用内容に不自然な点はないか
- アカウントのFromとToとCCに関わる部分で認識違いはないか
- `orderedAccountId` という引数には注文したアカウント=下位階層のアカウントが入るという理解であっているか等
## UIの変更
- なし
## 動作確認状況
- npm run testは通過
|
2023-12-20 07:54:47 +00:00 |
|
masaaki
|
48ff009d39
|
Merged PR 630: ファイルプロパティをcloseするとたすく一覧が更新される
## 概要
[Task3333: ファイルプロパティをcloseするとたすく一覧が更新される](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3333)
- File Property画面にて「Close」ボタンを押下した際に画面がリロードされる問題について対応しました。
「Close」ボタン押下時に「href=""」としていたため、デフォルトのURL遷移(/dictations)が行われていたのを、
onClickによるポップアップ閉じる処理を呼び出す実装に変更しました。
## レビューポイント
- 特にありません
## UIの変更
- 変更なし
## 動作確認状況
- ローカルで確認済
## 補足
- 相談、参考資料などがあれば
|
2023-12-19 05:13:32 +00:00 |
|
makabe.t
|
b2fef69ea9
|
Merged PR 635: アカウント登録完了通知 [U-101] の実装
## 概要
[Task3301: アカウント登録完了通知 [U-101] の実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3301)
- アカウント登録完了(認証完了)後にメール送信をする機能を追加しました。
- 合わせてテスト修正をしています。
## レビューポイント
- テンプレートの適用は適切でしょうか。
- テスト修正で対象Sendgridメソッドを上書きしていますが対応として不自然な点はないでしょうか?
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-12-19 02:00:35 +00:00 |
|
makabe.t
|
4d325d1751
|
Merged PR 638: ライセンス発行依頼受付通知 [U-105] の実装
## 概要
[Task3302: ライセンス発行依頼受付通知 [U-105] の実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3302)
- ライセンス注文時のメール通知処理を既存処理に追加しました。
## レビューポイント
- テンプレートの適用に問題はないでしょうか?
- メールに渡す値の取得処理は認識通りでしょうか?
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-12-19 01:16:50 +00:00 |
|
maruyama.t
|
afa05f381c
|
Merged PR 628: 第5階層で規約同意画面を表示する際に、DPAに同意が一瞬表示される
## 概要
[Task3281: 【Commit諸作業終わったら優先対応】第5階層で規約同意画面を表示する際に、DPAに同意が一瞬表示される](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3281)
selectorから別途tierの値が入っているかをselectIsTierLoadingで取得し、まだ入っていない場合(tier===0)Loadingの表示を行うようにした。
```
export const selectIsTierLoading = (state: RootState) =>
state.terms.domain.tier === 0;
```
## レビューポイント
- とくになし
## 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/Task3281?csf=1&web=1&e=D9t5di
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-12-18 04:07:03 +00:00 |
|
水本 祐希
|
8fe649cb7f
|
Merged PR 618: 画面修正(第5のライセンス一覧の項目順を変更)
## 概要
[Task3271: 画面修正(第5のライセンス一覧の項目順を変更)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3271)
- 何をどう変更したか、追加したライブラリなど
- 項目の配置を変更
- License Inventory→Number of unused licenses→Number of licenses available for reuse→Number of licenses allocatedの順で修正
- Total number of licenses on orderをlicense情報の一番下に配置
- 不要な項目を削除
- Total number of orderを削除
- レイアウトを最新のものに適用
- このPull Requestでの対象/対象外
項目名の変更は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/Task3271?csf=1&web=1&e=ac0SrH
## 動作確認状況
- ローカルで確認
|
2023-12-12 02:08:18 +00:00 |
|
湯本 開
|
375e8e87a8
|
Merged PR 621: STG/PRODでSourceMap出力をOFFにしておく
## 概要
[Task3165: STG/PRODでSourceMap出力をOFFにしておく](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3165)
- STG/PRODでsourcemap(ビルド後の圧縮・結合されたjsからデバッグ用の実行行やコメントを復元するための存在)を出力しないように修正
- コメント等がユーザーに見えないようにする目的
## レビューポイント
- 情報共有
## 動作確認状況
- ローカルで確認
|
2023-12-11 06:35:59 +00:00 |
|
maruyama.t
|
f7ec740e7a
|
Merged PR 615: PrivacyNoticeの多言語対応の文言修正
## 概要
[Task3292: PrivacyNoticeの多言語対応の文言修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3292)
- 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず)
- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
- 影響範囲(他の機能にも影響があるか)
## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認、develop環境で確認など
## 補足
- 相談、参考資料などがあれば
|
2023-12-08 05:59:30 +00:00 |
|
makabe.t
|
a0da277c05
|
Merged PR 607: メールアドレス入力チェックの修正
## 概要
[Task3232: メールアドレス入力チェックの修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3232)
- メールアドレス入力に対して規約に則ったチェックを実施するように修正しました。
- 画面での入力チェックとAPIのバリデーションを修正しています。
## レビューポイント
- クライアントのチェック内容は規約に則った内容になっているでしょうか?
- 正規表現の構文として不自然な点はないでしょうか?
- サーバー側のチェックをIsEmail+禁止文字の追加という形で対応していますが適切でしょうか?
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-12-08 02:14:25 +00:00 |
|
水本 祐希
|
c246586498
|
Merged PR 602: ライセンス確認画面(第五階層)に会社名を表示する(client側)
## 概要
[Task3225: ライセンス確認画面(第五階層)に会社名を表示する(client側)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3225)
- 何をどう変更したか、追加したライブラリなど
ライセンス確認画面(第5階層)の会社名取得APIを呼び出す実装を追加。
- このPull Requestでの対象/対象外
server側の実装は別ブランチで行うためここでは対象外
## レビューポイント
特になし
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認(完了)
|
2023-12-06 02:10:01 +00:00 |
|
maruyama.t
|
94f34a0fde
|
Merged PR 606: 画面修正(Terms画面)
## 概要
[Task3210: 画面修正(Terms画面)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3210)
[Task3211:API修正(バージョン取得API)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_sprints/taskboard/OMDSDictation%20%E3%83%81%E3%83%BC%E3%83%A0/OMDSDictation/%E3%82%B9%E3%83%97%E3%83%AA%E3%83%B3%E3%83%88%2023-1?workitem=3211)
[Task3212:API修正(バージョン更新API))](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_sprints/taskboard/OMDSDictation%20%E3%83%81%E3%83%BC%E3%83%A0/OMDSDictation/%E3%82%B9%E3%83%97%E3%83%AA%E3%83%B3%E3%83%88%2023-1?workitem=3212)
- このPull Requestでの対象/対象外
Click here to read the terms of use.の文言は多言語対応の対象のため、現在一律同じ文言がでます。
第一~第四階層は
上からEULA,PrivacyNotice,DPAが表示されています
第五階層は、
上から、PrivacyNotice,DPAが表示されています
- 影響範囲(他の機能にも影響があるか)
ユーザアーカイブテーブルにPrivacyNoticeのバージョンを追加
## レビューポイント
同意済みプライバシーポリシーはユーザーアーカイブの対象だと認識しているが正しいか。
termsテーブルのdocument_typeの値をPrivacyNoticeにしているが、PRIVACY_NOTICEにしたほうがよいか。
ユニットテストに不足はないか。
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
https://ndstokyo.sharepoint.com/sites/Piranha/Shared%20Documents/Forms/AllItems.aspx?csf=1&web=1&e=hzPw9b&cid=7737ed1b%2D0eb4%2D4331%2Da238%2D14dd35b27e18&FolderCTID=0x012000C0DCEE65AC2177479C3C761CD137C9C9&id=%2Fsites%2FPiranha%2FShared%20Documents%2FGeneral%2FOMDS%2F%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%2FTask3210&viewid=786a81cf%2Dd15f%2D4dc2%2D9e55%2Dc7a729fbc72f
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-12-05 09:10:49 +00:00 |
|
makabe.t
|
62a5155ee1
|
Merged PR 608: develop動作確認不具合対応
## 概要
[Task3238: develop動作確認不具合対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3238)
- タスク一覧画面にあるアプリケーションインストールボタンのラベルを翻訳対応しました。
## レビューポイント
- 共有
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-12-05 08:10:58 +00:00 |
|
maruyama.t
|
ebfc05040b
|
Merged PR 604: 画面実装(ファイル情報ポップアップ画面実装)
## 概要
[Task3195: 画面実装(ファイル情報ポップアップ画面実装)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3195)
- 影響範囲(他の機能にも影響があるか)
画面実装のみのため、なし
## レビューポイント
- 表示される値はDictationタブの表示のされ方と同じだが問題ないか
(拡張子やサイズなど)
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
https://ndstokyo.sharepoint.com/sites/Piranha/Shared%20Documents/Forms/AllItems.aspx?csf=1&web=1&e=hzPw9b&cid=f9d17150%2D1d33%2D49da%2D8bd4%2Df5a27cbbed66&FolderCTID=0x012000C0DCEE65AC2177479C3C761CD137C9C9&id=%2Fsites%2FPiranha%2FShared%20Documents%2FGeneral%2FOMDS%2F%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%2FTask3195&viewid=786a81cf%2Dd15f%2D4dc2%2D9e55%2Dc7a729fbc72f
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-12-05 05:04:25 +00:00 |
|
masaaki
|
cca9cf6e5e
|
Merged PR 601: ヘルプページ画面実装
## 概要
[Task3204: ヘッダ修正対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3204)
- ヘルプページを追加して、ヘッダのタブに表示されるようにしました。
- ヘッダのタブはロールの制限なく見えるようにしています。
- ヘルプページからさらに個別のヘルプへのリンクは仮のURLにしています。
- 動作としては別タブで開くようにしています。
## レビューポイント
- パスをタブ名に合わせて`/support`としましたが不自然な点はないでしょうか?
- 仮のURLがxxxとなっているため、別タブで開かれたページがNot Foundとなってしまっていますが問題ないでしょうか?
## UIの変更
- [Task3204](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/Task3204?csf=1&web=1&e=N9yn0D)
## 動作確認状況
- ローカルで確認
|
2023-12-05 02:40:09 +00:00 |
|
水本 祐希
|
cc36b806c5
|
Merged PR 596: 第一階層のshortage(-)が赤字となる
## 概要
[Task3169: 第一階層のshortage(-)が赤字となる](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3169)
- 何をどう変更したか、追加したライブラリなど
第一階層のshotage"-"が赤文字にならないよう修正
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-11-29 06:06:04 +00:00 |
|
水本 祐希
|
c3750e3c1c
|
Merged PR 591: 修正対応
## 概要
[Task3158: 修正対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3158)
- 何をどう変更したか、追加したライブラリなど
shotageの値が1以上の場合、赤文字になる
## 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/Task3158?csf=1&web=1&e=3qaLfD
## 動作確認状況
- ローカルで確認
|
2023-11-28 06:35:56 +00:00 |
|
makabe.t
|
8b04adf095
|
Merged PR 587: 画面実装(ダウンロード処理)
## 概要
[Task3122: 画面実装(ダウンロード処理)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3122)
- ファイルバックアップポップアップからのダウンロード実行処理を実装しました。
- ファイルURL取得をAdminが適切に実施できるように権限を修正しました。
- SASトークンの開始時刻を修正しました。
- ダウンロード処理を実行できるようにするためにblobストレージのCORS設定を修正しました。
- faviconとタイトルを正式なものに差し替えました。
## レビューポイント
- faviconはアセットフォルダに一緒に入れてしまっていますが配置場所として問題ないでしょうか?
- `Operation`内でループで一件ずつダウンロード・バックアップ処理を実行していますが認識あっていますでしょうか?
- SASトークンの生成時の開始時刻10秒前にしていますが問題ないでしょうか?
- SASトークン付きURLの発行直後にURLでダウンロード実行するとSASトークンエラーとなることがありましたので、その対応です。
- 発行直後に使うはずなのでSASトークンの開始を10秒前にしても影響はないはずと考えています。
- ほかの対応としてトークン取得後に数秒待つことも考えましたが、動作全体が遅くなってしまうのとSASトークンの作りの問題だとがんが得ているのでこのような対応をしています。
## UIの変更
- [Task3122](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/Task3122?csf=1&web=1&e=2JPi0J)
## 動作確認状況
- ローカルで確認
|
2023-11-28 06:21:23 +00:00 |
|
masaaki
|
501fc5a15d
|
Merged PR 588: 画面実装(タスク一覧画面:タスクキャンセル操作追加)
## 概要
[Task3131: 画面実装(タスク一覧画面:タスクキャンセル操作追加)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3131)
- タスク 3131: 画面実装(タスク一覧画面:タスクキャンセル操作追加)
- タスク一覧に対してCancel Dictationボタンを実装しました
## レビューポイント
- タスク関連のほかの処理で実施しているものについて、キャンセルで実施必要か確認いただきたいです。
- キャンセル後のデスクトップアプリ起動
- 不要の認識
- キャンセル前のソート条件保存
- ステータスが変わるため、必要な認識
- ログイン名を取得するためにloginのselectorを使用していますが問題ないか確認いただきたいです
(他フィーチャと余計な関連を作るのはあまりよくないと思いつつ、同じような取得処理を作るのも嫌だと思いつつ。。)
## 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/Task3131?csf=1&web=1&e=EdWfPJ
## 動作確認状況
- ローカルで確認実施
## 補足
- 相談、参考資料などがあれば
|
2023-11-22 01:13:03 +00:00 |
|
makabe.t
|
68aaa0e548
|
Merged PR 586: バックアップAPI実装
## 概要
[Task3121: バックアップAPI実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3121)
- タスクのバックアップAPIとテストを実装しました。
- 実装に当たり、タスクエンティティ定義の型を修正しています。
## レビューポイント
- テストケースに過不足はないでしょうか?
- エンティティの値を取得した際に`is_job_number_enabled`が数値となってしまうのでtypeをBooleanに変更しましたが問題ないでしょうか?
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-11-20 08:57:56 +00:00 |
|
makabe.t
|
99ac6be9fd
|
Merged PR 585: 画面実装(ポップアップ表示)
## 概要
[Task3120: 画面実装(ポップアップ表示)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3120)
- ディクテーション画面から開くバックアップポップアップを実装しました。
- ポップアップを開いて、対象タスクが表示されるところまでの実装です。
- バックアップボタンの挙動は対象外です。
## レビューポイント
- チェックボックスの挙動は適切でしょうか?
- SharePointの挙動を参考にしています。
- デザインの適用で不自然な点はないでしょうか?
## UIの変更
- [Task3120](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/Task3120?csf=1&web=1&e=KxCeU4)
## 動作確認状況
- ローカルで確認
|
2023-11-20 07:52:43 +00:00 |
|
maruyama.t
|
35923f84e2
|
Merged PR 576: dev環境不具合修正
## 概要
[Task3046: dev環境不具合修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3046)
- 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず)
- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
- 影響範囲(他の機能にも影響があるか)
## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認、develop環境で確認など
## 補足
- 相談、参考資料などがあれば
|
2023-11-13 06:01:49 +00:00 |
|
makabe.t
|
e228d06cc7
|
Merged PR 571: アカウント画面で管理者設定している箇所の対応
## 概要
[Task3054: アカウント画面で管理者設定している箇所の対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3054)
- アカウント画面での管理者設定でEmail認証していないユーザーを設定しないように対応しました。
- 管理者選択のドロップダウンに表示されるユーザーをクライアント側で認証済みでフィルタする対応
- アカウント情報更新APIで管理者に未認証ユーザーを設定しようとするとエラーとなるように修正
## レビューポイント
- ユーザー取得は画面側でフィルタしているが対応としては適切でしょうか?
- 同じAPIをフィルタせずに使うところもあるため
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-11-10 08:09:09 +00:00 |
|
湯本 開
|
7421203bc4
|
Merged PR 567: テスト用Attributeを追加
## 概要
[Task3060: テスト用Attributeを追加](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3060)
- E2Eテストから指定しやすいように代表的な要素(ログアウトボタン等)に一意の名前を付ける実装を追加
- `data-*` はカスタムデータ属性と呼ばれるもので、「標準的な取り扱い方」が存在しない属性
- https://developer.mozilla.org/ja/docs/Learn/HTML/Howto/Use_data_attributes
- 一般的に、E2Eテストで「このボタン」とか「このラベル」とかを指定したい時に使われる
- https://docs.cypress.io/guides/references/best-practices
- 本番環境などでも属性としては残したままにする想定だが、`data-test-id` 等の見るからにテスト用の名前を指定すると、テスト処理が入っているような変な誤解を招きそうなので名前は `data-tag` とした
- 実際にはテストにも使える、副作用が特にない属性
## レビューポイント
- 動作が変わるような変更が入り込んでしまっていないか
- タグ名は妥当か
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-11-10 03:50:33 +00:00 |
|
saito.k
|
4b0110856c
|
Merged PR 566: xxxのサンプルページを消し去る
## 概要
[Task3055: xxxのサンプルページを消し去る](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3055)
- 仮で置いてあったサンプルページ削除
- タブからも削除
- ルーティングからも削除
- ログイン成功時、階層とロールに応じて遷移する画面を変更する処理を追加
## レビューポイント
- 遷移先を変更する処理は関数に切り出した方が良いか?
- ほかで使わないからべた書きでも良いかなと思い、こうしました。
## 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/Task3055?csf=1&web=1&e=B0E9Ud
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-11-10 02:22:34 +00:00 |
|
makabe.t
|
6d163a9067
|
Merged PR 565: 代行操作中はサインアウトを非活性or非表示にする
## 概要
[Task3066: 代行操作中はサインアウトを非活性or非表示にする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3066)
- 代行操作中はサインアウトボタンが非活性となるように修正しました。
- ログアウト時に自ユーザー情報取得APIを呼ばないように修正しました。
- アクセストークンの削除後に呼ばれ、401エラーになるため
## レビューポイント
- 代行操作中はサインアウトボタンのマウスイベントを無効にすることで非活性としていますが対応として適切でしょうか?
- ログアウト時(アクセストークンがない場合)はヘッダで自ユーザー情報取得APIを呼ばないように修正しましたが、処理として適切でしょうか?
- エラーになる呼び出しはしないようにしたい&使っているのはヘッダでのみなのでほかの箇所への影響はない認識です。
## UIの変更
- [Task3066](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/Task3066?csf=1&web=1&e=3ztNOy)
## 動作確認状況
- ローカルで確認
|
2023-11-09 06:40:14 +00:00 |
|
oura.a
|
09ff4327ee
|
Merged PR 564: Revert "Merged PR 548: ログイン回り修正
Revert "Merged PR 548: ログイン回り修正
## 概要
[Task1828: IDトークンを一度しか使えないようにする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1828)
- 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず)
ログイン時にRedisを参照し、同じIDトークンでアクセスされていた場合はアクセスを拒否する。
初めて認証で使われるIDトークンの場合は、Redisに保存する(有効期限は300秒(IDトークンの有効期限の最小値))
ログイン後、クライアントはローカルストレージの不要な情報を破棄する(accessToken,refreshToken,displayInfo以外)
- 影響範囲(他の機能にも影響があるか)
新規機能のため、なし。
## レビューポイント
- keyの形式id-token:{idトークンの中身}は想定通りか。
- ログイン後のローカルストレージの状態が想定通りか。
- controllerやservice層の実装箇所が妥当か。
- stg環境、本番環境のIDトークンの有効期限を最小値に設定するタスクを備忘用PBIに切り出しました。
https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_sprints/taskboard/OMDSDictation%20%E3%83%81%E3%83%BC%E3%83%A0/OMDSDictation/%E3%82%B9%E3%83%97%E3%83%AA%E3%83%B3%E3%83%88%2021-1?workitem=3041
## 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/Task1828?csf=1&web=1&e=1Sc1VP
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば"
Reverted commit `0cdb0f42`.
|
2023-11-08 02:55:28 +00:00 |
|
水本 祐希
|
0cdb0f4267
|
Merged PR 548: ログイン回り修正
## 概要
[Task1828: IDトークンを一度しか使えないようにする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1828)
- 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず)
ログイン時にRedisを参照し、同じIDトークンでアクセスされていた場合はアクセスを拒否する。
初めて認証で使われるIDトークンの場合は、Redisに保存する(有効期限は300秒(IDトークンの有効期限の最小値))
ログイン後、クライアントはローカルストレージの不要な情報を破棄する(accessToken,refreshToken,displayInfo以外)
- 影響範囲(他の機能にも影響があるか)
新規機能のため、なし。
## レビューポイント
- keyの形式id-token:{idトークンの中身}は想定通りか。
- ログイン後のローカルストレージの状態が想定通りか。
- controllerやservice層の実装箇所が妥当か。
- stg環境、本番環境のIDトークンの有効期限を最小値に設定するタスクを備忘用PBIに切り出しました。
https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_sprints/taskboard/OMDSDictation%20%E3%83%81%E3%83%BC%E3%83%A0/OMDSDictation/%E3%82%B9%E3%83%97%E3%83%AA%E3%83%B3%E3%83%88%2021-1?workitem=3041
## 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/Task1828?csf=1&web=1&e=1Sc1VP
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-11-08 02:22:20 +00:00 |
|
makabe.t
|
c9124f8661
|
Merged PR 563: develop動作確認不具合修正
## 概要
[Task3042: develop動作確認不具合修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3042)
- ライセンスページで代行操作中の挙動を修正しました。
- 自アカウント取得APIにガードを設定して実行可能に修正
- 代行操作中は第五階層ユーザーとして扱われるように修正
## レビューポイント
- 共有
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-11-07 10:23:15 +00:00 |
|
makabe.t
|
0212c61bbc
|
Merged PR 550: 画面実装(トークンを定期的に更新する仕組み)
## 概要
[Task2910: 画面実装(トークンを定期的に更新する仕組み)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2910)
- タイマーで定期的に代行操作用のアクセストークンを更新する処理を実装しました。
## レビューポイント
- 通常のアクセストークンのタイマー内で同じタイミングでチェックするように実装していますが分けたほうがいいなどありますでしょうか?
## UIの変更
- [Task2910](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/Task2910?csf=1&web=1&e=g0RdIf)
## 動作確認状況
- ローカルで確認
|
2023-11-07 07:15:25 +00:00 |
|
makabe.t
|
ef4f22029b
|
Merged PR 551: 画面実装(代行操作中のヘッダー表示)
## 概要
[Task2911: 画面実装(代行操作中のヘッダー表示)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2911)
- 代行操作中に表示されるヘッダのタブを以下のタブのみに設定しました。
- User
- Workflow
- License
## レビューポイント
- 代行操作タブ表示の判断材料としてヘッダコンポーネント内でselectorで代行操作の有無を判断してタブ表示用メソッドに渡していますが、データの取り扱いとして不自然なところはないでしょうか?
## UIの変更
- [Task2911](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/Task2911?csf=1&web=1&e=zFW1zj)
## 動作確認状況
- ローカルで確認
|
2023-11-07 06:25:42 +00:00 |
|
makabe.t
|
bc110712b0
|
Merged PR 558: 代行操作の開始/終了時にダイアログを表示
## 概要
[Task3032: 代行操作の開始/終了時にダイアログを表示](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3032)
- 代行操作の開始/終了時に確認ダイアログを表示するように修正しました。
## レビューポイント
- 共有
## UIの変更
- [Task3032](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/Task3032?csf=1&web=1&e=jHFOdq)
## 動作確認状況
- ローカルで確認
|
2023-11-07 05:01:06 +00:00 |
|
makabe.t
|
e38112924d
|
Merged PR 557: ヘッダの表示文言がクッションページで正常に出るように修正
## 概要
[Task3031: ヘッダの表示文言がクッションページで正常に出るように修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3031)
- 未ログイン状態のヘッダについて、翻訳文言の反映が漏れていたので対応しました。
## レビューポイント
- 共有
## UIの変更
- [Task3031](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/Task3031?csf=1&web=1&e=6hBCEH)
## 動作確認状況
- ローカルで確認
|
2023-11-07 01:29:09 +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 |
|
makabe.t
|
d0f4971dc9
|
Merged PR 544: ヘッダの表示文言(タブ・タイトル)を翻訳ファイルを参照するように修正する
## 概要
[Task2895: ヘッダの表示文言(タブ・タイトル)を翻訳ファイルを参照するように修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2895)
- ヘッダに表示するタイトルとタブの文言を翻訳ファイルに定義したラベルを利用するように修正しました。
## レビューポイント
- タブのフィルタ管理用に`key`プロパティを追加したが、構成として適切か
- 表示項目の制御には従来通りの定数を利用したいのでkeyプロパティとして設定
- labelプロパティにタブに表示する文言の翻訳文言を参照するように設定
- 対応ラベルの箇所は適切か(各タブ名+タイトル名)
## UIの変更
- [Task2895](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/Task2895?csf=1&web=1&e=Xs3Tac)
## 動作確認状況
- ローカルで確認
|
2023-11-02 09:41:43 +00:00 |
|
水本 祐希
|
c089060162
|
Merged PR 542: 画面修正(サインアウト本実装)
## 概要
[Task2963: 画面修正(サインアウト本実装)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2963)
- 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず)
- 何をどう変更したか、追加したライブラリなど
**headerにサインアウトの実装を追加**
- この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/Task2963?csf=1&web=1&e=XbInfz
## 動作確認状況
- ローカルで確認
管理者ユーザーかつ第1~5階層、一般ユーザーでログインし、headerに表示されかつサインアウトできることを確認。
## 補足
- 相談、参考資料などがあれば
|
2023-10-31 10:28:43 +00:00 |
|
masaaki
|
93e86e952b
|
Merged PR 541: StaticWebConfigApp.jsonの定義を追加する
## 概要
[Task2928: StaticWebConfigApp.jsonの定義を追加する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2928)
- 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず)
- 何をどう変更したか、追加したライブラリなど
- globalHeadersに対して"X-Frame-Options": "SAMEORIGIN"の設定を追加しました。
- このPull Requestでの対象/対象外
- 対象外なし
- 影響範囲(他の機能にも影響があるか)
- 無し
## レビューポイント
- 特になし
## UIの変更
- 無し
## 動作確認状況
- ローカルで確認済。ただし、ローカルはもともと"X-Frame-Options": "SAMEORIGIN"となっていたのでdev反映後に再度確認実施予定。
## 補足
- 相談、参考資料などがあれば
|
2023-10-31 09:11:46 +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 |
|
湯本 開
|
9c9404367b
|
Merged PR 535: STGビルドの環境変数が間違ったファイルを読まないよう修正する(2)
## 概要
[Task2956: STGビルドの環境変数が間違ったファイルを読まないよう修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2956)
- ビルドコマンドでの引数の渡し方に問題があった部分を修正
## レビューポイント
- 情報共有
## 動作確認状況
- ローカルで確認
|
2023-10-26 08:46:30 +00:00 |
|
湯本 開
|
2a867f2a71
|
Merged PR 533: STGビルドの環境変数が間違ったファイルを読まないよう修正する
## 概要
[Task2956: STGビルドの環境変数が間違ったファイルを読まないよう修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2956)
- .envを上書きする方法から、VITEが本来想定しているビルド指定方法へ修正
- STGパイプラインでも `npm run build:stg` を使用するよう修正
## レビューポイント
- 修正方針に問題はないか
## 動作確認状況
- ローカルで動作する所まで確認
|
2023-10-26 08:20:32 +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 |
|
湯本 開
|
8474c6a4f5
|
Merged PR 529: Pipelineの仕組みを修正する
## 概要
[Task2949: Pipelineの仕組みを修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2949)
- .env.(環境名) の環境変数を用いてフロント側ビルドを行うよう修正
- ビルドパイプラインでstaging/production用の両方のフロント側ビルドを行うよう変更
## レビューポイント
- 仕組み上の不明点はないか
- 情報共有
## 動作確認状況
- 本番環境へデプロイされ、本当の最低限だけ動作することを確認
|
2023-10-26 04:37:58 +00:00 |
|
makabe.t
|
565db8c8b7
|
Merged PR 508: develop動作確認での不具合対応
## 概要
[Task2893: develop動作確認での不具合対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2893)
- ワークタイプが削除済みで、別タブで一覧更新前のワークタイプに対する挙動を修正しました。
- ワークタイプ編集実行時に更新を促すエラーメッセージに修正
- オプションアイテムボタンを押下してもポップアップが開かないようにしています。
## レビューポイント
- 表示メッセージは適切か
- ポップアップ表示制御は適切か
## UIの変更
- [Task2893](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/Task2893?csf=1&web=1&e=duKmyw)
## 動作確認状況
- ローカルで確認
|
2023-10-24 08:09:31 +00:00 |
|
水本 祐希
|
a4dd5addde
|
Merged PR 507: 第五のユーザでログインした際、アカウントタブが表示される
## 概要
[Task2886: 第五のユーザでログインした際、アカウントタブが表示される](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2886)
- 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず)
- 何をどう変更したか、追加したライブラリなど
一般ユーザーがログイン時にアカウントタブを表示しないようにする。
- このPull Requestでの対象/対象外
- 影響範囲(他の機能にも影響があるか)
## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場

## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-10-23 07:15:53 +00:00 |
|
水本 祐希
|
9323cd02e4
|
Merged PR 506: AUTHOR_IDを大文字に自動変換してあげる
## 概要
[Task2376: AUTHOR_IDを大文字に自動変換してあげる](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2376)
- 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず)
- 何をどう変更したか、追加したライブラリなど
AUTHOR_IDを大文字に自動変換するよう実装
入力時に大文字に変換するようにしています。
- このPull Requestでの対象/対象外
- 影響範囲(他の機能にも影響があるか)
## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-10-19 06:50:32 +00:00 |
|
saito.k
|
96848f5e54
|
Merged PR 499: 修正②(files,licenses , Repositoiesのlicenses)
## 概要
[Task2836: 修正②(files,licenses , Repositoiesのlicenses)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2836)
- feature
- files
- licenses
- repositories
- licenses
- users
- worktypes
- entity
- licenses
- audio_files
- audio_option_item
- checkout_permission
- アクセストークンをそのままService層に渡している箇所を修正し、必要なパラメータのみ渡すように修正
- クライアントの型生成
- 割り当て可能ライセンス取得APIのIFが変わったため
## レビューポイント
-
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルでテストが通ることを確認
## 補足
- 相談、参考資料などがあれば
|
2023-10-19 01:04:14 +00:00 |
|