湯本 開
|
ee72a280c8
|
Merged PR 172: WORK_TYPEでのソートが失敗する問題を修正する
## 概要
[Task2016: WORK_TYPEでのソートが失敗する問題を修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2016)
- WorkTypeによるソート条件構築の実装が漏れていた箇所を修正
## レビューポイント
- 修正内容に問題はないか
## 動作確認状況
- テスト通過、ビルド成功
|
2023-06-20 09:26:02 +00:00 |
|
saito.k
|
756f6155d5
|
Merged PR 171: B2Cserviceのエラーの解消
<内容>
- 不要な処理を削除(削除漏れだった箇所)
|
2023-06-20 08:06:04 +00:00 |
|
makabe.t
|
ef17e4ff0b
|
Merged PR 167: タイピストユーザー取得API実装
## 概要
[Task1931: タイピストユーザー取得API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1931)
- アカウント内のタイピストユーザーを取得する処理を実装しました。
## レビューポイント
- ADB2Cからのユーザー取得、マージは適切か
## UIの変更
なし
## 動作確認状況
- ローカルで確認
|
2023-06-20 06:16:19 +00:00 |
|
oura.a
|
8c75f7a1b1
|
Merged PR 168: static appについて環境ごとに設定値を切り替える
## 概要
[Task2004: static appについて環境ごとに設定値を切り替える](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2004)
ステージング環境用のB2C情報を設定したenvファイルを作成。
作成したenvファイルはpipelineの
「Dictation Static App Service Deploy」→「Bash Script」→「Script」
を以下のように修正することでの反映を考えています。
■修正前
cd dictation_client && npm run build
■修正後
cd dictation_client && cp -f .env.staging .env && npm run build
※もしも本修正がdevelop環境にも影響する場合は、develop側も同様にenvファイルを作成しpipelineにコピー処理を追加する。
## レビューポイント
設定内容が妥当か。
上記のpipelineへの追加処理が妥当か。
## UIの変更
なし
## 動作確認状況
未
## 補足
なし
|
2023-06-20 05:18:30 +00:00 |
|
maruyama.t
|
22bd761d10
|
Merged PR 161: DBマイグレーション
## 概要
[Task1990: DBマイグレーション](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1990)
- 何をどう変更したか、追加したライブラリなど
以下を追加
カードライセンステーブル
カードライセンス発行テーブル
- 影響範囲(他の機能にも影響があるか)
新規追加テーブルのため、なし
## レビューポイント
- 各カラムの物理名が妥当であるか
- NOT NULL制約は正しいか
## UIの変更
テーブルの新規追加のため、なし
## 動作確認状況
- ローカルで確認
server側のターミナルで「sql-migrate up -config=db/dbconfig.yml -env=local」を実行
→DBへ反映されていることを確認。
「sql-migrate down -config=db/dbconfig.yml -env=local」を実行
→1世代巻き戻ることを確認。
## 補足
- インプットと資料としてラフスケッチをご参照ください
|
2023-06-19 04:10:32 +00:00 |
|
maruyama.t
|
f0e0d5e6b1
|
Merged PR 162: API IF修正(LicenseSummaryInfo)
## 概要
[Task1961: API IF修正(LicenseSummaryInfo)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1961)
- 何をどう変更したか、追加したライブラリなど
API実装時に一時的にLicenseSummaryInfo2と命名していた定義を、LicenseSummaryInfoに修正。
それに伴い、既存のLicenseSummaryInfoはレスポンスに直接記載するよう修正。
openapi.jsonを再生成し、画面側のパラメータの取得の記載を修正。
- このPull Requestでの対象/対象外
storageSizeとusedSizeの取得はPBI対象外
- 影響範囲(他の機能にも影響があるか)
なし
## レビューポイント
- クライアント側に対する修正漏れがないか。
## UIの変更
なし
## 動作確認状況
- ローカルでAPIの戻り値の確認と、画面にその値が表示されていることを確認。


## 補足
- 相談、参考資料などがあれば
|
2023-06-19 04:06:44 +00:00 |
|
saito.k
|
d75c003b09
|
Merged PR 145: typist名をAzure AD B2Cから取得し表示できるようにする
## 概要
[Task1950: typist名をAzure AD B2Cから取得し表示できるようにする](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1950)
- チェックアウト権限にあるTypistのユーザー名をB2Cから取得する
- チェックアウト権限に含まれているuser.externalIdを列挙
- ExternalIdでフィルターをかけてユーザー情報を取得
- B2Cへのリクエスト上限超過時のエラーを制御するために専用エラーを定義
- import文が常に絶対パスで指定されていて、それでテストがこけるので相対パスでインポートするようにvscodeを設定
## レビューポイント
- convert.tsの修正は問題ないか
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認
## 補足
- 取得方法についてはMSに問い合わせ中
|
2023-06-19 00:48:32 +00:00 |
|
湯本 開
|
bb926f9feb
|
Merged PR 164: ユニットテスト実装(タスク一覧取得 | typist)
## 概要
[Task1965: ユニットテスト実装(タスク一覧取得 | typist)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1965)
- Typistによるタスク一覧取得のユニットテスト部分を実装
## レビューポイント
- 現状、ユニットテストとして問題ないか
- レビュー対象外の部分は対象外で問題ないか
## レビュー対象外
- ライセンス周辺のテストが失敗している
## 動作確認状況
- ローカルで確認
|
2023-06-16 12:08:05 +00:00 |
|
maruyama.t
|
fd69541e1a
|
Merged PR 142: API実装(第五階層用ライセンス情報取得API)
## 概要
[Task1846: API実装(第五階層用ライセンス情報取得API)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1846)
- 元PBI or タスクへのリンク(内容・目的などはそちらにあるはず)
- 何をどう変更したか、追加したライブラリなど
accounts.controller.tsからaccountService.getLicenseSummaryを呼び出す。
一度のトランザクションで処理を行うよう、serviceとrepositoryをリファクタリング。
license.entity.tsにはライセンス系のテーブルで不足していたエンティティを追加。
- このPull Requestでの対象/対象外
Storage Sizeの値はPBI1203では対象外のため0固定
Used Sizeの値はPBI1203では対象外のため0固定
LicenseSummaryInfo2と定義している個所は、別タスクで修正します。
[タスク 1961: API IF修正(LicenseSummaryInfo)](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%2011-1?workitem=1961)
- 影響範囲(他の機能にも影響があるか)
新規追加のため、なし
## レビューポイント
- 特にレビューしてほしい箇所
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
PostmanでAPI実行。
各ライセンス数値が期待通りの結果であることを確認。

## 補足
テスト内容は、添付のテストデータを参照ください。
[テストデータ.xlsx](https://dev.azure.com/ODMSCloud/6023ff7b-d41c-4fa7-9c6f-f576ba48c07c/_apis/git/repositories/302da463-a2d7-40f9-b2bb-6e8edf324fa9/pullRequests/142/attachments/%E3%83%86%E3%82%B9%E3%83%88%E3%83%87%E3%83%BC%E3%82%BF.xlsx)
|
2023-06-15 08:56:03 +00:00 |
|
湯本 開
|
2cdcae4924
|
Merged PR 152: API実装(タスク一覧取得 | typist)
## 概要
[Task1865: API実装(タスク一覧取得 | typist)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1865)
- Typist用のRepository呼び出しを追加
## レビュー対象外
- テスト実装(別Taskで対応予定)
## レビューポイント
- SQLの呼び出し回数が増え気味だが問題なさそうか
- Task一覧の取得条件は間違っていなさそうであるか
## 動作確認状況
- ローカルで確認(ユーザーグループ等は各1件程度のデータで確認)
|
2023-06-15 06:42:57 +00:00 |
|
Kentaro Fukunaga
|
4065268820
|
Merged PR 159: Task一覧取得の、文字起こし担当者の型を修正する
## 概要
[Task1987: Task一覧取得の、文字起こし担当者の型を修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1987)
- 型の修正および、既存コードでAssigeeを使うべきでないところをTypistを使うように修正しました
## レビューポイント
- 対応抜け漏れ無いか
## UIの変更
- なし
## 動作確認状況
- ローカルでビルド通ることを確認
- ローカルで既存テストに影響ないことを確認
|
2023-06-14 09:50:58 +00:00 |
|
makabe.t
|
418d568af2
|
Merged PR 153: Task一覧APIのResponseで省略可能でないといけないプロパティが必須になっている箇所を修正する
## 概要
[Task1956: Task一覧APIのResponseで省略可能でないといけないプロパティが必須になっている箇所を修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1956)
[Task1918: タスク一覧、タスク追加APIのコメント修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_workitems/edit/1918)
- Transcription開始日時と終了日時がNULLでも問題ないようにプロパティの型を修正しました。
- タスク一覧の音声ファイル録音時間のコメントも併せて修正しています。
## レビューポイント
- 修正内容は適切か
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-06-14 08:42:48 +00:00 |
|
Kentaro Fukunaga
|
42bc458632
|
Merged PR 155: API IF実装
## 概要
[Task1930: API IF実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1930)
- Typist一覧取得API、TypistGroup一覧取得API、チェックアウト候補変更APIの3本のAPIIFを実装しました。
- 既存のTask系APIで使用していた Typist クラスの名前をAssigneeに変更しました
- npm run formatでフォーマットした内容も含まれています。
## レビューポイント
- IFはラフスケッチで合意した内容に沿っているか
- 既存のTask系APIのTypistクラスの名前変更に関して、修正抜け漏れはないか
- チェックアウト候補変更APIのリクエスト/レスポンスのクラス名は適切か
## UIの変更
- なし
## 動作確認状況
- ローカルでPostmanにてAPIIFの値が返ることを確認しました
## 補足
- 特になし
|
2023-06-14 07:55:22 +00:00 |
|
makabe.t
|
bc442e1a2e
|
Merged PR 143: 認証エラーや自動ログアウト時にスナックバーを表示する実装を行う
## 概要
[Task1917: 認証エラーや自動ログアウト時にスナックバーを表示する実装を行う](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1917)
- 認証エラーなどでトップページに遷移する際にクエリパラメータを持たせて、トップページでスナックバーを出すようにしました。
## レビューポイント
- 認証エラーでの遷移かの判定をクエリパラメータで行っているが問題ないか
- 表示メッセージは適切か
## UIの変更
- [Task1917](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/Task1917?csf=1&web=1&e=q664WR)
## 動作確認状況
- ローカルで確認
|
2023-06-14 03:57:40 +00:00 |
|
saito.k
|
34b242684b
|
Merged PR 154: API実装(タスク一覧取得 | author)
## 概要
[Task1949: API実装(タスク一覧取得 | author)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1949)
- タスク一覧取得(Author用)
- AuthorIDとAccountIDを条件にタスクを取得する処理を実装
- テスト実装
- 成功ケースの時に、Repositoryのメソッドが正しい引数で呼ばれているか確認するテストを追加
## レビューポイント
- テスト実装の内容はこれでよいか
- DBから取得した値をレスポンス用の型に変換する処理はAdminと同様の認識だがあっているか
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認
- 実際にデータを取得して内容が正しいか確認
## 補足
- 相談、参考資料などがあれば
|
2023-06-14 03:57:30 +00:00 |
|
masaaki
|
3220ae0552
|
Merged PR 157: accounts.repository.service.tsの参照があやまっている
## 概要
[Task1960: accounts.repository.service.tsの参照があやまっている](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1960)
- constantsの参照についてsrcフォルダを参照しないように修正しました
## レビューポイント
- 特になし
## UIの変更
- 特になし
## 動作確認状況
- ビルドが成功することを確認
## 補足
- 相談、参考資料などがあれば
|
2023-06-13 09:45:30 +00:00 |
|
oura.a
|
201b6be260
|
Merged PR 150: API実装(自アカウント情報取得API)
## 概要
[Task1954: API実装(自アカウント情報取得API)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1954)
・アクセストークンからユーザ情報を取得する処理を追加。
(本タスクで返しているのはアカウントIDのみ)
・本タスクのレビュー対象は以下となります。
・accounts.servicce
getMyAccountメソッド
・accounts.controller
@Get('me')
・accounts.service.spec
'アクセストークンからユーザ情報を取得する'
'ユーザ情報が取得できない場合エラーとなる'
・別タスクで作成したテストコードを取り込むためマージしたところ、controllerの別タスク箇所でエラーが出ていたので、一時的にコメントアウトしています。本タスクの挙動には影響ありません。
## レビューポイント
処理に過不足がないか。
エラーハンドリングが適切か。
## UIの変更
なし
## 動作確認状況
ローカルでユニットテスト実施済み。
ローカルで外部からAPIを実行し、動作確認済み。
## 補足
なし
|
2023-06-13 07:39:10 +00:00 |
|
masaaki
|
1203d6bb99
|
Merged PR 148: 画面実装(第5階層用ライセンス情報)
## 概要
[Task1842: 画面実装(第5階層用ライセンス情報)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1842)
- 第5階層用ライセンス情報画面の本実装を行いました
- ボタン押下確認のため、遷移後の画面(注文履歴画面、カードライセンス取り込みポップアップ)も仮実装しています
- このPull Requestでの対象外
・api.ts、app.module.scss → openapi.json及びデザイナさんcssを取り込んだもののため
・cardLicenseImportPopup.tsx、LicenseOrderHistoryPage → 仮実装画面のため
・licenseOrderフォルダ構成を変更しただけのため
## レビューポイント
- License周りの構成について見直しを行っています。
これまではfeatures/licenseの直下にstateやoperationを配置していましたが、
LicenseページとURLを同じくする画面が増えるため、サブフォルダを作っています。
features/license
/licenseOrder // これまで直下にあったもの
/licenseSummary // 今回作成したもの
構成について問題ないか確認をお願いします。
## 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/Task1842?csf=1&web=1&e=OdR2mR
## 動作確認状況
- ローカルで確認
## 補足
- 特にありません
|
2023-06-13 07:07:06 +00:00 |
|
makabe.t
|
85d8221186
|
Merged PR 149: テスト実装(タスク一覧取得|admin)
## 概要
[Task1955: テスト実装(タスク一覧取得|admin)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1955)
- タスク一覧のテストを実装しました。
- adminの時のテストだけです
- テストとテストに必要なmockを実装しています。
## レビューポイント
- テスト項目として不足はないか
- mockの構成に問題はないか
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
- テスト実行
|
2023-06-13 04:23:21 +00:00 |
|
湯本 開
|
f5ce65add8
|
Merged PR 147: Task一覧APIのstatusの入力チェックを行うデコレータを実装する
## 概要
[Task1957: Task一覧APIのstatusの入力チェックを行うデコレータを実装する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1957)
- Statusに該当する文字列が `,` で連結されている場合のみバリデーション通過となるようなデコレータを実装
## レビューポイント
- 実装コストを考慮して汎用的な指定をできるようにしなかったが問題ないか
- 実装コードは妥当な実装になっているか
## 動作確認状況
- ローカルで確認
|
2023-06-12 05:26:26 +00:00 |
|
湯本 開
|
26098cc400
|
Merged PR 130: API実装(タスク一覧取得 | admin)
## 概要
[Task1831: API実装(タスク一覧取得 | admin)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1831)
- AdminがAPIを呼び出したときの処理である、アカウント内のTask情報すべてを取得するロジックを実装
## レビュー対象外
- Transcription開始日時と終了日時が必須プロパティになっている
- [Task一覧APIのResponseで省略可能でないといけないプロパティが必須になっている箇所を修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_workitems/edit/1956)で修正予定のため
- ユニットテストが未実装
- [テスト実装(タスク一覧取得 | admin)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_workitems/edit/1955)で実施予定のため
- API I/Fのstatusのバリデーションがされていない
- [Task一覧APIのstatusの入力チェックを行うデコレータを実装する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_workitems/edit/1957)で実施予定のため
## レビューポイント
1. SQLの発行方法は問題なさそうか
- ~.entityに依存関係を記述( `@OneToOne(...)` や `@OneToMany(...)` )し、TypeORMでの取得時の挙動に任せる方法でよさそうか
2. Permissionテーブル以下も含めて一括でTypeORMによるクエリビルダーに任せたかったが、他の上手くいっている構造と同じ指定をしてもSQL発行時に指定したカラム名を取ってこなくなるという問題が解決できなかったため、2回に分けて取ってくるようにしたが許容可能そうか?
3. 各テーブルでRepositoryを作ってEntityを定義し、他RepositoryからはそのRepository配下のディレクトリを参照するという形を取ってみたが、方針として問題ないか
- 各テーブルを個別に取得したい場合があるかも?という予想があったため
4. Serviceのつくりとして問題はなさそうか(roleによる呼び分けの実装方法など)
5. RepositoryDTO→ControllerDTOの型変換が複雑であったため、専用のconvert.tsというファイルに分離したが、方針として問題なさそうか
## 動作確認状況
- ローカルで確認
|
2023-06-12 03:50:48 +00:00 |
|
makabe.t
|
1082a48fe9
|
Merged PR 139: 認証・認可を宣言的に扱える仕組みを既存処理に適用する
## 概要
[Task1830: 認証・認可を宣言的に扱える仕組みを既存処理に適用する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1830)
- 既存処理について、認証認可をガードで実施するよう以下のAPIに適用しました。
- ユーザー一覧
- GET /users
- ユーザー追加
- POST /users/signup/
- 音声ファイルアップロード先取得
- GET /files/audio/upload-location/
- ライセンス注文
- POST /licenses/orders
ロールガードのテストで定数を使うように修正
## レビューポイント
- 対応APIの抜け漏れはないか
- 対応内容に問題はないか
- 付与権限
- トークン内容取得
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-06-09 08:39:44 +00:00 |
|
masaaki
|
15ee0c2e98
|
Merged PR 144: 画面実装(ライセンス情報画面(親))
## 概要
[Task1841: 画面実装(ライセンス情報画面(親))](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1841)
- タスク 1841: 画面実装(ライセンス情報画面(親))
- 階層情報を取得し、第五の場合とそれ以外の場合でページを振り分ける処理を実装しました
- 遷移先のページについてはレビュー対象外になります。
・第五階層のページ「licenseSummary.tsx」→タスク 1842: 画面実装(第5階層用ライセンス情報)で本実装
・第1-4階層のページ「partnerLicense.tsx」→本PBI対象外
## レビューポイント
- index.tsxについて、アクセストークンが不正だった場合の処理として、NotFoundPageに遷移させています。
通常アクセストークンが不正なことはないと考えているため、そのためのエラーページを作るほどでもないと
考えてこうしていますが、問題あれば指摘ください。
## UIの変更
- 無し
## 動作確認状況
- ローカルで確認
## 補足
- 無し
|
2023-06-09 07:35:35 +00:00 |
|
masaaki
|
f126b0976e
|
Merged PR 146: API IF実装(accounts/me復活)
## 概要
[Task1953: API IF実装(accounts/me復活)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1953)
- accounts/meのIFを追加しました
※別タスク「タスク 1837: API I/F実装」でレビューまでしてもらったものの、階層をアクセストークンに含める方針としたため削除としたものを復活させています。
- prettierがエラーを出力するようになっていたので、以下URLを参考にsettings.jsonを更新しています
https://qiita.com/TellMin/items/a634149730b777e2e6d0
- このPull Requestでの対象外
・openapi.jsonのL952とL2094については本PBI対象外になります
## レビューポイント
- 特にありません
## UIの変更
- 無し
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-06-09 07:34:49 +00:00 |
|
saito.k
|
0907bd28af
|
Merged PR 138: SendGridService内のprivateキーの取得方法を修正する
## 概要
[Task1736: SendGridService内のprivateキーの取得方法を修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1736)
- CryptoServiceの削除
- トークン発行・認証に使用するKeyを環境変数から取得するように修正
- 既存テストの修正
## レビューポイント
- Keyの取得関数の配置場所は妥当か
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルでテストが通ることを確認
- keyを使う処理(ユーザー追加)を実行して、成功することを確認
## 補足
- 修正した箇所のロールチェックはレビュー対象外
- 「タスク 1830: 認証・認可を宣言的に扱える仕組みを既存処理に適用する」で調整してもらう想定
|
2023-06-09 05:35:39 +00:00 |
|
maruyama.t
|
25072f558f
|
Merged PR 132: 画面実装(ページヘッダ修正)
## 概要
[Task1840: 画面実装(ページヘッダ修正)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1840)
- JWTからアクセストークン内のロールを確認する
- 管理者権限(admin)を持たない場合はライセンスタブを表示しない
## レビューポイント
- ライセンスタブを表示しない処理を.filterで行ってるが、問題ないか
## UIの変更
- ライセンスタブの表示非表示
## 動作確認状況
- ローカルで確認、adminユーザで表示されること

standardユーザで表示されないことを確認

## 補足
- `// userRole: "user" | "partner"; ログインユーザーのロールに応じてタブの活性非活性に使用する想定`
の記述は今回使わなかったので削除しました。
|
2023-06-08 09:22:06 +00:00 |
|
maruyama.t
|
c4aaee07b1
|
Merged PR 137: アクセストークン内に階層情報を含める
## 概要
[Task1925: アクセストークン内に階層情報を含める](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1925)
- アクセストークンとリフレッシュトークンにtierを追加
## レビューポイント
- tierのチェックは必要か
- DBの値をそのまま入れているが問題ないか
**以下は別タスクとして切り出す**
- RoleGuardsを拡張してtierもチェックできるようにする処理を追加し、階層ごとに許可される操作をI/Fの属性として宣言的にチェックできるように修正した箇所について、使いやすそうか。
例)
@UseGuards(RoleGuard.requireds({ roles: ['admin', 'author'] }))
の場合(階層の宣言はしていない場合)許可@UseGuards(RoleGuard.requireds({ roles: ['admin', 'author'], tier [2] }))の場合(階層の宣言をしている場合)ユーザのアカウントの階層を見て、2以上なら許可、2未満なら拒否
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
## 補足
- https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_wiki/wikis/OMDSDictation_wiki/202/%E3%82%A2%E3%82%AF%E3%82%BB%E3%82%B9%E3%83%88%E3%83%BC%E3%82%AF%E3%83%B3
wikiにアクセストークンとリフレッシュトークンについてのページを記載しました。
|
2023-06-08 08:28:10 +00:00 |
|
makabe.t
|
3dca5c604b
|
Merged PR 126: 画面実装(タスク一覧画面)
## 概要
[Task1834: 画面実装(タスク一覧画面)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1834)
- タスク一覧画面を実装しました。
- 表示項目、フィルターを制御できるようにしています
- 表示項目はlocalStorageに保存するようにしています
- ソート順の取得、変更、更新をする処理を追加
## レビューポイント
- デザインの反映は適切か
- フィルター、表示の反映ロジックは適切か
- ソートの反映を追加したので適切か確認をお願いします。
## UIの変更
- [Task1834](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/Task1834?csf=1&web=1&e=zBcQ6i)
## 動作確認状況
- ローカルで確認
- API呼び出しは未検証
|
2023-06-08 08:04:34 +00:00 |
|
saito.k
|
942ac30d8f
|
Merged PR 136: API実装(ソート条件取得API)
## 概要
[Task1923: API実装(ソート条件取得API)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1923)
- ソート条件取得APIを実装
- テスト追加
## レビューポイント
- ソート条件取得APIのレスポンスのデータは足りているか
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-06-07 09:21:20 +00:00 |
|
saito.k
|
9441049201
|
Merged PR 134: API修正(IF)
## 概要
[Task1922: API修正(IF)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1922)
- タスク取得APIの引数を修正
- directionとparamNameを追加
- 定数を文字列配列からオブジェクトに変更
## レビューポイント
- 引数の型を自作のものに修正したが問題ないか。
- Pull Request 128: API実装(ソート条件変更)で議論中
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-06-07 07:45:30 +00:00 |
|
saito.k
|
d5e5e59f8c
|
Merged PR 128: API実装(ソート条件変更)
## 概要
[Task1835: API実装(ソート条件変更)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1835)
- ソート条件変更APIを実装
- トークンからB2CのIDを取得→userテーブルからユーザー情報を取得
- ユーザーIDでソート条件テーブルを検索→レコードを更新
- ソート条件テーブルにレコードを作成する
- アカウント作成時の処理にソート条件レコードを作成する処理を追加
- ユーザー追加時にも処理を追加
- テスト修正
## レビューポイント
- APIの引数をチェックする関数をControllerに配置してもよいか
- ソート条件のレコードを作成するタイミングに漏れはないか
- 実装漏れはないか
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-06-06 08:20:21 +00:00 |
|
masaaki
|
cfca98e53b
|
Merged PR 133: API I/F実装
## 概要
[Task1837: API I/F実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1837)
- ラフスケッチに基づき、「アカウント情報取得API」「ライセンス集計取得API(ラフスケッチ時は「第五階層用ライセンス情報取得API」)」のI/F実装とopenapi化を行いました。
## レビューポイント
- 管理者のみのためUseGuardsを使用しています。実装内容について問題ないか特に確認いただきたいです
## UIの変更
- 無し
## 動作確認状況
- ローカルで確認済
## 補足
- 無し
|
2023-06-06 07:00:11 +00:00 |
|
oura.a
|
85b8ab0919
|
ci: add Azure Static Web Apps workflow file
on-behalf-of: @Azure opensource@microsoft.com
|
2023-06-06 02:05:13 +00:00 |
|
maruyama.t
|
890b004a9c
|
Merged PR 124: [PBI1221指摘対応]ライセンス注文APIの中で画面から渡ってきたパラメータのバリデーションを行う
## 概要
[Task1901: [PBI1221指摘対応]ライセンス注文APIの中で画面から渡ってきたパラメータのバリデーションを行う](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1901)
- バリデーションチェック処理をコントローラに追加
- 未使用のimportを削除
- エラーテストで比較するメッセージが間違っていたので合わせて修正
## レビューポイント
- バリデーションのタイミングは適切か
- バリデーションの範囲に過不足がないか
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
## 補足
- licenses.controller.ts以外もセルフチェックを行いました。
|
2023-06-05 07:06:20 +00:00 |
|
saito.k
|
0ca766f309
|
Merged PR 127: Task1856: DBマイグレーション(タスク一覧取得)
## 概要
[Task1856: DBマイグレーション(タスク一覧取得)](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%2010-1?workitem=1856)
- 必要なテーブルを作成
- ユーザーグループテーブル
- ユーザーグループ所属テーブル
- ソート条件テーブル
- チェックアウト権限テーブル
## レビューポイント
- テーブル名は妥当か
- 各カラムの型や制約、コメントは妥当か
- ユーザーグループ名はNOT NULLでよい?
- ソート条件のカラム名はNOT NULLでよい?
- チェック制約を追加したが問題なさそうか
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認
- Migrate up /Migrate downでテーブル追加・削除を確認
- チェック制約で片方のカラムのみ値が入ることを確認(両方値がある or 両方NULLではNG)
## 補足
|
2023-06-02 08:13:33 +00:00 |
|
湯本 開
|
1ced5ef66d
|
Merged PR 123: API実装(I/F実装)
## 概要
[Task1836: API実装(I/F実装)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1836)
- タスクのソート条件更新APIのI/Fを実装
- タスク一覧APIのI/Fを修正
- TODO/XXXですぐ対処可能、または単純に消し忘れているものを対処
## レビューポイント
- API I/Fの修正は妥当な内容であるか
- 特にfilterはカンマ区切りでいいか、指定したものを除外という形式でいいか等
- TODO/XXXの対処は妥当な対処であるか
## 動作確認状況
- ローカルでswagger表示されることを確認
|
2023-06-02 06:12:35 +00:00 |
|
makabe.t
|
e4f84f78ba
|
Merged PR 121: リフレッシュトークンの発行方法・発行内容が不適切な問題の修正
## 概要
[Task1904: リフレッシュトークンの発行方法・発行内容が不適切な問題の修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1904)
セキュリティ・権限管理の要であるリフレッシュトークンのロール設定に関する修正ですので、開発メンバー全員にご確認いただき、変更内容について認識を合わせていただきたいです。
- リフレッシュトークンの発行で使用するロールがクライアントから送られた値をそのまま利用する実装になっていましたので、APIのロールの取り扱いを修正しました
- ユーザー追加APIで想定している(author/typist/none)のみを受け付けるようにする
- 想定外のロールの場合は400エラー
- クライアントからも想定通りのロールを送信するようロールの定数値を修正
- リフレッシュトークンを発行する際にDBの値をそのまま使わず、想定値のどれかを判断して定数を設定する
- DBに登録されているロールが想定外の文字列の場合は500エラーとなる
## レビューポイント
- トークンのロール設定が安全にできるようになっていることをメンバー全員で合意
- APIのパラメータを受け付ける際にIsInで想定ロール文字列のみを受け付けるようにしているが問題はないか
- リフレッシュトークンの生成時に直接DBのロールを設定しないようにしたが、トークンのロール設定として問題はないか
- 意図しないトークンへの権限の付与などは起こりえないか。
- クライアントからadminなどのユーザー追加時に想定していない権限を付与しようとしてもAPIではじかれるか
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
## 補足
- ユーザー追加時に登録されるロールが変更となったのでマージ後に利用する場合は再度の登録をお願いします。
|
2023-06-02 04:55:28 +00:00 |
|
oura.a
|
ab618984eb
|
Merged PR 120: DBマイグレーション
## 概要
[Task1839: DBマイグレーション](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1839)
- migrationファイルの追加(ライセンステーブル/ライセンス割り当て履歴テーブル)
## レビューポイント
-カラム名などに違和感がないか
-NOT NULL制約や型が正しいか
## UIの変更
- なし
## 動作確認状況
- MySQL Workbenchでの確認
DBのマイグレート
sql-migrate up -config=db/dbconfig.yml -env=localを実行してテーブルが作成されるのを確認
sql-migrate down -config=db/dbconfig.yml -env=localを実施して巻き戻せることも確認
## 補足
- licenses.type(ライセンス種別:トライアル/通常/カード)
- licenses.status(ライセンス状態:未割当/割り当て済/削除済)
- licenses_history.oparation_type(割り当て種別:割り当て/割り当て解除)
- licenses_history.exchange_type(ライセンス切り替え種別:なし/トライアル→通常/紙→通常)
|
2023-06-02 04:34:25 +00:00 |
|
masaaki
|
785535e4e6
|
Merged PR 125: licenseのAppRouterについて、タスク1791対応後にRouteAuthGuardの設定を入れる
## 概要
[Task1789: licenseのAppRouterについて、タスク1791対応後にRouteAuthGuardの設定を入れる](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1789)
- licenseページについてRouteAuthGuardを実装しました
※「バグ 1791: 言語情報を切り替えるとログインできない」の影響で多言語対応の確認ができないので、一時的にRouteAuthGuardを外していたのを、バグ解消により修正しました。
## レビューポイント
- 修正内容に問題ないか
## UIの変更
- 特になし
## 動作確認状況
- ローカルで確認済
## 補足
- 特になし
|
2023-06-02 04:26:08 +00:00 |
|
masaaki
|
bc3e0090ca
|
Merged PR 122: [PBI1221指摘対応]ライセンス注文ポップアップ画面のレイアウト修正
## 概要
[Task1900: [PBI1221指摘対応]ライセンス注文ポップアップ画面のレイアウト修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1900)
- 画面レイアウトについて、「Licenses」の項目がタイトルの線と重なっていたのを修正しました
- 併せて、「Licenses」の値がデザイナ画面とことなっていたので修正しました(One Year→One year)
※翻訳Excelも併せて修正しています
## レビューポイント
- 修正内容が問題ないかどうか
## 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/Task1900?csf=1&web=1&e=JHL5ZE
## 動作確認状況
- ローカルで確認済
## 補足
- 相談、参考資料などがあれば
|
2023-06-01 06:11:23 +00:00 |
|
saito.k
|
fa9027a177
|
Merged PR 116: 原因調査&対応
## 概要
[Task1792: 原因調査&対応](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1792)
- ログイン画面の描画が複数回行われる最中に、app.tsxの言語切り替えのuseEffectが走ると、B2Cのユーザー情報が取得できないので、言語切り替えのタイミングをずらすことで対応する
## レビューポイント
- 対応方法に問題はあるか
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認
## 補足
|
2023-05-31 04:38:54 +00:00 |
|
maruyama.t
|
2c935c8b52
|
Merged PR 101: API実装(ライセンス注文登録)
## 概要
[Task1685: API実装(ライセンス注文登録)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1685)
タスク 1685: API実装(ライセンス注文登録)
ライセンス注文APIを追加
## レビューポイント
登録時のDB処理方法に問題がないか。
処理、エラーハンドリングに過不足がないか。
## UIの変更
なし
## 動作確認状況
ローカルでユニットテストを実施。
ローカルでAPIを実行し、DBに登録できること・poNumberの重複チェックが想定通りに動作していることを確認。
## 補足
なし
|
2023-05-30 07:17:43 +00:00 |
|
makabe.t
|
3191e22ab6
|
Merged PR 119: 音声ファイルアップロード完了API実装
## 概要
[Task1712: 音声ファイルアップロード完了API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1712)
- 音声ファイルアップロード完了APIを実装しました。
- DBへの登録処理を追加しています
- 音声ファイルテーブル
- オプションアイテムテーブル
- 文字起こしタスクテーブル
- jwtトークンデコードがうまくいかないことがありましたので応急対応を入れています。
- 参考:https://github.com/auth0/node-jsonwebtoken/issues/875
## レビューポイント
- DBへの登録処理・内容は適切か
- JOBナンバーの採番は適切か
- jwtデコードの対応は適切か
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
|
2023-05-30 03:40:36 +00:00 |
|
oura.a
|
81dca16d5f
|
Merged PR 113: 画面実装(ライセンス注文ポップアップ)
## 概要
[Task1684: 画面実装(ライセンス注文ポップアップ)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1684)
- ライセンス注文ポップアップ画面について実装を行いました
- ライセンス情報画面について、ボタン押下時にポップアップを表示する実装も併せて実施しております
- ポップアップの表示について、表示/非表示を制御するのではなくそもそも表示しない場合はhtmlとして生成しないよう制御を入れています。
- ポップアップ表示の対応により、ポップアップクローズ時の対応が簡易化しています。(これまでは、ポップアップを閉じる際に値を初期化しなおす必要があったのですが、生成しなおす形になるので初期生成の考慮のみでよくなる)
- cssについて、5/26時点で最新のファイルを取り込みました
## レビューポイント
- ポップアップ生成周りの考え方、初期化の実装内容
- images、cssについては、最新のデザイナさんのものを取り込んだ形なので、レビュー対象外でお願いします
## 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/Task1684?csf=1&web=1&e=xLRn4D
## 動作確認状況
- ローカルで確認
## 補足
- 無し
|
2023-05-30 01:07:52 +00:00 |
|
makabe.t
|
a430bd01e5
|
Merged PR 118: DBマイグレーション
## 概要
[Task1713: DBマイグレーション](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1713)
[Task1711: [タスク追加PBIでまとめて対応] Userテーブルのaccepted_terms_versionのNOT NULL制約を削除する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_workitems/edit/1711)
- タスク追加で必要となるテーブル作成するマイグレーションファイルを追加
- 音声ファイル
- 文字起こしタスク
- オプションアイテム(音声ファイルメタデータ)
- Userテーブルの「同意済み利用規約バージョン」カラムのNOT NULL制約を外すマイグレーションファイルを追加
## レビューポイント
- タスク追加で必要となるテーブルを1ファイルにまとめているが問題ないか
- 各テーブルのカラム名、型は適切か
## UIの変更
- 無し
## 動作確認状況
- ローカルで確認
- UP/DOWN
|
2023-05-26 06:23:41 +00:00 |
|
masaaki
|
75dc79c5e7
|
Merged PR 117: [Sp9-1絶対着手] eslintのanchor-is-validルールを無効にする(ライセンス注文機能の画面実装と同時にやる)
## 概要
[Task1664: [Sp9-1絶対着手] eslintのanchor-is-validルールを無効にする(ライセンス注文機能の画面実装と同時にやる)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1664)
- eslintのanchor-is-validルール無効化について、個別に記載しなくても全体的に無効化されるよう対応しました
- 合わせて、既存ソースで個別に無効化していた部分を削除しました
## レビューポイント
- 特にありません
## UIの変更
- 無し
## 動作確認状況
- ローカルで確認確認済
## 補足
- 無し
|
2023-05-25 11:28:47 +00:00 |
|
oura.a
|
4ec3d7c51f
|
Merged PR 114: [MISOチーム] ユーザ追加時のAuthorIDの重複チェックがエラーの場合にAzureにアカウント追加しないよう修正する
## 概要
[Task1765: [MISOチーム] ユーザ追加時のAuthorIDの重複チェックがエラーの場合にAzureにアカウント追加しないよう修正する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1765)
タスク 1765: [MISOチーム] ユーザ追加時のAuthorIDの重複チェックがエラーの場合にAzureにアカウント追加しないよう修正する
・ユーザ追加時、Azureへのユーザ登録前に同一アカウント内のAuthorIDの重複チェックを追加。
## レビューポイント
処理に問題がないか。
## UIの変更
なし
## 動作確認状況
ローカルでユーザ追加APIを実行し、同一AuthorIDの際にエラーが出ることを確認。
## 補足
なし
|
2023-05-25 06:54:50 +00:00 |
|
oura.a
|
24a784b02f
|
Merged PR 115: [MISOチーム] typeが「None」のステータスについてアカウント追加時(none)とユーザ追加時(None)で表記ゆれがある
## 概要
[Task1766: [MISOチーム] typeが「None」のステータスについてアカウント追加時(none)とユーザ追加時(None)で表記ゆれがある](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1766)
タスク 1766: [MISOチーム] typeが「None」のステータスについてアカウント追加時(none)とユーザ追加時(None)で表記ゆれがある
・アカウント追加時のtypeを「none」から「None」に修正。
## レビューポイント
なし
## UIの変更
なし
## 動作確認状況
ローカルでアカウント登録APIを実行し、DBへの登録内容が「None」に修正されていることを確認。
## 補足
なし
|
2023-05-25 01:37:03 +00:00 |
|
iwata
|
fa23f7f2e2
|
Revert "anchor-is-validルールを全体的に無効化"
This reverts commit dc8fa3564b8aaed37a2f8ef1d0f1334302661a1c.
|
2023-05-24 11:48:05 +09:00 |
|
iwata
|
9d73f24b55
|
Revert "個別にanchor-is-validを使っている処理を削除"
This reverts commit 2c5909189e741ccba98add5ce581e7855b40b65e.
|
2023-05-24 11:47:47 +09:00 |
|