PR

【10/25】海賊応援パック #2361

kouho opened this at 2022-10-25T15:58:16; closed

Description

opened by kouho at 2022-10-25T15:58:16; updated 2022-10-26T10:23:43

概要

https://docs.google.com/spreadsheets/d/1QWcw1jlSBVG5Q3lUBD_GEojao3VoV88i0MLa7_ZmIhM/edit#gid=1969531836

DB更新

https://git.dev.d-es.co.jp/gitbucket/jolly/jrye-db/pull/208

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

記載内容に誤りがないかどうか

TODO

ショップでの売場で画像を使わずパックの中身表示させる方式に変更

Comment

#23237 by kouho at 2022-10-25T15:58:51; updated 2022-10-25T15:58:51

@wada
レビューお願いいたします

Comment

#23238 by wada at 2022-10-25T16:25:35; updated 2022-10-25T16:25:35

@kouho
なぜgold_packで実装しましたか?
パックにGOLDは含まれていないので確認です。

Comment

#23239 by kouho at 2022-10-25T16:31:25; updated 2022-10-25T16:31:25

@wada
お試しパックも同時開催しているためitem_packよりもgold_packのほうが問題なく開催できるかと思いました。
バナーの表示等も気にせずできそうでしたので。

Comment

#23240 by wada at 2022-10-25T16:49:35; updated 2022-10-25T16:49:35

@kouho
了解しました。

Comment

#23241 by wada at 2022-10-25T16:52:25; updated 2022-10-25T16:52:25

@kouho
アイテムパックとGOLDパックは配布方法が違うのですが、こちらは問題ありませんか?

  • アイテムパック

直接配布

  • GOLDパック

プレゼント配布

Comment

#23242 by wada at 2022-10-25T16:57:05; updated 2022-10-25T16:57:05

@kouho
以下の判定でエラーが出ていました。

// ゴールドパック購入判定
else if( $shop_item_no >= $_df['shop_gold_set']['shop_item_no'][1] && $shop_item_no <= $_df['shop_gold_set']['shop_item_no'][3] ){

Comment

#23243 by wada at 2022-10-25T17:07:44; updated 2022-10-25T17:09:09

@kouho
GOLDパックは、カード配布の対応がまだなので、対応を入れる必要があります。

Shop_payment.send_presents(array $user_info)

Comment

#23244 by kouho at 2022-10-25T18:08:21; updated 2022-10-25T18:08:21

@wada
確認ありがとうございます。指摘箇所を修正しました。

> 戻すのを忘れそうなので、10165~10167を残しておいて欲しいです。

10165~10167を残しておきました。

> 管理画面側で問題が出ているなら、上書き更新で大丈夫です。

野田くんに確認したところ管理画面側で海賊応援パックがゴールドパックに含まれるのが好ましくないとのことでしたが、
表示をゴールドパックからアイテムパック2に変更すればよいとのことなので管理画面の表示をそのように修正予定です。

> アイテムパックとGOLDパックは配布方法が違うのですが、こちらは問題ありませんか?
>
アイテムパック
直接配布
GOLDパック
プレゼント配布

こちらはプレゼント配布で問題ないとのことでした

>以下の判定でエラーが出ていました。
>

// ゴールドパック購入判定
else if( $shop_item_no >= $_df['shop_gold_set']['shop_item_no'][1] && $shop_item_no <= $_df['shop_gold_set']['shop_item_no'][3] ){

in_arrayに判定を変更して修正しました。

> GOLDパックは、カード配布の対応がまだなので、対応を入れる必要があります。
>


カード配布の対応とコインガチャチケット配布対応を入れました。

Comment

#23245 by wada at 2022-10-25T18:35:36; updated 2022-10-25T18:35:36

@kouho
対応ありがとうございます。確認しました。
lib/ajax/shop_select.inc$_df['shop_gold_set']['shop_item_no'][1]も対応お願いします。

Comment

#23246 by kouho at 2022-10-26T10:08:45; updated 2022-10-26T10:08:45

@wada
確認ありがとうございます。
> lib/ajax/shop_select.incの$_df['shop_gold_set']['shop_item_no'][1]も対応お願いします。

こちらも対応しました。

Comment

#23247 by wada at 2022-10-26T10:21:32; updated 2022-10-26T10:21:32

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

wada add_label at 2022-10-26T10:21:37: 和田 LGTM
kouho merge at 2022-10-26T10:23:43: 【10/25】海賊応援パック
kouho close at 2022-10-26T10:23:43: Close

commit comment

#142 by wada at 2022-10-25T16:50:35; lib/conf/define_y.inc / old:None new:2375
01b2c89fa0dd697a5800be65422b9871728564fd

@kouho
戻すのを忘れそうなので、10165~10167を残しておいて欲しいです。
管理画面側で問題が出ているなら、上書き更新で大丈夫です。

Commit range

base317b7131c324cf72eff2396ba1eb6f221fecc17f
headb3193d80de09401ed08df7e8cd2260af4cf24dd7
base branchjolly/jr-web:develop
head branchjolly/jr-web:item_pack_1027

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed23 filesView patch
statusfile
modifiedlib/ajax/shop_payment.inc
modifiedlib/ajax/shop_select.inc
modifiedlib/conf/define_y.inc
modifiedlib/models/Shop_payment.php
addedpublic_html/img/assortment/10188/head.jpg
addedpublic_html/img/assortment/10188/spgacha_item_1.jpg
addedpublic_html/img/assortment/10188/spgacha_item_2.jpg
addedpublic_html/img/assortment/10188/spgacha_item_3.jpg
addedpublic_html/img/assortment/10189/spgacha_item_1.jpg
addedpublic_html/img/assortment/10189/spgacha_item_2.jpg
addedpublic_html/img/assortment/10189/spgacha_item_3.jpg
addedpublic_html/img/info_event/info_event_560_1.jpg
addedpublic_html/img/info_event/info_event_560_2.jpg
addedpublic_html/img/item/120/10188.jpg
addedpublic_html/img/item/120/10189.jpg
addedpublic_html/img/item/150/10188.jpg
addedpublic_html/img/item/150/10189.jpg
addedpublic_html/img/item/60/10188.jpg
addedpublic_html/img/item/60/10189.jpg
addedpublic_html/img/item/70/10188.jpg
addedpublic_html/img/item/70/10189.jpg
addedpublic_html/img/shop/banner/gold_pack/10188_start_1.jpg
addedpublic_html/img/shop/banner/gold_pack/10188_start_2.jpg
Patch body is stored as a separate file to keep this page lightweight.

Review comment

#142 by wada at 2022-10-25T16:50:35; lib/conf/define_y.inc / old:None new:2375
01b2c89fa0dd697a5800be65422b9871728564fd

@kouho
戻すのを忘れそうなので、10165~10167を残しておいて欲しいです。
管理画面側で問題が出ているなら、上書き更新で大丈夫です。