PR

【7/16】海域ページの育成パックバナー #3597

wada opened this at 2024-07-16T09:49:59; closed

Description

opened by wada at 2024-07-16T09:49:59; updated 2024-07-16T17:06:50

概要

育成パック販売中にバナー表示
・販売終了、全購入で非表示
販売カードの所持状況によってテキストを変更

テキスト変化(優先度順)

  • 転生後最大覚醒
  • 転生後スキルスロットあり
  • 転生後所持
  • 転生前所持
  • 未所持

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

TODO

カードによってバナー表示/非表示を切り替える(未定)

Comment

#35622 by wada at 2024-07-16T09:51:05; updated 2024-07-16T09:51:28

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

当日のレビュー依頼なので、締切過ぎても大丈夫ですが、
海域イベントで使用率の高いニャッカのパック販売中にはバナー掲載したいです。

kouho add_label at 2024-07-16T13:13:24: コメントあり

Comment

#35633 by kouho at 2024-07-16T13:25:29; updated 2024-07-16T13:25:29

@wada
TODOに入っている以下の箇所ですが、ニャッカ育成パック限定もしくは以外の処理が見つかりませんのでパックに応じてテキストを修正するか特定の育成パックのみという判定を追加できませんか?

>カードによってバナー表示/非表示を切り替える(未定)

また、新ドリンクのあるなしの判定の取り方が思いつきませんが、新ドリンクが仕様上ない場合は別途対応でしょうか?

Comment

#35635 by wada at 2024-07-16T13:39:21; updated 2024-07-16T13:39:21

@kouho
技玉条件修正しました。

出発点が海域デッキ使用上位のニャッカ、ワドルのバック販売をアピールしたいなので、2つのパックを基準としてます。
TODOに関しては、おいおい対応します。

> TODOに入っている以下の箇所ですが、ニャッカ育成パック限定もしくは以外の処理が見つかりませんのでパックに応じてテキストを修正するか特定の育成パックのみという判定を追加できませんか?
カードによってバナー表示/非表示を切り替える(未定)
また、新ドリンクのあるなしの判定の取り方が思いつきませんが、新ドリンクが仕様上ない場合は別途対応でしょうか?

Comment

#35636 by kouho at 2024-07-16T13:54:38; updated 2024-07-16T13:54:38

@wada
ありがとうございます。技玉条件修正確認しました。

> 出発点が海域デッキ使用上位のニャッカ、ワドルのバック販売をアピールしたいなので、2つのパックを基準としてます。
TODOに関しては、おいおい対応します。

了解です。ワドルに関してはニャッカパック終了の7/21までに対応するということでしょうか?

Comment

#35637 by wada at 2024-07-16T15:29:49; updated 2024-07-16T15:29:49

@kouho
すいません、テキストがニャッカ固定になっているから、card_name参照にしてほしいと言うことですね。
こちら対応します。

Comment

#35638 by wada at 2024-07-16T16:38:34; updated 2024-07-16T16:38:34

@kouho
カード名を動的に取得するように変更しました。

Comment

#35640 by kouho at 2024-07-16T16:46:52; updated 2024-07-16T16:46:52

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

kouho add_label at 2024-07-16T16:47:00: 皇甫 LGTM
wada merge at 2024-07-16T17:06:47: 【7/16】海域ページの育成パックバナー
wada close at 2024-07-16T17:06:47: Close
wada delete_branch at 2024-07-16T17:06:50: training_pack_banner

commit comment

#255 by kouho at 2024-07-16T11:43:19; lib/models/Training_pack_banner.php / old:None new:80
68a2ee04e7020433f5884b659cc2100ba42aa692

@wada
スキルスロットに技玉ありの場合 $card['max_lv'] != $card['max_max_lv'] の条件も加えていただきたいです。

Commit range

base4b1ec324a51378f0d24810336ad40d423c58adf2
headff3804f63c6b860de9a7c5a8120475bddfb44994
base branchjolly/jr-web:master
head branchjolly/jr-web:training_pack_banner

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/SubFunc.inc
addedlib/models/Training_pack_banner.php
modifiedlib/public/Battle-ResultOceanArea.inc
modifiedlib/public/Battle-ResultOceanAreaTeam.inc
modifiedlib/public/OceanArea-Default.inc
modifiedpublic_html/template/Battle-ResultOceanArea.tpl
modifiedpublic_html/template/Battle-ResultOceanAreaTeam.tpl
modifiedpublic_html/template/OceanArea-Default.tpl
modifiedpublic_html/template/Shop.tpl
Patch body is stored as a separate file to keep this page lightweight.

Review comment

#255 by kouho at 2024-07-16T11:43:19; lib/models/Training_pack_banner.php / old:None new:80
68a2ee04e7020433f5884b659cc2100ba42aa692

@wada
スキルスロットに技玉ありの場合 $card['max_lv'] != $card['max_max_lv'] の条件も加えていただきたいです。