OMDSCloud

nik/OMDSCloud

Fork 0

Commit Graph

Author	SHA1	Message	Date
masaaki	422622c9ef	Merged PR 351: license_allocation_historyのテーブル名が誤っている ## 概要 [Task2469: license_allocation_historyのテーブル名が誤っている](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2469) - 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず） - 何をどう変更したか、追加したライブラリなど - このPull Requestでの対象/対象外 - 影響範囲（他の機能にも影響があるか） ## レビューポイント - 特にレビューしてほしい箇所 - 軽微なものや自明なものは記載不要 - 修正範囲が大きい場合などに記載 - 全体的にや仕様を満たしているか等は本当に必要な時のみ記載 ## UIの変更 - Before/Afterのスクショなど - スクショ置き場 ## 動作確認状況 - ローカルで確認、develop環境で確認など ## 補足 - 相談、参考資料などがあれば	2023-08-23 01:40:13 +00:00
maruyama.t	bdd10aabf6	Merged PR 324: API修正（ライセンス割り当てAPI）_履歴部分 ## 概要 [Task2366: API修正（ライセンス割り当てAPI）_履歴部分](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2366) - 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず） - 何をどう変更したか、追加したライブラリなど allocateLicenseの割り当て履歴テーブルのレコードを作成するロジックを追加しました。 createLicenseの引数を追加して、種別を指定して作成できるようにしました。既存テストで項目の値などでべた書きしていた箇所を、定数を使うよう修正しました。 - このPull Requestでの対象/対象外 - 影響範囲（他の機能にも影響があるか） createLicenseの引数を追加して、種別を指定して作成できるようにしました。ユニットテストでcreateLicenseを使っている既存テストは再テスト済み ## レビューポイント - 特にレビューしてほしい箇所 - 軽微なものや自明なものは記載不要 - 修正範囲が大きい場合などに記載 - 全体的にや仕様を満たしているか等は本当に必要な時のみ記載 lisence_allocation_historyテーブルのスキーマを変更しています。プライマリとしてのidを追加（ユーザIDになっていた）、 allocate_typeをallocatedに変更して、boolの値で管理するように変更（割り当て解除or割り当てしかもたないので）。 ## UIの変更なし ## 動作確認状況 - ローカルで確認 ## 補足 - 相談、参考資料などがあれば	2023-08-22 06:06:22 +00:00
maruyama.t	077b63b0dc	Merged PR 312: DBマイグレーション ## 概要 [Task2357: DBマイグレーション](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2357) - 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず） - 何をどう変更したか、追加したライブラリなど - このPull Requestでの対象/対象外 - 影響範囲（他の機能にも影響があるか）新規テーブルのため、なし ## レビューポイント - 特にレビューしてほしい箇所ラフスケッチ上、正式ライセンス切り替えフラグと表現されている項目について、 `switch_from_type VARCHAR(1024) NOT NULL COMMENT '切り替え元種別(特になし/カード/トライアル)',` としたが、認識あっているか。（直近の、ライセンス.ライセンス種別の意かどうか） ## UIの変更 - Before/Afterのスクショなどとくになし ## 動作確認状況 - ローカルで確認「sql-migrate up -config=db/dbconfig.yml -env=local」を実行、DB反映確認。「sql-migrate down -config=db/dbconfig.yml -env=local」を実行すると、1世代分巻き戻せることを確認 ## 補足 - 相談、参考資料などがあれば	2023-08-09 02:49:33 +00:00

Author

SHA1

Message

Date

masaaki

422622c9ef

Merged PR 351: license_allocation_historyのテーブル名が誤っている

## 概要
[Task2469: license_allocation_historyのテーブル名が誤っている](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2469)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）

## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載

## UIの変更
- Before/Afterのスクショなど
- スクショ置き場

## 動作確認状況
- ローカルで確認、develop環境で確認など

## 補足
- 相談、参考資料などがあれば

2023-08-23 01:40:13 +00:00

maruyama.t

bdd10aabf6

Merged PR 324: API修正（ライセンス割り当てAPI）_履歴部分

## 概要
[Task2366: API修正（ライセンス割り当てAPI）_履歴部分](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2366)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
allocateLicenseの割り当て履歴テーブルのレコードを作成するロジックを追加しました。
createLicenseの引数を追加して、種別を指定して作成できるようにしました。
既存テストで項目の値などでべた書きしていた箇所を、定数を使うよう修正しました。
- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）
createLicenseの引数を追加して、種別を指定して作成できるようにしました。
ユニットテストでcreateLicenseを使っている既存テストは再テスト済み

## レビューポイント
- 特にレビューしてほしい箇所
- 軽微なものや自明なものは記載不要
- 修正範囲が大きい場合などに記載
- 全体的にや仕様を満たしているか等は本当に必要な時のみ記載
lisence_allocation_historyテーブルのスキーマを変更しています。
プライマリとしてのidを追加（ユーザIDになっていた）、
allocate_typeをallocatedに変更して、boolの値で管理するように変更（割り当て解除or割り当てしかもたないので）。

## UIの変更
なし

## 動作確認状況
- ローカルで確認

## 補足
- 相談、参考資料などがあれば

2023-08-22 06:06:22 +00:00

maruyama.t

077b63b0dc

Merged PR 312: DBマイグレーション

## 概要
[Task2357: DBマイグレーション](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/2357)

- 元PBI or タスクへのリンク（内容・目的などはそちらにあるはず）
- 何をどう変更したか、追加したライブラリなど
- このPull Requestでの対象/対象外
- 影響範囲（他の機能にも影響があるか）
新規テーブルのため、なし

## レビューポイント
- 特にレビューしてほしい箇所
ラフスケッチ上、正式ライセンス切り替えフラグと表現されている項目について、
`switch_from_type VARCHAR(1024) NOT NULL COMMENT '切り替え元種別(特になし/カード/トライアル)',`
としたが、認識あっているか。
（直近の、ライセンス.ライセンス種別の意かどうか）

## UIの変更
- Before/Afterのスクショなど
とくになし

## 動作確認状況
- ローカルで確認
「sql-migrate up -config=db/dbconfig.yml -env=local」を実行、DB反映確認。
「sql-migrate down -config=db/dbconfig.yml -env=local」を実行すると、1世代分巻き戻せることを確認

## 補足
- 相談、参考資料などがあれば

2023-08-09 02:49:33 +00:00

3 Commits