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 |
|
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 |
|
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 |
|
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 |
|
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 |
|
oura.a
|
810be5c055
|
Merged PR 99: ライセンス注文テーブルをMySQL上に作成する
## 概要
[Task1680: ライセンス注文テーブルをMySQL上に作成する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1680)
タスク 1680: ライセンス注文テーブルをMySQL上に作成する
ライセンス注文テーブルを作成するマイグレーションファイルを作成。
## レビューポイント
テーブル定義・カラム定義が正しいかどうか。
テーブル名・カラム名が適切かどうか。
## UIの変更
なし
## 動作確認状況
ローカルで動作確認し、テーブルが作成されることを確認。
また、世代巻き戻しでテーブルが削除されることを確認。
## 補足
なし
|
2023-05-24 01:30:36 +00:00 |
|
oura.a
|
3f16e84317
|
Merged PR 88: usersテーブルにAuto Renew/License Alert/Notificationの列を追加する
## 概要
Task1619: usersテーブルにAuto Renew/License Alert/Notificationの列を追加する
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%207-1?workitem=1619
ユーザテーブルへ「autoRenew、licenseAlert、notification」のカラムを追加するマイグレーションファイルを作成
レビューポイント
カラム定義が認識通りであるか
既存処理には定義追加以外の影響はないと判断したが、問題ないか
(user.entityとusers.repository.serviceへの定義追加は
タスク1593「API実装(ユーザー登録)」で事前に行っていました。)
UIの変更
なし
動作確認状況
ローカルDBでマイグレーションを実施。
カラムの追加と「autoRenew、licenseAlert、notification」が未指定でも「1」となることを確認。
補足
なし
|
2023-05-11 09:08:16 +00:00 |
|
湯本 開
|
9f5252baf8
|
Merged PR 56: API実装(アカウント登録)
## 概要
[Task1496: API実装(アカウント登録)](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1496)
- アカウント登録のAzureAD B2Cへのユーザー作成部分以外について実装
- migration SQLに足りない要素があった箇所を修正(default追加)
- 各種環境変数を追加
- 秘密鍵/公開鍵を取得する方法を環境変数に変更(KeyVaultからWebAppsの機能で環境変数へ流し込む想定)
## レビューポイント
- 実装方法として問題がありそうな箇所が存在しないか
- 可読性の低い箇所が存在しないか
- Moduleの分け方、つなげ方などは問題ないか
- ラフスケッチと違い、Account作成と管理者ユーザー作成を同一トランザクションで行うよう修正したが問題ないか
## 動作確認状況
- メール送信以外はローカルで確認、メール送信部分は未確認
|
2023-03-29 03:55:44 +00:00 |
|
湯本 開
|
6fe1cc4d6d
|
Merged PR 50: タスク 1500: DBのテーブル作成SQL実装
[タスク 1500: DBのテーブル作成SQL実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_workitems/edit/1500)
- SQLを実装
- sql-migrate前提で設定ファイルを追加
# レビューポイント
- 本PBIの範囲で不足しているものはなさそうか
# 動作確認状況
- ローカルのMySQL DBにSQLを流して確認
|
2023-03-22 08:34:36 +00:00 |
|