PR

【6/24】復刻プレミアムガチャペンネ #3565

kouho opened this at 2024-06-20T11:24:22; closed

Description

opened by kouho at 2024-06-20T11:24:22; updated 2024-06-25T10:42:00

概要

https://docs.google.com/spreadsheets/d/1K1rj3Ci1r8FDMamNZmWsBijPzfxobJtwllqAKHwrGU8/edit#gid=1735506337

  • [x] ガチャ祭り外対応
  • [x] 画像更新(ガチャ画像 & 転生後魅力紹介)

Comment

#35299 by kouho at 2024-06-20T11:39:09; updated 2024-06-20T11:39:09

@wada
画像が仮入りですがレビューお願いいたします

Comment

#35306 by wada at 2024-06-20T18:28:36; updated 2024-06-20T18:28:36

@kouho
> public_html/img/gacha/433/gacha433_info4.jpg

転生可能です。

Comment

#35307 by wada at 2024-06-20T18:28:57; updated 2024-06-20T18:28:57

@kouho
鋼扇蝶翔のセンリのトレード解禁をお願いします。

Comment

#35308 by wada at 2024-06-20T18:40:48; updated 2024-06-20T18:42:08

@kouho
初回10連時にガチャボタン上画像が初回画像になっていません。

<img src="https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/1718876430815aESQ1ibUdU" width="400">

とくぱんは獲得できました。
<img src="https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/1718876504218tu0TpI8KGc" width="400">

wada add_label at 2024-06-21T13:27:14: コメントあり

Comment

#35319 by kouho at 2024-06-21T14:10:30; updated 2024-06-21T14:10:30

@wada
確認ありがとうございます。以下の指摘点を修正いたしました。
> public_html/img/gacha/433/gacha433_info4.jpg
>
転生可能です。

> 鋼扇蝶翔のセンリのトレード解禁をお願いします。

> 初回10連時にガチャボタン上画像が初回画像になっていません。

Comment

#35331 by wada at 2024-06-24T08:54:00; updated 2024-06-24T08:54:00

@kouho
画像確認OKです。

Comment

#35332 by wada at 2024-06-24T08:58:24; updated 2024-06-24T08:58:24

@kouho
ガチャ結果ページの10連ボタン上画像にも、初回ガチャ10連表示修正対応をお願いします。

Comment

#35333 by kouho at 2024-06-24T13:20:12; updated 2024-06-24T13:20:12

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

> ガチャ結果ページの10連ボタン上画像にも、初回ガチャ10連表示修正対応をお願いします。

ご指摘ありがとうございます。$reprint_fes_gacha_flg = falseと初期化するのを省く意図で3項演算子でbool値を返すようにしていましたが、
そもそも、条件式が直接 bool 値を返すため、初期化は不要であることを理解しました。コードをシンプルにするために修正いたしました。

> 条件がbool値を返すので、3項演算子でbool値を返す必要は低いように感じました。
返り値を明示することは、どのような意図があるんでしょうか?

Comment

#35334 by wada at 2024-06-24T15:27:29; updated 2024-06-24T15:27:29

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

wada delete_label at 2024-06-24T15:27:32: コメントあり
wada add_label at 2024-06-24T15:27:33: 和田 LGTM
kouho merge at 2024-06-25T10:42:00: 【6/24】復刻プレミアムガチャペンネ
kouho close at 2024-06-25T10:42:00: Close

commit comment

#253 by wada at 2024-06-24T09:02:19; lib/public/Bar-Gacha.inc / old:None new:20
d92fef7167caf36d2fa81ce557e2fa3ee0de1bf5

@kouho
条件がbool値を返すので、3項演算子でbool値を返す必要は低いように感じました。
返り値を明示することは、どのような意図があるんでしょうか?

Commit range

basec86b173179870c19efd348604c817faee75b1248
head5f0fb19711b63fd344f697f9a530a632bbd50919
base branchjolly/jr-web:master
head branchjolly/jr-web:gacha_re_0626

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed24 filesView patch
statusfile
modifiedlib/public/Bar-Gacha.inc
modifiedlib/public/Bar-GachaResult.inc
addedpublic_html/img/gacha/433/banner_gacha433.jpg
addedpublic_html/img/gacha/433/banner_gacha433_2.jpg
addedpublic_html/img/gacha/433/etc_lineup.jpg
addedpublic_html/img/gacha/433/gacha433.jpg
addedpublic_html/img/gacha/433/gacha433_2.jpg
addedpublic_html/img/gacha/433/gacha433_info.jpg
addedpublic_html/img/gacha/433/gacha433_info2.jpg
addedpublic_html/img/gacha/433/gacha433_info3.jpg
addedpublic_html/img/gacha/433/gacha433_info4.jpg
addedpublic_html/img/gacha/433/gacha433_title.jpg
addedpublic_html/img/gacha/433/gacha433_top.jpg
addedpublic_html/img/info_jrye/CharmInfo/card/1156/info1.jpg
addedpublic_html/img/info_jrye/CharmInfo/card/1156/info2.jpg
addedpublic_html/img/info_jrye/CharmInfo/card/1156/info2_2.jpg
addedpublic_html/img/info_jrye/CharmInfo/card/1156/info3.jpg
addedpublic_html/img/info_jrye/GachaImage/433/info2.jpg
addedpublic_html/img/info_jrye/GachaImage/433/info3.jpg
addedpublic_html/img/info_jrye/GachaImage/reprint/info5_1014.jpg
addedpublic_html/img/info_jrye/GachaImage/reprint/info5_1094.jpg
addedpublic_html/img/info_jrye/GachaImage/reprint/info5_1160.jpg
modifiedpublic_html/template/Bar-GachaReprint3.tpl
modifiedpublic_html/template/Bar-GachaResultReprint3.tpl
Patch body is stored as a separate file to keep this page lightweight.

Review comment

#253 by wada at 2024-06-24T09:02:19; lib/public/Bar-Gacha.inc / old:None new:20
d92fef7167caf36d2fa81ce557e2fa3ee0de1bf5

@kouho
条件がbool値を返すので、3項演算子でbool値を返す必要は低いように感じました。
返り値を明示することは、どのような意図があるんでしょうか?