shimoda.m
|
ff5533b647
|
Merged PR 1010: 2025/1/27 本番リリース
|
2025-01-21 05:23:54 +00:00 |
|
shimoda.m
|
b3845187f6
|
Merged PR 1009: Revert "Merged PR 1006: 2025/1/27 PH1エンハンス 本番リリース"
Revert "Merged PR 1006: 2025/1/27 PH1エンハンス 本番リリース"
Reverted commit `b5293888`.
デプロイミスによる切り戻し
|
2025-01-21 04:47:21 +00:00 |
|
shimoda.m
|
b529388871
|
Merged PR 1006: 2025/1/27 PH1エンハンス 本番リリース
|
2025-01-21 02:59:31 +00:00 |
|
x.itou.t
|
a07cfe51aa
|
Merged PR 948: PH1エンハンス先行リリース対応
## 概要
[ユーザー ストーリー 4489: 【PH1エンハンス】Dictation Finishedになったファイルのステータスを変更したい](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation-2nd/_workitems/edit/4489)
[ユーザー ストーリー 4491: 【PH1エンハンス】通知にユーザーIDを付加する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation-2nd/_workitems/edit/4491)
|
2024-11-11 05:23:38 +00:00 |
|
saito.k
|
6b1650a634
|
Merged PR 910: メール文面修正&ユーザー一括登録のテンプレートファイル修正
## 概要
[Task4190: メール文面修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/4190)
[Task4191: ユーザー一括登録のテンプレートファイル修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/4191)
- ユーザー一括追加時に送信するメールの内容を修正
- SCVとなっていたところをCSVに修正
- ユーザー一括追加用のエクセルの項目名を画面上の項目名と合わせる
- auto_renewとなっていたところをauto_assignに修正
## レビューポイント
- 修正内容の認識は合っているか
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認
- 行った修正がデグレを発生させていないことを確認できるか
- 具体的にどのような確認をしたか
- CSV変換のテストが通ることを確認
- メール内容はdev動作確認で確認
## 補足
- 相談、参考資料などがあれば
|
2024-06-04 06:54:42 +00:00 |
|
saito.k
|
5860da285e
|
Merged PR 899: 階層構造変更API修正
## 概要
[Task4179: 階層構造変更API修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/4179)
- 階層構造変更API修正
- 第四⇔第五の切り替え処理の時は国・リージョンの一致を確認しない
- 不要なエラーコード・エラーハンドリングを削除
- テスト修正
## レビューポイント
- 修正箇所の認識は合っているか
- テストケースは足りているか
## 動作確認状況
- ローカルで確認
- 国の違うDealerに付け替える処理を行い成功した
- 行った修正がデグレを発生させていないことを確認できるか
- 具体的にどのような確認をしたか
- 国の違う第四⇔第五間の付け替えのテストが成功することを確認
- ほかのテストが成功することを確認
## 補足
- 相談、参考資料などがあれば
|
2024-05-17 06:49:26 +00:00 |
|
makabe.t
|
0a714f8484
|
Merged PR 870: ファイル情報ポップアップ画面修正
## 概要
[Task4051: ファイル情報ポップアップ画面修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/4051)
- ファイル情報ポップアップを修正し、ファイル名を変更できるように修正しました。
- 音声ファイル名変更APIのファイル名の文字数を修正しました。
- フロントの入力欄で64文字にしたので、プラス拡張子で68文字としました。
## レビューポイント
- 画面イメージは認識通りでしょうか?
- 表示では拡張子を取って、APIに渡す際にはつけているのですが処理として不自然ではないでしょうか?
## UIの変更
- [Task4051](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/Task4051?csf=1&web=1&e=nbYIE0)
## クエリの変更
- なし
## 動作確認状況
- ローカルで確認
- 行った修正がデグレを発生させていないことを確認できるか
- 具体的にどのような確認をしたか
- タスク一覧、ファイル情報ポップアップが正常に見えることを確認
|
2024-04-16 10:12:44 +00:00 |
|
makabe.t
|
e6d6e477d9
|
Merged PR 862: パートナー一覧画面&パートナー編集ポップアップ実装
## 概要
[Task3935: パートナー一覧画面&パートナー編集ポップアップ実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3935)
- パートナー一覧画面からパートナー編集ポップアップを表示して情報を変更できる画面実装をしています。
## レビューポイント
- エラーの表示は適切でしょうか?
- 画面イメージは認識通りでしょうか?
## UIの変更
- [Task3935](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/Task3935?csf=1&web=1&e=FdaUMT)
## クエリの変更
- なし
## 動作確認状況
- ローカルで確認
- 行った修正がデグレを発生させていないことを確認できるか
- 新規機能なので問題なし
|
2024-04-08 07:41:05 +00:00 |
|
masaaki
|
eb6b413adb
|
Merged PR 848: パートナー一覧画面修正
## 概要
[Task3833: パートナー一覧画面修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3833)
- パートナー一覧画面のパートナーにカーソルを合わせた際に、Delete Partnerリンクが表示される。
押下した場合は、本当に削除するのかを確認するメッセージが表示され、どのPartnerを削除するかメッセージ欄に表示されること。
## レビューポイント
- 特筆する点はなし
## UIの変更
https://ndstokyo.sharepoint.com/sites/Piranha/Shared%20Documents/Forms/AllItems.aspx?csf=1&web=1&e=hzPw9b&cid=e8e0702d%2D3730%2D4295%2Dbb9d%2D40e6b1998906&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%2FTask3833&viewid=786a81cf%2Dd15f%2D4dc2%2D9e55%2Dc7a729fbc72f
## クエリの変更
なし
## 動作確認状況
- ローカルで確認(APIからの返却値を直接指定する方式で確認)
- 行った修正がデグレを発生させていないことを確認できるか
新規機能追加のみなので問題なし
## 補足
- 相談、参考資料などがあれば
|
2024-03-22 07:50:58 +00:00 |
|
makabe.t
|
66c643677d
|
Merged PR 829: 音声ファイルバックアップポップアップ修正
## 概要
[Task3882: 音声ファイルバックアップポップアップ修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3882)
- 音声ファイルバックアップ中にファイル削除された場合の処理を追加しました。
- ダウンロードで対象ファイルが削除されていた場合に特別なエラーとなるようにしています。
- タスクバックアップで対象ファイルが削除されていた場合でも成功となるようにしています。
## レビューポイント
- 対応するエラーは適切でしょうか?
## UIの変更
- [Task3882](https://ndstokyo.sharepoint.com/:i:/r/sites/Piranha/Shared%20Documents/General/OMDS/%E3%82%B9%E3%82%AF%E3%83%AA%E3%83%BC%E3%83%B3%E3%82%B7%E3%83%A7%E3%83%83%E3%83%88/Task3882/%E3%83%95%E3%82%A1%E3%82%A4%E3%83%AB%E5%89%8A%E9%99%A4%E6%B8%88%E3%81%BF.png?csf=1&web=1&e=1BKVh8)
## クエリの変更
- なし
## 動作確認状況
- ローカルで確認
- 行った修正がデグレを発生させていないことを確認できるか
- 具体的にどのような確認をしたか
- ダウンロード実行中にファイル削除
- backupAPIからタスク不在エラーを返却して成功するか確認
|
2024-03-15 06:53:41 +00:00 |
|
Kentaro Fukunaga
|
83e297cc9b
|
Merged PR 821: 画面実装(パートナーライセンス一覧画面&階層構造変更ポップアップ)
## 概要
[Task3854: 画面実装(パートナーライセンス一覧画面&階層構造変更ポップアップ)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3854)
- パートナーライセンス一覧に「Change Owner」ボタンを配置し、表示制御およびクリック時にポップアップ表示する処理の実装
- アカウント階層構造変更ポップアップの処理全体実装
- サーバー側のエラーコード定義
## レビューポイント
- 「一括」を表現するためのドロップダウンの構築や処理周りで改善点ないか(to:斎藤くん)
- コンポーネントでの状態管理でお作法に違反しているところないか(to:斎藤くん)
- 修正箇所がほかの機能に影響していないか
- パートナーライセンス一覧の画面表示に何らか悪影響ないか?(to:ガンさん)
## 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/Task3854?csf=1&web=1&e=jBGQrR
## クエリの変更
- なし
## 動作確認状況
- ローカルで確認しました
- 第一階層でログインしてかつ第三または第四視点での一覧を確認しているときにChangeOwnerボタンが表示される
- ボタン押下すると、仕様通りにポップアップの表示が行われる
- ポップアップにて入力項目に入力できる&バリデーション効いている
- ポップアップにて実行ボタン押下するとAPI実行できる&処理結果に応じて仕様通りの挙動をすること
- 行った修正がデグレを発生させていないことを確認できるか
- 具体的にどのような確認をしたか
- パートナーライセンス画面に新規ボタンを配置した&新規ポップアップの実装のみのため、
ポップアップでの処理が正常終了/失敗/何もせず閉じた場合に元の画面の表示が今まで通り動くことを確認済み。
|
2024-03-13 07:41:25 +00:00 |
|
SAITO-PC-3\saito.k
|
311eb98236
|
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
# dictation_server/src/app.module.ts
|
2024-03-12 11:48:25 +09:00 |
|
makabe.t
|
2e6b7c8ab5
|
Merged PR 818: function修正
## 概要
[Task3879: function修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3879)
- 行番号をExcelの表記通りとなるように修正
- 途中から始められるようにforループを修正
## レビューポイント
- 共有
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## クエリの変更
- Repositoryを変更し、クエリが変更された場合は変更内容を確認する
- Before/Afterのクエリ
- クエリ置き場
## 動作確認状況
- ローカルで確認、develop環境で確認など
- 行った修正がデグレを発生させていないことを確認できるか
- 具体的にどのような確認をしたか
- どのケースに対してどのような手段でデグレがないことを担保しているか
## 補足
- 相談、参考資料などがあれば
|
2024-03-08 05:45:53 +00:00 |
|
saito.k
|
aca9bcf496
|
Merged PR 782: 翻訳反映+α
## 概要
[Task3781: 翻訳反映+α](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3781)
- 翻訳情報の反映
- タスクのチェックアウト時(/tasks/{audioFileId}/checkout)にライセンスをチェックするようになったため、それに対応するエラーを追加
- ライセンスが未割当の時とライセンスが有効期限切れの時にそれぞれ専用のエラーメッセージを表示するように修正する
## レビューポイント
- エラーのハンドリングに誤りはないか
## 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/Task3781?csf=1&web=1&e=dCIGih
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2024-03-08 05:11:20 +00:00 |
|
makabe.t
|
146b8a6e40
|
Merged PR 816: function修正
## 概要
[Task3879: function修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3879)
- 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず)
- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
- 影響範囲(他の機能にも影響があるか)
## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
- 修正箇所がほかの機能に影響していないか
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## クエリの変更
- Repositoryを変更し、クエリが変更された場合は変更内容を確認する
- Before/Afterのクエリ
- クエリ置き場
## 動作確認状況
- ローカルで確認、develop環境で確認など
- 行った修正がデグレを発生させていないことを確認できるか
- 具体的にどのような確認をしたか
- どのケースに対してどのような手段でデグレがないことを担保しているか
## 補足
- 相談、参考資料などがあれば
|
2024-03-08 01:09:14 +00:00 |
|
SAITO-PC-3\saito.k
|
9dc80b4965
|
Merge branch 'develop' into ccb
|
2024-03-07 20:55:44 +09:00 |
|
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 |
|
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-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
|
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
|
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 |
|
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 |
|
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の変更
大した変更ではないので、スクショ置き場に置く手間を省いてここに貼り付けます

## 動作確認状況
- 期待される動作を一通りローカルで確認しました。
- 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 |
|
makabe
|
f00861702a
|
Merge branch 'develop' into ccb
|
2024-01-29 15:52:55 +09: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
|
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 |
|
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 |
|
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 |
|
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 |
|
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 |
|
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 |
|
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 |
|
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 |
|
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 |
|