PR

【7/21 16:00まで】レイド概要お知らせオススメカード対応 #2111

kouho opened this at 2022-07-21T14:00:42; closed

Description

opened by kouho at 2022-07-21T14:00:42; updated 2022-07-21T15:09:27

概要

レイド概要のオススメカードをパーツ化して
ガチャ開催期間のみガチャ誘導関連は表示するようにする。

開発のレイド概要1588をローカルに持ってきて確認してください。

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

ガチャの開催と日開催を切り替えて表示確認

TODO

確認できしだい本番に反映予定

Comment

#21007 by kouho at 2022-07-21T14:00:50; updated 2022-07-21T14:00:50

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

Comment

#21010 by kouho at 2022-07-21T14:38:20; updated 2022-07-21T14:38:20

@wada
確認ありがとうございます。
issetの削除と$_partsへの統一を行いました。

Comment

#21012 by wada at 2022-07-21T14:57:16; updated 2022-07-21T14:57:16

@kouho
ありがとうございます。確認しました。
達成報酬側のフラグ対応を忘れていたので、このブランチに追加しても良いですか?

Comment

#21013 by kouho at 2022-07-21T14:57:40; updated 2022-07-21T14:57:40

@wada
大丈夫です。お願いいたします。

Comment

#21014 by wada at 2022-07-21T15:03:33; updated 2022-07-21T15:03:33

@kouho
追加しました。
一緒にテンプレートのエラーも対応しました。

wada add_label at 2022-07-21T15:06:36: 和田 LGTM

Comment

#21018 by kouho at 2022-07-21T15:08:15; updated 2022-07-21T15:08:15

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

kouho merge at 2022-07-21T15:09:27: 【7/21 16:00まで】レイド概要お知らせオススメカード対応
kouho close at 2022-07-21T15:09:27: Close

commit comment

#129 by wada at 2022-07-21T14:22:28; public_html/template/parts/parts_raid_info_template.tpl / old:None new:3
30028516243160e98b591deb24033cc8394633a2

@kouho
isset!empty を同時に使用するのであれば、!emptyissetを内包しているので、!emptyだけで良いと思います。

commit comment

#130 by wada at 2022-07-21T14:30:00; public_html/template/parts/parts_raid_info_template.tpl / old:None new:141
230f85d6d29f61254c9076f002132bde2bea2301

@kouho
$_parts$parts が共存しているので、どちらかに統一をお願いします。

Commit range

base38e060363d60e7bda5bc88a67370e2310ca06453
head230f85d6d29f61254c9076f002132bde2bea2301
base branchjolly/jr-web:master
head branchjolly/jr-web:raid_recommend_card

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed8 filesView patch
statusfile
modifiedlib/ajax/default_info.inc
modifiedlib/conf/class/RaidMonsterConfig.class.php
modifiedlib/models/Raid_event.php
addedpublic_html/img/info_jrye/RaidEventImg/guide/guide_496.jpg
addedpublic_html/img/info_jrye/RaidEventImg/guide/guide_604.jpg
addedpublic_html/img/info_jrye/RaidEventImg/guide/guide_gacha.png
addedpublic_html/img/info_jrye/RaidEventImg/guide/guide_header.jpg
modifiedpublic_html/template/parts/parts_raid_info_template.tpl
Patch body is stored as a separate file to keep this page lightweight.

Review comment

#129 by wada at 2022-07-21T14:22:28; public_html/template/parts/parts_raid_info_template.tpl / old:None new:3
30028516243160e98b591deb24033cc8394633a2

@kouho
isset!empty を同時に使用するのであれば、!emptyissetを内包しているので、!emptyだけで良いと思います。

Review comment

#130 by wada at 2022-07-21T14:30:00; public_html/template/parts/parts_raid_info_template.tpl / old:None new:141
230f85d6d29f61254c9076f002132bde2bea2301

@kouho
$_parts$parts が共存しているので、どちらかに統一をお願いします。