Description
opened by kouho at 2023-05-22T18:43:14; updated 2023-05-23T16:34:34
概要
ロキの宝箱がリリースされていない時にSTEP81以降にSTEPごとに勝利の証50個が獲得できるようにする対応用のPRです。画像も入れています。
テスト方法、確認した内容
STEP81以降達成時の確認
シーズン1
- [x] 獲得した証はプレゼントに表示されるか
- [x] 獲得した証の個数は正しく表示されるか
- [x] デイリーミッションの達成時のSTEP増加時は問題ないか
- [x] シーズンミッションの達成時のSTEP増加時は問題ないか
- [x] STEP購入での達成時のSTEP増加時は問題ないか
シーズン2
TODO
Comment
#27673 by kouho at 2023-05-22T18:43:34; updated 2023-05-22T18:43:34
Comment
#27693 by wada at 2023-05-23T11:24:21; updated 2023-05-23T11:24:21
@kouho
S_error_logの削除をお願いします。
Comment
#27694 by wada at 2023-05-23T11:41:27; updated 2023-05-23T11:41:27
@kouho
バナー画像がボタンの体をなしているので、告知などに飛ばした方が自然かと思います。
シーズン2以降、同じ役割のロキの宝箱バナー画像がリンクになっているので統一したいというのもあります。
<img src="https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/1684809487275w61EJoU8oH" width="400">
Comment
#27695 by wada at 2023-05-23T11:53:16; updated 2023-05-23T11:53:16
@kouho
すいません、lib/models/Shop_payment.phpのget_return_data()でpresent_flgの対応が漏れていました。
対応できるようであればお願いします。
//プレゼントするか
$present_items = array_merge(array_values($_df['shop_gold_set']['shop_item_no']), [10003, 10004, 10005]);
$return['item_data']['present_flg'] = in_array($this->shop_item_no, $present_items);
Comment
#27696 by kouho at 2023-05-23T14:43:19; updated 2023-05-23T14:43:19
@wada
確認ありがとうございます。以下対応いたしました。
> S_error_logの削除をお願いします。
すみません。削除しました。
> バナー画像がボタンの体をなしているので、告知などに飛ばした方が自然かと思います。
STEP報酬追加の告知にリンクする形にしようと思います。
> lib/models/Shop_payment.phpのget_return_data()でpresent_flgの対応が漏れていました。
Season_passのクラスのget_return_dataにpresent_flgを入れて、
item_classのget_return_dataでpresent_flgがあればin_arrayの判定を省く形にしてみました。
Comment
#27697 by wada at 2023-05-23T15:44:49; updated 2023-05-23T15:44:49
@kouho
ありがとうございます。確認OKです。
wada add_label at 2023-05-23T15:44:52: 和田 LGTM
kouho merge at 2023-05-23T16:00:34: 【5/23】ロキの宝箱がリリースされていない時STEP81以降の対応
kouho close at 2023-05-23T16:00:34: Close
kouho refer at 2023-05-23T16:34:34: 2804:【5/23】パイレーツパスSTEP81以降解放リリース
commit comment
#168 by wada at 2023-05-23T11:28:25; public_html/template/SeasonPass-Home.tpl / old:2126 new:None
c8abf2503fc21513eeb828f3e5c6670cb2a592c8
@kouho
~~証名は自動的に取得するように対応お願いします。~~
1シーズンだけだし大丈夫ですね。