PR

【1/26】バレンタインガチャ前半 #2559

wada opened this at 2023-01-25T17:22:56; closed

Description

opened by wada at 2023-01-25T17:22:56; updated 2023-01-30T17:50:56

概要

https://docs.google.com/spreadsheets/d/1pp0RyHtpDkzmfhNk7hlMScqpsZiJN0u6JJOOqPhKHIE/edit#gid=2032159297

バレンタインガチャの抽選処理を最新の仕様に調整
10連おまけを取り切りアイテムに変更
専用ガチャページの作成

DB更新内容

【1/26】バレンタインガチャ前半 jolly/jrye-db#288

テスト方法、確認した内容

Comment

#25272 by wada at 2023-01-25T18:17:33; updated 2023-01-25T18:17:33

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

wada add_label at 2023-01-25T18:17:36: レビュー待ち

Comment

#25284 by kouho at 2023-01-26T11:13:19; updated 2023-01-26T11:13:19

@wada
Bar-SPGacha_309.tpl ・SPGachaConfig.class.php ガチャ名が仲間連携ガチャになっていました。

Comment

#25285 by kouho at 2023-01-26T11:25:35; updated 2023-01-26T11:25:35


display_valentine_bonusの指摘についてはそもそもいらない判定なら消しても良さそうです。
$gacha_result['valentine_bonus_item']もundefinedのようですし。

Comment

#25286 by kouho at 2023-01-26T11:32:55; updated 2023-01-26T11:32:55

@wada
public_html/img/gacha/gacha_coin_object309.png
おまけの強化アイテムの画像ですが、
仮にダマスカスの剣【怒りの黒髭ティーチ】を獲得した後に上の画像が表示され続けるのは大丈夫でしょうか?
画像の意図は強化アイテムのどれかが手に入るという意味で画像の中央に?がつけられていると思いますが、
暗くしていてもどの強化アイテムか明らかにわかる状態なので出ることのない強化アイテムが表示され続けることが気になりました。

Comment

#25287 by kouho at 2023-01-26T11:44:50; updated 2023-01-26T11:44:50

@wada
おまけを引き切った後も

の表示が出続けていることに違和感がありまして、すぐ下の注意事項にも同様の記載がありますので上部の方は消してよいのではと思います。

![スクリーンショット 2023-01-26 11](https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/1674700828828g57xtEJLtm)

Comment

#25288 by kouho at 2023-01-26T11:48:43; updated 2023-01-26T11:48:43

@wada
一部ユーザーからのガチャの注意事項に関する問い合わせが続いているのが気になり、
注意事項について以下の表記の方がわかりやすいかと思いました。

★10回連続ガチャをひくと10枚+おまけ(強化アイテム1個)が必ずついてきます。
★おまけの強化アイテムは、1種類につき1回(合計8回)獲得できます。

★8回目までは10回連続ガチャをひくと10枚+おまけ(強化アイテム1個)が必ずついてきます。
★おまけの強化アイテムは、1種類につき1回獲得できます。

Comment

#25293 by wada at 2023-01-26T11:55:13; updated 2023-01-26T11:55:13

@kouho
ガチャ名と不要処理削除しました。

Comment

#25294 by kouho at 2023-01-26T11:56:23; updated 2023-01-26T11:56:23

@wada
バレンタインガチャのヘッダー画像がいつものものになっておりましたので告知と同じ
親愛と入った画像に変更した方がよいかと思います。

 public_html/img/gacha/309/gacha_result_top.jpg
 public_html/img/gacha/309/gacha309_title.jpg

Comment

#25295 by kouho at 2023-01-26T11:58:35; updated 2023-01-26T11:58:35

@wada
> ガチャ名と不要処理削除しました。

ありがとうございます。確認しました。

Comment

#25296 by wada at 2023-01-26T12:00:41; updated 2023-01-26T12:00:41

@kouho
画像と文言の分かりやすさは、以前チーム内で話が出たことがあるのですが、
どんな表現をしても、受取手によって分かりやすさは変わってしまうので、必要な情報が記載されているかを重視しています。

今回のケースでは、
> おまけの強化アイテムの画像ですが、

強化アイテムは被りなしの記載があるので問題なし

> おまけを引き切った後も

引き切った後の情報があるので問題なし

> 一部ユーザーからのガチャの注意事項に関する問い合わせが続いているのが気になり、

情報に変化がないため問題なし

かなと思っています。

@noda さんはどう思いますか?

Comment

#25298 by noda at 2023-01-26T12:19:30; updated 2023-01-26T12:20:16

いずれもこちらではまだ開発環境に反映されていないため、ページ全体が確認できないので現時点での判断となります。

>おまけの強化アイテムの画像ですが、

とくに問題は感じない

>おまけを引き切った後も

記載箇所によりますが、10連ガチャボタンすぐ下にあるのであれば消したほうがいいかと思います。

>一部ユーザーからのガチャの注意事項に関する問い合わせが続いているのが気になり、

理解しやすさで言えば皇甫さんの記載のほうが勘違いが起こらなさそうです。
@wada

Comment

#25299 by wada at 2023-01-26T12:24:00; updated 2023-01-26T12:24:00

@noda @kouho
了解しました。では一番下の注意文言のみ対応します。

Comment

#25301 by wada at 2023-01-26T12:38:48; updated 2023-01-26T12:38:48

@kouho
文言変更しました。

Comment

#25302 by kouho at 2023-01-26T13:25:53; updated 2023-01-26T13:25:53

@wada
> 文言変更しました。

対応ありがとうございます。確認しました。

Comment

#25303 by kouho at 2023-01-26T13:27:17; updated 2023-01-26T13:27:17

@wada
ガチャページと結果ページのヘッダー画像以外確認OKです

Comment

#25311 by wada at 2023-01-26T16:15:46; updated 2023-01-26T16:15:46

@kouho
ヘッダ画像更新しました。

kouho add_label at 2023-01-26T16:25:23: 皇甫 LGTM

Comment

#25313 by kouho at 2023-01-26T16:25:40; updated 2023-01-26T16:25:40

@wada
ありがとうございます。確認OKです。

wada merge at 2023-01-27T09:57:28: 【1/26】バレンタインガチャ前半
wada close at 2023-01-27T09:57:28: Close
wada refer at 2023-01-30T10:41:15: 2568:【2/1 12:00】バレンタインガチャ前半リリース
wada refer at 2023-01-30T17:50:56: 2568:【1/31 12:00】バレンタインガチャ前半リリース

commit comment

#156 by kouho at 2023-01-26T11:21:34; lib/conf/class/SPGachaConfig.class.php / old:50 new:None
6c37f821326bfbedc82645d011d5a6abab535b3a

@wada
display_valentine_bonus が定義されないことでBar-SPGachaExec.incの以下のif文で Undefined offset: 309 のエラーが出ておりました。
display_valentine_resultに判定側を変えた方がよいかもしれません。確認お願いいたします。

if($_df['sp_gacha']['display_valentine_bonus'][$gacha_id] && $gacha_result['valentine_bonus_item']){

Commit range

base713129e2dfa911aa164d2ad2aac1609de65ae7c7
head6c37f821326bfbedc82645d011d5a6abab535b3a
base branchjolly/jr-web:develop
head branchjolly/jr-web:season_gacha_0201

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed26 filesView patch
statusfile
modifiedlib/SubFunc.inc
modifiedlib/conf/class/SPGachaConfig.class.php
addedlib/models/gacha/Gacha_interface.php
modifiedlib/models/gacha/SPGacha_manager.php
modifiedlib/models/gacha/Season_random.php
modifiedlib/models/gacha/Simple_random.php
modifiedlib/models/gacha/Valentine.php
modifiedlib/public/Bar-SPGacha.inc
modifiedlib/public/Bar-SPGachaExec.inc
modifiedlib/public/Bar-SPGachaResult.inc
addedpublic_html/img/gacha/309/banner_gacha309.jpg
addedpublic_html/img/gacha/309/banner_gacha309_2.jpg
addedpublic_html/img/gacha/309/gacha309.jpg
addedpublic_html/img/gacha/309/gacha309_2.jpg
addedpublic_html/img/gacha/309/gacha309_cardinfo.jpg
addedpublic_html/img/gacha/309/gacha309_info1.jpg
addedpublic_html/img/gacha/309/gacha309_info2.jpg
addedpublic_html/img/gacha/309/gacha309_info3.jpg
addedpublic_html/img/gacha/309/gacha309_info4.jpg
addedpublic_html/img/gacha/309/gacha309_title.jpg
addedpublic_html/img/gacha/gacha_coin_object309.png
addedpublic_html/img/gacha/gacha_coin_object309_2.png
modifiedpublic_html/template/Bar-SPGachaCheck.tpl
modifiedpublic_html/template/Bar-SPGachaList.tpl
modifiedpublic_html/template/Bar-SPGachaResult.tpl
addedpublic_html/template/Bar-SPGacha_309.tpl
Patch body is stored as a separate file to keep this page lightweight.

Review comment

#156 by kouho at 2023-01-26T11:21:34; lib/conf/class/SPGachaConfig.class.php / old:50 new:None
6c37f821326bfbedc82645d011d5a6abab535b3a

@wada
display_valentine_bonus が定義されないことでBar-SPGachaExec.incの以下のif文で Undefined offset: 309 のエラーが出ておりました。
display_valentine_resultに判定側を変えた方がよいかもしれません。確認お願いいたします。

if($_df['sp_gacha']['display_valentine_bonus'][$gacha_id] && $gacha_result['valentine_bonus_item']){