PR

【11/7】クリスマスガチャ前半 #2374

wada opened this at 2022-11-02T13:56:17; closed

Description

opened by wada at 2022-11-02T13:56:17; updated 2022-11-10T17:05:18

概要

【11/14, 12/7】クリスマスガチャ前後半 #2367
https://docs.google.com/spreadsheets/d/1FEr_iFhyzBWU9OH8g_Ffgnwjy7Cgb2xUe6piCc6FAwI/edit#gid=2032159297

ドリンクのIDを拡張した(1500001~)ため、ドリンク判定の修正
10連のドリンクおまけは、新しくログテーブルを作成して管理

DB更新内容

【11/7】クリスマスガチャ前半 jolly/jrye-db#211

追加テーブル

CREATE TABLE `log_user_gacha_bonus` (
  `id` int(11) unsigned NOT NULL AUTO_INCREMENT,
  `user_no` int(10) unsigned NOT NULL,
  `gacha_id` int(10) unsigned NOT NULL,
  `item_type` int(10) unsigned NOT NULL,
  `item_no` int(10) unsigned NOT NULL,
  `item_count` int(10) unsigned NOT NULL,
  `reg_date` datetime NOT NULL,
  PRIMARY KEY (`id`),
  KEY `user_gacha_bonus` (`user_no`,`gacha_id`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4;
wada add_label at 2022-11-02T14:41:37: レビュー待ち

Comment

#23394 by wada at 2022-11-02T14:42:01; updated 2022-11-02T14:42:01

@kouho
レビューお願いします。

Comment

#23396 by kouho at 2022-11-02T14:51:41; updated 2022-11-02T14:51:41

@wada
ドリンクのIDを1500001以降にする件について一点気になったのですが、
アイテムパックでのID設定についてどうしましょう?
ID100000超えはカードという前提の処理が入っています。

今現在は問題ありませんので一旦気にせずいきますか?
もしもアイテムパックやゴールドパックに新規IDを割り振ったドリンクを封入する場合ややこしくなりそうだと思いました。

それともこれを期に master_shop_item に item_typeを追加する対応を並行してこちらで進めましょうか?

Comment

#23401 by wada at 2022-11-02T15:05:31; updated 2022-11-02T15:05:31

@kouho
現状ショップに追加される予定はないので、
追加される際に、master_shop_itemのissueを対応したら良いかと思います。

wada merge at 2022-11-08T17:38:53: Merged by user
wada close at 2022-11-08T17:38:53: Close

Comment

#23531 by wada at 2022-11-08T17:40:23; updated 2022-11-08T17:40:23

@kouho
間違えてマージしちゃいましたが引き続きレビューお願いします。

Comment

#23534 by kouho at 2022-11-08T18:07:35; updated 2022-11-08T18:07:35

@wada
専用ドリンクをタップした時の挙動で他の専用ドリンクをタップしたら消えるようにできませんか?今は同じドリンクをタップしないと消えないようです。
!スクリーンショット 2022-11-08 18

クリスマスガチャのラインナップのページ下部の文言の仲間スキルについては今回関係ないかと思います
!スクリーンショット 2022-11-08 18

Comment

#23535 by kouho at 2022-11-08T18:52:27; updated 2022-11-08T18:52:27

@wada
もしかしたらこれまでの季節ガチャでも同じかもですが....
限定カードが獲得できると書いていますが限定カードが何かガチャページ内に明記されていないように見えました。

ラインナップ一覧の表示を以下のステップアップガチャの表記を参考に
今回の限定海賊カードはコレ!の表示にしていただけないでしょうか?

!スクリーンショット 2022-11-08 18

!スクリーンショット 2022-11-08 18

Comment

#23536 by kouho at 2022-11-08T18:57:06; updated 2022-11-08T18:57:06

@wada
!スクリーンショット 2022-11-08 18

クリスマスガチャを引ききった後の表示について
獲得できる専用アイテムの帯の下に 専用ドリンクを全て引ききりました のようなテキストを表示させていただけないでしょうか?グレーアウトされていないものとグレーアウトされているものが混ざっている場合どれを引いたかわかりやすいと思いますが全てグレーアウトになると分かりにくいように思いました。

Comment

#23540 by wada at 2022-11-09T11:46:44; updated 2022-11-09T11:46:44

@kouho
以下対応しました。

> 同じドリンクをタップしないと消えない
ラインナップのページ下部の文言の仲間スキルについて
ラインナップ一覧の表示
クリスマスガチャを引ききった後の表示について

Comment

#23563 by kouho at 2022-11-09T15:19:42; updated 2022-11-09T15:20:22

@wada
すみません、以下コメント書く場所間違えました。
対応ありがとうございます。修正確認しました。

その他動作は問題ないですが、仕様面の中身について改めて気になりました。
以下問題がないか再度確認していただきたいです。

レアリティの違うクリスマスプレゼントが同一確率
クリスマスプレゼントの確率が低い
クリスマスプレゼントが経験値アップアイテムなので海賊の魂と役割が被る。
→ 恒常カードやアイテムの確率を下げてクリスマスプレゼントの確率を上げてもよいのではと思いました。

以下は比較した過去の仕様書です。

昨年のクリスマスガチャ
https://docs.google.com/spreadsheets/d/1POt1TsuqH4yG54SHgHZWV3ECfN_eP72fQue_R6oyUco/edit#gid=2032159297

復刻クリスマスガチャ
https://docs.google.com/spreadsheets/d/13pUhJ-pu6DLPciy_qEMoJAm__Foh0o94wnbUNoHlCm8/edit#gid=2032159297

wada refer at 2022-11-10T17:05:18: 2396:【11/14 12:00】クリスマスガチャ前半

Commit range

based208ac074d18efae1ef82a3f17ad49f10522da38
head7339da619bc3ed3f8bd3ddb1b792a880a8478859
base branchjolly/jr-web:develop
head branchjolly/jr-web:season_gacha_276

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed5 filesView patch
statusfile
modifiedlib/public/Bar-SPGachaCheck.inc
modifiedlib/public/Bar-SPGachaExec.inc
modifiedlib/public/Bar-SPGachaResult.inc
modifiedpublic_html/template/Bar-SPGachaCheck.tpl
modifiedpublic_html/template/Bar-SPGachaResult.tpl
Patch body is stored as a separate file to keep this page lightweight.