PR

【5/23】ロキの宝箱がリリースされていない時STEP81以降の対応 #2801

kouho opened this at 2023-05-22T18:43:14; closed

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

  • [x] 問題なく動作するか

TODO

  • [ ] 既に80以上の人の対応

Comment

#27673 by kouho at 2023-05-22T18:43:34; updated 2023-05-22T18:43:34

@wada
確認お願いいたします

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.phpget_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シーズンだけだし大丈夫ですね。

Commit range

basebd532c628183330dd9459e228963d9a029022a2f
headc8abf2503fc21513eeb828f3e5c6670cb2a592c8
base branchjolly/jr-web:develop
head branchjolly/jr-web:step_upper_limit

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed9 filesView patch
statusfile
modifiedlib/ajax/seasonPass_clearDailyQuest.inc
modifiedlib/ajax/seasonPass_clearSeasonQuest.inc
modifiedlib/class/season_pass/SeasonPass.class.php
modifiedlib/conf/class/SeasonPassConfig.class.php
modifiedlib/models/Shop_payment.php
modifiedlib/models/shop/Season_pass.php
addedpublic_html/img/season_pass/loki/nyakka_congra4.png
addedpublic_html/img/season_pass/loki/step_banner_season1.png
modifiedpublic_html/template/SeasonPass-Home.tpl
Patch body is stored as a separate file to keep this page lightweight.

Review comment

#168 by wada at 2023-05-23T11:28:25; public_html/template/SeasonPass-Home.tpl / old:2126 new:None
c8abf2503fc21513eeb828f3e5c6670cb2a592c8

@kouho
~~証名は自動的に取得するように対応お願いします。~~
1シーズンだけだし大丈夫ですね。