PR

【4/30】復刻プレミアムガチャ「GWSP」 #751

iwamoto opened this at 2021-04-22T16:42:01; closed

Description

opened by iwamoto at 2021-04-22T16:42:01; updated 2024-06-05T11:16:22

【4/30】復刻プレミアムガチャ「GWSP」 #725

各ブランチ
webブランチ : ``gacha_0430``
dbブランチ : ``gacha_0430``

db変更箇所 `` https://git.dev.d-es.co.jp/gitbucket/jolly/jrye-db/commits/gacha_0430 ``

master_gacha
master_gacha_item
master_item_album
master_card_skill

仕様

  • [ ] ランダムガチャ
  • [ ] 10連購入5回まではおまけ付き
 - [ ] ガチャ毎におまけはリセットされる
 - [ ] 1日1回100コイン
 - [ ] 各ガチャの限定カード専用の強化アイテム入り

ハンターナイフ【氷眼のロロノア】(id:1135)
黒曜石の指輪【鉄掟のバーソロミュー 】 (1072)
香水【疾風のラカム】(1007)


ガチャスケジュール
ロロノア    : ```4/30 12:00 ~ 5/1 23:59```
バーソロミュー : ```5/2    00:00 ~ 5/3 23:59```
ラカム     : ```5/4    00:00 ~ 5/5 23:59```
iwamoto add_label at 2021-04-22T16:42:06: デベロップへマージ
iwamoto add_label at 2021-04-22T16:42:08: イベント
iwamoto delete_label at 2021-04-22T16:42:09: イベント
iwamoto change_milestone at 2021-04-22T16:42:19: No milestone:review_20210426
iwamoto assign at 2021-04-22T16:42:23: Not assigned:iwamoto
iwamoto add_label at 2021-04-22T19:57:20: レビュー待ち

Comment

#10502 by iwamoto at 2021-04-22T20:01:54; updated 2021-04-22T20:01:54

@wada @kouho
諸々対応完了しましたので、レビューをお願い致します。

通常と異なるのは下記になります

・10連購入時のおまけ(5回まであり、ガチャ毎にリセット)
・限定カードの強化アイテムがある
iwamoto change_title at 2021-04-22T20:02:14: [WIP]【4/30】復刻プレミアムガチャ「GWSP」 【4/30】復刻プレミアムガチャ「GWSP」

Comment

#10506 by wada at 2021-04-23T10:47:33; updated 2021-04-23T10:47:33

@iwamoto
csv/master_gacha.csv閃の軌跡第2弾限定ボックスがstart_timeが0になっているので、できれば日付入れておいて欲しいです。

Comment

#10507 by wada at 2021-04-23T10:52:22; updated 2021-04-23T10:52:22

違う、start_timeのカラムが消えてますね。

Comment

#10508 by iwamoto at 2021-04-23T10:59:41; updated 2021-04-23T10:59:41

@wada
そちら確認し、修正を行いました

Comment

#10510 by kouho at 2021-04-23T12:18:12; updated 2021-04-23T12:20:39

@iwamoto
!スクリーンショット 2021-04-23 12

ガチャ酒場での一覧ページの販売終了まで残り:○日
という表示と各ガチャ毎のページの残り:△日
の表示が一致しておりませんでして、ガチャ一覧は define_yの復刻ガチャ販売期間を参照して、
各ガチャ毎のページではmaster_gachaを参照しているからと思いますが、こちら問題ないでしょうか?

Comment

#10513 by kouho at 2021-04-23T14:15:54; updated 2021-04-23T14:15:54

@iwamoto
ガチャについては期間の表記以外問題ないかと思います。

お酒については今回初出の機能なのでしょうか?カード詳細ページのカードを強化するの折り畳みリンクがあるのですが、
こちらの中身が表示されていませんでして対応が必要かと思いました。

Comment

#10515 by iwamoto at 2021-04-23T14:48:14; updated 2021-04-23T14:48:14

@kouho
カード詳細ページについては修正対応を行いました。

Comment

#10516 by iwamoto at 2021-04-23T15:20:54; updated 2021-04-23T15:20:54

@kouho
ガチャページの期間表示に関しては問題ないと考えています。

ガチャ酒場の一覧に表示される時間は「GW復刻プレミアムガチャ(復刻ガチャ祭り)」終了までの時間であって
各ガチャページに表示される時間は「GW復刻プレミアムガチャ(ロロノア/バーソロミュー/ラカム)」が切り替わる(終了)までの残り時間のため

Comment

#10519 by wada at 2021-04-23T15:48:17; updated 2021-04-23T15:48:17

このガチャページに1~5回目のボーナス画像追加となると、ボタンまでがちょっと遠くなるなぁ

Comment

#10523 by kouho at 2021-04-23T18:11:34; updated 2021-04-23T18:11:34

@iwamoto
すみません、どうやら僕の勘違いで一個ずつガチャを表示させたら期間表示は問題なかったです。
ただ、今後複数の復刻ガチャを同時開催した場合にユーザーさんが勘違いしてしまう可能性がありますので一覧ページのガチャ期間の表示をそれぞれのガチャの終了期間を参照する形に修正していただければと思います。

Comment

#10524 by kouho at 2021-04-23T18:24:28; updated 2021-04-23T18:24:28

@iwamoto
カード詳細画面の修正について確認しました。
強化アイテム・ドリンクがあるよ の部分を元々の 強化アイテムがあるよ に修正しておいた方が良いかと思います。
他は大丈夫かと思います。

Comment

#10531 by iwamoto at 2021-04-23T19:47:55; updated 2021-04-23T19:47:55

@kouho
カード詳細画面については修正対応を行いました。

また、ガチャの期間表示の指摘に関しましては
ガチャ一覧ページの販売終了までの時間と各ガチャページの残り時間については同じgacha_master.end_timeから取得しています
なので


このため、先の私のガチャページの期間表示については誤りでした
こちらは私が指摘頂いた内容より、

Comment

#10532 by iwamoto at 2021-04-23T19:50:40; updated 2021-04-23T19:50:40

@wada
おまけ画像をなくされますか?
また、注意事項などのように折りたたんでしまうのも1つの案かとは思います。

Comment

#10536 by iwamoto at 2021-04-26T11:51:09; updated 2021-04-26T14:04:16

@wada
今朝話されてました、下記3点追加対応完了しました。

・TOP画像とガチャインフォ画像の間の背景色変更
・強化アイテム欄の表示調整
・ガチャするボタンを追加

また、指摘頂きましたtop画像の余白についてはこれから対応していきます。
->(追記)こちらも対応完了しました。

Comment

#10551 by wada at 2021-04-26T15:19:00; updated 2021-04-26T15:19:00

@iwamoto
対応ありがとうございます。ページの表示周りについて対応お願いします。

<img src="https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/1619417498677MgxyR8UYkF" width="200">
専用アイテムページは閉じているので、酒使用ページのリンクを非表示にしてください。

<img src="https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/1619417629617RSqFEqOHIG" width="200">
レジェンドを最大覚醒すると、限界突破薬のリンクが表示されるので、非表示にしてください。
(フラグ管理漏れてました)

<img src="https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/1619417744878tr7WUY9rAt" width="200">
酒使用確認ページでフッターメニュー(右下)が開けないので、タグの閉じ忘れがあるかもしれません。

Comment

#10552 by iwamoto at 2021-04-26T16:06:12; updated 2021-04-26T16:06:12

@wada
上記3点対応しました

Comment

#10560 by wada at 2021-04-26T19:37:38; updated 2021-04-26T19:37:38

@iwamoto
確認しました。OKです。

wada add_label at 2021-04-26T19:37:42: 和田 LGTM

Comment

#10563 by wada at 2021-04-26T19:47:58; updated 2021-04-26T19:47:58

@iwamoto
細かいですが、スキル【ボニー&メアリー】加勢の説明に公開されていないカードがあるので、削除をお願いします。
・海の舞姫メアリー
・桜花のボニー

Comment

#10569 by iwamoto at 2021-04-27T12:43:30; updated 2021-04-27T12:43:30

@wada
削除対応しました

Comment

#10577 by kouho at 2021-04-27T15:08:56; updated 2021-04-27T15:08:56

@iwamoto
確認しました。こちら問題ないかと思います。

kouho add_label at 2021-04-27T15:09:03: 皇甫 LGTM
iwamoto merge at 2021-04-27T16:46:38: 【4/30】復刻プレミアムガチャ「GWSP」
iwamoto close at 2021-04-27T16:46:38: Close

commit comment

#51 by wada at 2021-04-26T14:36:15; public_html/template/Bar-GachaReprint3.tpl / old:318 new:319
fe06a48f93fd7e7f67a8079f8045fb79197bc9f2

@iwamoto
ヤマダでは復刻ガチャと復刻プレミアムガチャは別仕様なので、
復刻ガチャ祭りって?GW復刻プレミアムガチャ(今回のキャンペーン名)って?に変更お願いします。
復刻ガチャ復刻プレミアムガチャに変更お願いします。
注意事項はすぐ目に入るようにしたいので、アコーディオンメニューではなくそのまま表示をお願いします。

Commit range

base2778a2dd1a0caeaeac7774305df208e9bc9c22c8
headef8a6d013fcf965ab8a48864b00167459e4aaeab
base branchjolly/jr-web:develop
head branchjolly/jr-web:gacha_0430

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed81 filesView patch
statusfile
modifiedlib/conf/class/GachaConfig.class.php
addedlib/conf/class/ReprintFesGachaConfig.class.php
modifiedlib/conf/define_y.inc
modifiedlib/models/Gacha_payment.php
addedlib/models/gacha/Reprint_fes.php
modifiedlib/public/Bar-Gacha.inc
modifiedlib/public/Bar-GachaResult.inc
modifiedpublic_html/img/gacha/137/banner_gacha137.jpg
modifiedpublic_html/img/gacha/137/banner_gacha137_2.jpg
modifiedpublic_html/img/gacha/137/gacha137.jpg
addedpublic_html/img/gacha/137/gacha137_10bonus.jpg
modifiedpublic_html/img/gacha/137/gacha137_2.jpg
deletedpublic_html/img/gacha/137/gacha137_bg.jpg
addedpublic_html/img/gacha/137/gacha137_card_info18.jpg
addedpublic_html/img/gacha/137/gacha137_card_info23.jpg
deletedpublic_html/img/gacha/137/gacha137_card_info391.jpg
deletedpublic_html/img/gacha/137/gacha137_card_info392.jpg
deletedpublic_html/img/gacha/137/gacha137_card_info393.jpg
deletedpublic_html/img/gacha/137/gacha137_card_info394.jpg
deletedpublic_html/img/gacha/137/gacha137_card_info395.jpg
addedpublic_html/img/gacha/137/gacha137_card_info71.jpg
addedpublic_html/img/gacha/137/gacha137_card_info93.jpg
modifiedpublic_html/img/gacha/137/gacha137_info.jpg
modifiedpublic_html/img/gacha/137/gacha137_info2.jpg
modifiedpublic_html/img/gacha/137/gacha137_info3.jpg
modifiedpublic_html/img/gacha/137/gacha137_info4.jpg
deletedpublic_html/img/gacha/137/gacha137_info5.jpg
modifiedpublic_html/img/gacha/137/gacha137_title.jpg
addedpublic_html/img/gacha/137/gacha137_top.jpg
addedpublic_html/img/gacha/137/updateinfo1.jpg
addedpublic_html/img/gacha/169/banner_gacha169.jpg
addedpublic_html/img/gacha/169/banner_gacha169_2.jpg
deletedpublic_html/img/gacha/169/banner_gacha169_reprint.jpg
modifiedpublic_html/img/gacha/169/gacha169.jpg
addedpublic_html/img/gacha/169/gacha169_10bonus.jpg
modifiedpublic_html/img/gacha/169/gacha169_2.jpg
deletedpublic_html/img/gacha/169/gacha169_bg.jpg
modifiedpublic_html/img/gacha/169/gacha169_card_info119.jpg
modifiedpublic_html/img/gacha/169/gacha169_card_info17.jpg
modifiedpublic_html/img/gacha/169/gacha169_card_info35.jpg
modifiedpublic_html/img/gacha/169/gacha169_card_info90.jpg
modifiedpublic_html/img/gacha/169/gacha169_info.jpg
modifiedpublic_html/img/gacha/169/gacha169_info2.jpg
modifiedpublic_html/img/gacha/169/gacha169_info3.jpg
addedpublic_html/img/gacha/169/gacha169_info4.jpg
modifiedpublic_html/img/gacha/169/gacha169_title.jpg
addedpublic_html/img/gacha/169/gacha169_top.jpg
addedpublic_html/img/gacha/169/updateinfo1.jpg
modifiedpublic_html/img/gacha/185/banner_gacha185.jpg
modifiedpublic_html/img/gacha/185/banner_gacha185_2.jpg
modifiedpublic_html/img/gacha/185/gacha185.jpg
addedpublic_html/img/gacha/185/gacha185_10bonus.jpg
modifiedpublic_html/img/gacha/185/gacha185_2.jpg
addedpublic_html/img/gacha/185/gacha185_card_info25.jpg
addedpublic_html/img/gacha/185/gacha185_card_info50.jpg
deletedpublic_html/img/gacha/185/gacha185_card_info546.jpg
deletedpublic_html/img/gacha/185/gacha185_card_info547.jpg
deletedpublic_html/img/gacha/185/gacha185_card_info548.jpg
deletedpublic_html/img/gacha/185/gacha185_card_info549.jpg
addedpublic_html/img/gacha/185/gacha185_card_info66.jpg
addedpublic_html/img/gacha/185/gacha185_card_info80.jpg
modifiedpublic_html/img/gacha/185/gacha185_info.jpg
modifiedpublic_html/img/gacha/185/gacha185_info2.jpg
modifiedpublic_html/img/gacha/185/gacha185_info3.jpg
modifiedpublic_html/img/gacha/185/gacha185_info4.jpg
modifiedpublic_html/img/gacha/185/gacha185_title.jpg
addedpublic_html/img/gacha/185/gacha185_top.jpg
addedpublic_html/img/gacha/185/updateinfo1.jpg
modifiedpublic_html/img/gacha/gacha_coin_reprint2.png
modifiedpublic_html/img/gacha/gacha_coin_reprint3.png
modifiedpublic_html/img/gacha/gacha_coin_reprint4.png
modifiedpublic_html/img/gacha/gacha_coin_reprint5.png
modifiedpublic_html/template/Bar-Default.tpl
modifiedpublic_html/template/Bar-GachaReprint3.tpl
modifiedpublic_html/template/Bar-GachaResultReprint3.tpl
modifiedpublic_html/template/Card-Detail.tpl
modifiedpublic_html/template/Card-StatusUp.tpl
modifiedpublic_html/template/Card-StatusUpCheck.tpl
modifiedpublic_html/template/Card-StatusUpSelect.tpl
modifiedpublic_html/template/Default.tpl
modifiedpublic_html/template/Main.tpl
Patch body is stored as a separate file to keep this page lightweight.

Review comment

#51 by wada at 2021-04-26T14:36:15; public_html/template/Bar-GachaReprint3.tpl / old:318 new:319
fe06a48f93fd7e7f67a8079f8045fb79197bc9f2

@iwamoto
ヤマダでは復刻ガチャと復刻プレミアムガチャは別仕様なので、
復刻ガチャ祭りって?GW復刻プレミアムガチャ(今回のキャンペーン名)って?に変更お願いします。
復刻ガチャ復刻プレミアムガチャに変更お願いします。
注意事項はすぐ目に入るようにしたいので、アコーディオンメニューではなくそのまま表示をお願いします。