makabe
|
8bee0c2a73
|
Merge branch 'main' into develop
|
2023-10-25 15:47:55 +09:00 |
|
maruyama.t
|
4b812fc0b3
|
Merged PR 520: API IF作成
## 概要
[Task2919: API IF作成](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2919)
- 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず)
- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
- 影響範囲(他の機能にも影響があるか)
新規のためなし
## レビューポイント
メソッド名がふさわしいか
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-10-24 08:35:31 +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 |
|
saito.k
|
48b45d2773
|
Merged PR 518: Entityのtransformer横展開対応
## 概要
[Task2899: 横展開対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2899)
- Bigintがコード上でstringとして扱われるのを考慮し、number型に変換する処理を実装
- テスト実装
- 各Entityに展開
## レビューポイント
- 適用漏れはないか
- 実装内容に疑問はないか
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルでテストが通ることを確認。ローカルで軽く画面を確認
## 補足
- 相談、参考資料などがあれば
|
2023-10-24 07:54:46 +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 |
|
oura.a
|
b2f21cf4f8
|
Merged PR 516: Revert "Merge branch 'develop' into main"
Revert "Merge branch 'develop' into main"
Reverted commit `279fecec`.
|
2023-10-23 06:54:48 +00:00 |
|
oura.a
|
81d966ad1f
|
Merged PR 515: Revert "Merge branch 'develop' into main"
Revert "Merge branch 'develop' into main"
Reverted commit `815082d5`.
|
2023-10-23 06:54:00 +00:00 |
|
oura.a
|
c844837aec
|
Merged PR 514: Revert "Merge branch 'develop' into main"
Revert "Merge branch 'develop' into main"
Reverted commit `463b372c`.
|
2023-10-23 06:53:23 +00:00 |
|
iwata
|
463b372c9b
|
Merge branch 'develop' into main
|
2023-10-20 08:34:09 +09:00 |
|
masaaki
|
7ea9f5b811
|
Merged PR 511: 直接の事象修正(ログイン時のリフレッシュトークン処理修正)
## 概要
[Task2898: 直接の事象修正(ログイン時のリフレッシュトークン処理修正)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2898)
- リフレッシュトークン生成時の管理者判定について、stringとnumberでの比較となっているため厳密ではない等価演算子を使用するよう修正しました。
## レビューポイント
- 特にありません
## UIの変更
- 特にありません
## 動作確認状況
- ユニットテスト+ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-10-19 10:27:35 +00:00 |
|
oura.a
|
815082d584
|
Merge branch 'develop' into main
|
2023-10-19 17:33:33 +09:00 |
|
oura.a
|
ac56629332
|
Merge branch 'develop' of https://dev.azure.com/ODMSCloud/ODMS%20Cloud/_git/ODMS%20Cloud into develop
|
2023-10-19 17:27:28 +09:00 |
|
saito.k
|
83e923d15e
|
mysqlで対応していないtypeを修正
|
2023-10-19 17:17:48 +09:00 |
|
oura.a
|
279fecec88
|
Merge branch 'develop' into main
|
2023-10-19 16:50:27 +09:00 |
|
湯本 開
|
fe3910cbe1
|
Merged PR 509: Pipelineの環境変数追従2
## 概要
[Task2896: Pipelineの環境変数追従2](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2896)
- 環境変数を追加
## レビューポイント
- 情報共有
## 動作確認状況
- なし
|
2023-10-19 07:35:30 +00:00 |
|
oura.a
|
215ccbee0c
|
Merge branch 'develop' of https://dev.azure.com/ODMSCloud/ODMS%20Cloud/_git/ODMS%20Cloud into develop
|
2023-10-19 16:33:11 +09:00 |
|
saito.k
|
f553bfc95b
|
Merged PR 501: strictNullChecks修正①(accounts,auth,Repositoiesのaccounts,common)
## 概要
[Task2835: 修正①(accounts,auth,Repositoiesのaccounts,common)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2835)
- features
- accounts
- auth
- common
- repositories
- accounts
- 各entity
- Nullableの項目の`@Column`デコレータに`type`を追加しないとTypeORMがエラーになりテストが通らないので追加
- https://qiita.com/maruware/items/08c9ad594e14e4ea1497#%E5%95%8F%E9%A1%8C
## レビューポイント
- コメントとして記載
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認、develop環境で確認など
## 補足
- レビュー完了後、TODOコメント(strictNullChecks対応)は削除します
|
2023-10-19 07:13:56 +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 |
|
湯本 開
|
c46d2bad61
|
Merged PR 505: Production/Stagingパイプラインの環境変数を更新する
## 概要
[Task2858: Production/Stagingパイプラインの環境変数を更新する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2858)
- テスト用の環境変数を追加
## レビューポイント
- 情報共有
## 動作確認状況
- 未実施
|
2023-10-18 04:27:49 +00:00 |
|
水本 祐希
|
434e3d9c27
|
Merged PR 502: users.repository.serviceのログ修正
## 概要
[Task2859: users.repository.serviceのログ修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2859)
- 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず)
- 何をどう変更したか、追加したライブラリなど
idTokenとversionのログ出力が正常にできるよう修正
- このPull Requestでの対象/対象外
- 影響範囲(他の機能にも影響があるか)
## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-10-18 02:02:38 +00:00 |
|
水本 祐希
|
364bfb5135
|
Merged PR 504: 利用規約テーブルの初期値を設定
## 概要
[Task2853: 利用規約テーブルの初期値を設定](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2853)
- termsテーブルの初期データを登録するmigrateファイルを追加しました。
## レビューポイント
- 特にありません
## UIの変更
- なし
## 動作確認状況
- migrate upでデータ登録されること、downで削除されることを確認
## 補足
- 相談、参考資料などがあれば
|
2023-10-18 01:29:02 +00:00 |
|
水本 祐希
|
2f5516ec81
|
Merged PR 503: features/accept/constants.tsを削除
## 概要
[Task2863: features/accept/constants.tsを削除](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2863)
- features/acceptとfeatures/termsで同じconstantsの定義があったのでtermsに統一
## レビューポイント
- 特になし
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-10-17 08:53:35 +00:00 |
|
makabe.t
|
a6206a624a
|
Merged PR 500: ActiveWorktypeIDの削除処理を追加
## 概要
[Task2861: ActiveWorktypeIDの削除処理を追加](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2861)
- 削除で指定したワークタイプIDをActiveWorktypeIDに指定していた場合にNULLをセットする処理を追加しました。
## レビューポイント
- ActiveWorktypeID設定内容は適切か
- テストケース追加に不足はないか
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-10-17 08:21:35 +00:00 |
|
水本 祐希
|
7196491cf0
|
Merged PR 472: 画面作成(利用規約同意画面)
## 概要
[Task2802: 画面作成(利用規約同意画面)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2802)
- 何をどう変更したか、追加したライブラリなど
- 利用規約同意画面の実装を行いました
- このPull Requestでの対象/対象外
- api.tsおよびstyles
- 影響範囲(他の機能にも影響があるか)
- ありません
## レビューポイント
- 特にレビューしてほしい箇所
- URLの妥当性(動作確認のため別タスクで追加していますが、内容は本タスクで見てほしいです)
違和感ないか確認お願いします。
<Route path="/accept-to-use" element={<AcceptToUsePage />} />
- 各処理のエラーハンドリングについて
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
https://ndstokyo.sharepoint.com/:f:/r/sites/Piranha/Shared%20Documents/General/OMDS/%E3%82%B9%E3%82%AF%E3%83%AA%E3%83%BC%E3%83%B3%E3%82%B7%E3%83%A7%E3%83%83%E3%83%88/Task2802?csf=1&web=1&e=otF5YX
## 動作確認状況
- ローカルで確認済
## 補足
- 相談、参考資料などがあれば
|
2023-10-17 07:15:49 +00:00 |
|
makabe.t
|
74bf434786
|
Merged PR 496: ワークタイプ削除画面処理実装
## 概要
[Task2612: ワークタイプ削除画面処理実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2612)
- ワークタイプ削除の画面実装です。
## レビューポイント
- エラー処理に問題はないか
## UIの変更
- [Task2612](https://ndstokyo.sharepoint.com/:f:/r/sites/Piranha/Shared%20Documents/General/OMDS/%E3%82%B9%E3%82%AF%E3%83%AA%E3%83%BC%E3%83%B3%E3%82%B7%E3%83%A7%E3%83%83%E3%83%88/Task2612?csf=1&web=1&e=IgC7j3)
## 動作確認状況
- ローカルで確認
|
2023-10-17 06:29:25 +00:00 |
|
oura.a
|
58674f7773
|
Merged PR 498: アカウント登録画面修正
## 概要
[Task2809: アカウント登録画面修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2809)
アカウント作成時の管理者ユーザー作成時に、同意済みEULAバージョンを登録する処理を追加しました。
## レビューポイント
なし
## UIの変更
なし
## 動作確認状況
ローカルで動作確認済み
## 補足
なし
|
2023-10-17 06:06:12 +00:00 |
|
saito.k
|
d2c2223acf
|
Merged PR 497: strictNullChecks修正③(task,users ,Repositoriesのusers)
## 概要
[Task2837: 修正③(task,users ,Repositoriesのusers)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2837)
- feature
- tasks
- users
- Repositories
- users
## レビューポイント
- 該当箇所にコメントで記載
## 動作確認状況
- ローカルでテストが通ることを確認
## 補足
- 相談、参考資料などがあれば
|
2023-10-16 10:25:45 +00:00 |
|
makabe.t
|
c9bc6393c6
|
Merged PR 490: ワークタイプ削除API実装
## 概要
[Task2611: ワークタイプ削除API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2611)
- ワークタイプの削除APIとテストを実装しました。
※API IF部分は対象外です。
## レビューポイント
- リポジトリのエラーチェックに問題はないか
- テストケースは適切か
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-10-16 09:39:01 +00:00 |
|
maruyama.t
|
00f4966aa9
|
Merged PR 470: [Sp-19]アカウント削除時に削除するテーブルについて、on delete cascadeではなくコード上で削除を行うよう修正する
## 概要
[Task2783: [Sp-19]アカウント削除時に削除するテーブルについて、on delete cascadeではなくコード上で削除を行うよう修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2783)
アカウント削除時に削除するテーブルについて、on delete cascadeではなくコード上で削除を行うよう修正
一部ユニットテスト用にutilityのcreateForeignKeyConstraints: falseに指定
LGTM後、すべてのテーブルのレコード削除の動作確認をDEV環境で実施します。
## レビューポイント
DBマイグレーションファイルが正しく修正されているか
## UIの変更
なし
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-10-16 09:02:38 +00:00 |
|
masaaki
|
897bad289b
|
Merged PR 480: 画面修正(ログイン画面)
## 概要
[Task2801: 画面修正(ログイン画面)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2801)
- 以下の修正を実施しました
- ログイン画面について、未同意バージョンがある場合、利用規約同意画面に遷移する処理を実装
- 利用規約同意画面(ADB2C以外の画面)からログイン画面に遷移した際も処理継続できるよう対応を実施
- このPull Requestでの対象/対象外
- AcceptToUsePageについては、遷移確認用のダミーページなので対象外でお願いします。
- 影響範囲(他の機能にも影響があるか)
- ありません。
## レビューポイント
- 特にレビューしてほしい箇所
1. 既存のLoginPageを以下のように分割しています。
実装内容のイメージあっているか確認お願いします。
- LoginPage→AADB2Cからのリダイレクトを元にLocalStorageアクセス用のキーを生成
- TokenSettingPage→LocalStorageアクセス用のキーを使用してidTokenを取得し各種token生成を実施
1. TokenSettingPage/index.tsxにて、型ガード(isErrorObject)を作成し使用しています。
使い方やガードの実装が妥当か確認お願いします。
## UIの変更
- 無し
## 動作確認状況
- ローカルで確認を実施
## 補足
- 相談、参考資料などがあれば
|
2023-10-16 06:52:08 +00:00 |
|
makabe.t
|
45350d0ab8
|
Merged PR 485: API IF実装
## 概要
[Task2610: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2610)
- WorkTypeID削除API IFを実装し、OpenAPI定義を更新しました。
## レビューポイント
- パスは適切か
- レスポンスは想定通りか
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-10-16 02:14:22 +00:00 |
|
oura.a
|
cdb6931d57
|
Merge branch 'develop' of https://dev.azure.com/ODMSCloud/ODMS%20Cloud/_git/ODMS%20Cloud into develop
|
2023-10-16 11:00:09 +09:00 |
|
saito.k
|
d258d569f7
|
Merged PR 483: strictNullCheck修正④(gateways ,notification)
## 概要
[Task2838: 修正④(gateways ,notification)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2838)
- strictNullCheckの対応
- gateways配下
- feartures
- notification
## レビューポイント
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認、develop環境で確認など
## 補足
- 相談、参考資料などがあれば
|
2023-10-16 01:34:28 +00:00 |
|
oura.a
|
273ba588ce
|
Merged PR 495: API作成(バージョン更新API)
## 概要
[Task2804: API作成(バージョン更新API)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2804)
同意済み利用規約バージョン更新APIを実装しました。
## レビューポイント
なし
## UIの変更
なし
## 動作確認状況
UT,ローカルで動作確認済み
## 補足
なし
|
2023-10-16 01:31:30 +00:00 |
|
oura.a
|
162470838d
|
Merged PR 489: API作成(利用規約情報取得API)
## 概要
[Task2808: API作成(利用規約情報取得API)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2808)
利用規約情報取得APIを作成しました。
## レビューポイント
なし
## UIの変更
なし
## 動作確認状況
UT、ローカルで動作確認済み
## 補足
なし
|
2023-10-13 06:36:57 +00:00 |
|
makabe.t
|
69ff6f3432
|
Merged PR 493: API作成(アカウント情報取得(未認証時最小アクセス)API)
## 概要
[Task2807: API作成(アカウント情報取得(未認証時最小アクセス)API)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2807)
- 未ログインユーザーについて、IDトークンを受け取ってユーザの所属するアカウントの階層情報を返却するAPIを実装しました。
## レビューポイント
- ContorollerでIDトークンをデコードしているが問題ないか?
- ※ログインAPIを参考にしています。
- テストケースは適切か
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-10-13 05:33:02 +00:00 |
|
makabe.t
|
685a8f6c3e
|
Merged PR 487: 削除済みエラーを成功扱いとするように画面修正
## 概要
[Task2841: 削除済みエラーを成功扱いとするように画面修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2841)
- ワークフロー画面からワークフローを削除した際に、対象がすでに削除済みの場合も成功扱いになるように修正しました。
## レビューポイント
- 対応内容は適切か
## UIの変更
- [Task2841](https://ndstokyo.sharepoint.com/:f:/r/sites/Piranha/Shared%20Documents/General/OMDS/%E3%82%B9%E3%82%AF%E3%83%AA%E3%83%BC%E3%83%B3%E3%82%B7%E3%83%A7%E3%83%83%E3%83%88/Task2841?csf=1&web=1&e=56pcKY)
## 動作確認状況
- ローカルで確認
|
2023-10-13 05:23:37 +00:00 |
|
oura.a
|
cafacb761d
|
Merged PR 494: App.ts修正
## 概要
[Task2845: App.ts修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2845)
App.tsxのエラーコード判定部分を修正しました。
## レビューポイント
なし
## UIの変更
なし
## 動作確認状況
ローカルでビルドエラーが出ないことを確認。
ローカルで無理やりエラーを投げ、以下を確認。
E010209を入れて401エラーを投げる
→ログアウトされない
E010208を入れて401エラーを投げる
→ログアウトされる
## 補足
なし
|
2023-10-13 05:23:21 +00:00 |
|
saito.k
|
370d143c2c
|
Merged PR 473: strictNullCheckの対応を部分的に行う
## 概要
[Task2795: 部分的に修正を行う](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2795)
- strictNullChecks対応
- features
- template
- workflow
- gateways
- adb2c
- repositories
- template
- workflow
## レビューポイント
- entityの修正内容
- nullを追加する項目はあってるか
- adb2cの環境変数を取得している箇所
- getOrThrowで値が取得できなければエラーになる関数があったので使用しています。

## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルでテストが通ることを確認
## 補足
- 相談、参考資料などがあれば
|
2023-10-13 04:07:18 +00:00 |
|
oura.a
|
55b854af36
|
Merged PR 492: ビルドエラー修正
## 概要
[Task2844: ビルドエラー修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2844)
- 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず)
- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
- 影響範囲(他の機能にも影響があるか)
## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認、develop環境で確認など
## 補足
- 相談、参考資料などがあれば
|
2023-10-12 09:17:35 +00:00 |
|
oura.a
|
18adadce28
|
Revert "リポジトリサービス作成"
This reverts commit 7793ecdb1e65d4977deaba1c4e99264f05c151da.
|
2023-10-12 18:09:58 +09:00 |
|
oura.a
|
7793ecdb1e
|
リポジトリサービス作成
|
2023-10-12 18:08:55 +09:00 |
|
oura.a
|
274292c389
|
Merged PR 477: API修正(トークン生成API)
## 概要
[Task2800: API修正(トークン生成API)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2800)
既存のトークン生成APIに、利用規約バージョンのチェック処理を追加しました。
また、チェック処理で同意済みバージョンが最新でないときのエラー時にログアウトしないような処理を追加しました。
## レビューポイント
なし
## UIの変更
なし
## 動作確認状況
UT,ローカルでの動作確認済み
## 補足
なし
|
2023-10-12 08:59:36 +00:00 |
|
湯本 開
|
ba802dec40
|
Merged PR 491: Stagingパイプラインを作成する(ビルドのみ)
## 概要
[Task2842: Stagingパイプラインを作成する(ビルドのみ)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2842)
- Stagingパイプラインで稼働させる用のpipeline.ymlを実装
- STGビルドのみを行い、ビルド結果を前提とする本番環境デプロイが通常ルートで出来るようにすることが目的
- 現在のSTG環境には一切触らない想定
## レビューポイント
- 今稼働している環境へ影響するような処理がコメントアウトされて動かないようになっているか?
- STG環境のWebApps/StaticWebApps/DBに影響ある操作をしていなさそうか
## 動作確認状況
- 未確認
|
2023-10-12 08:23:13 +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 |
|
湯本 開
|
49bd0e5ffe
|
Merged PR 476: DeplymentSlotを使ってデプロイする形式でPipelineを稼働するよう構成する
## 概要
[Task2816: DeplymentSlotを使ってデプロイする形式でPipelineを稼働するよう構成する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2816)
- 本番環境用のデプロイパイプラインを更新
- バックエンド側起動時にビルドバージョンをログに出す機能を実装
## レビューポイント
- 情報共有
## 動作確認状況
- 本番環境で確認
|
2023-10-12 06:30:43 +00:00 |
|
oura.a
|
a5b07596c1
|
Merged PR 486: IF不具合修正
## 概要
[Task2840: api.ts最新化](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2840)
IFの修正を行いました。
・/api/termsがPOSTになっていたのでGETに修正
・/api/termsのtypesの記載が足りていなかったので修正
## レビューポイント
なし
## UIの変更
なし
## 動作確認状況
swaggerUIで確認
## 補足
なし
|
2023-10-12 05:37:25 +00:00 |
|
Kentaro Fukunaga
|
be475c29a6
|
Merged PR 481: 音声ファイルアップロードできる権限制御を外す
## 概要
[Task2764: 音声ファイルアップロードできる権限制御を外す](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2764)
- Authorでなくとも音声ファイルのアップロードやタスク追加をするケースがあるとのことだったので対応しました。
- アップロード要求APIと音声ファイルアップロード完了APIのRoleGuardを外しました
## レビューポイント
- ほか気になるケースあれば(この挙動はできるようにしなくて大丈夫?とか)
## 動作確認状況
- ローカルでツールを使ってアップロードでき、タスク追加できるところまで確認
|
2023-10-11 23:28:41 +00:00 |
|
makabe.t
|
d48afdbffd
|
Merged PR 468: API実装(ワークフロー削除API)
## 概要
[Task2785: API実装(ワークフロー削除API)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2785)
- ワークフロー削除APIとテストを実装しました。
## レビューポイント
- リポジトリの削除ロジックは適切か
- テストケースは適切か
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-10-11 09:14:41 +00:00 |
|