PR

【6/3】4周年カウントダウンCP #3516

kouho opened this at 2024-05-29T15:07:57; closed

Description

opened by kouho at 2024-05-29T15:07:57; updated 2024-06-01T20:06:59

概要

https://docs.google.com/spreadsheets/d/1wM5UqwnD2fns2c4gks6NSCwBPokPhFFKz3erQ_WManE/edit#gid=442703856

カウントダウンCPのみの対応

TODO

  • [x] 画像
kouho add_label at 2024-05-29T17:26:05: レビュー待ち
kouho change_title at 2024-05-29T17:26:11: [WIP]【6/3】4周年カウントダウンCP 【6/3】4周年カウントダウンCP

Comment

#34790 by kouho at 2024-05-29T17:26:29; updated 2024-05-29T17:26:29

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

Comment

#34800 by wada at 2024-05-30T16:19:54; updated 2024-05-30T16:19:54

@kouho
期限切れ対応時にラインナップ表示判定処理が消えていたので追加対応お願いします。
3ef011f9d4229c49653ce49ad4406290773c38ab
lib/ajax/item_select.inc 203行目

Comment

#34801 by wada at 2024-05-30T16:21:12; updated 2024-05-30T16:21:12

@kouho
アイテムの期限付き対応は、バッチ処理を利用せず手動で実行ということであっていますか?

wada add_label at 2024-05-30T16:26:34: コメントあり

Comment

#34806 by kouho at 2024-05-31T12:18:39; updated 2024-05-31T12:18:39

@wada
確認ありがとうございます。以下修正しました。
コインガチャチケットスタンプは1個に修正しています。
また、SQLは手動の直接実行で行こうと思っていました。

> 期限切れ対応時にラインナップ表示判定処理が消えていたので追加対応お願いします。
3ef011f
lib/ajax/item_select.inc 203行目

> コインガチャチケットスタンプのみ3個ですか?

Comment

#34807 by wada at 2024-05-31T15:27:50; updated 2024-05-31T15:30:08

@kouho
修正箇所確認しました。

前回の期限付きアイテム「【紅白】2024年おみくじ」と違う点について質問です。

  • アイテム名に期限を追記しますか?

アイテムリストで期限を確認できるように

  • 期限はoo:59表記で統一しなくても良いですか?
  • 期限切れ時にGOLD変換しますか?

おみくじは、cronで午前3時に自動GOLD変換

  • 期限切れ時に使用導線を非表示にしますか?

おみくじ実装時は、config['limit_time']で実現

Comment

#34812 by kouho at 2024-05-31T17:24:56; updated 2024-05-31T17:24:56

@wada
確認ありがとうございます。「【紅白】2024年おみくじ」のことを失念していました。
変更する必要はないため、合わせるように以下の修正をいたしました。

  • アイテム名の変更 = 期限を追記 + BOXに

手動での回収が念頭にありましたので12:00表記をしておりましたが、スクリプトで回収できるためおみくじと同じ日付表示のみにしています。
追記の際にアイテム名が長くなり表示が若干崩れるためボックスをBOXに修正しました。

  • limit_timeの設定

config['limit_time']の設定を追加

Comment

#34814 by wada at 2024-05-31T17:46:33; updated 2024-05-31T17:46:33

@kouho
修正内容確認しました。
表示で気になった点だけ確認お願いします。

<img src="https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/1717145070817pnLdVCx1Sn" width="400">
端末によっては、ラインナップボタンの文字が途中で切れてしまいます。

<img src="https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/17171451173125nmwnpLA6N" width="400">
使用可能の後のが1つ余分に付いています。

Comment

#34815 by kouho at 2024-05-31T18:09:51; updated 2024-05-31T18:09:51

@wada
確認ありがとうございます。それぞれ修正しました。
他のボックスも含まれますが、ラインナップボタンについては改行をいれてみました。

Comment

#34822 by wada at 2024-05-31T18:29:34; updated 2024-05-31T18:29:34

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

wada delete_label at 2024-05-31T18:29:36: コメントあり
wada add_label at 2024-05-31T18:29:38: 和田 LGTM
kouho merge at 2024-06-01T20:06:59: 【6/3】4周年カウントダウンCP
kouho close at 2024-06-01T20:06:59: Close

commit comment

#234 by wada at 2024-05-30T16:22:48; lib/conf/class/RandomBoxConfig.class.php / old:None new:190
74db44142494aa9e2d359228669c485454269ec7

@kouho
コインガチャチケットスタンプのみ3個ですか?

Commit range

base3157abd2b3ce1913e93a3fe6911f549856cf96c3
head1a337998b0f1205ea8af4c2f342bc0e7d3c10e19
base branchjolly/jr-web:master
head branchjolly/jr-web:cd_login_bonus_4th_anniv

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed33 filesView patch
statusfile
modifiedlib/ajax/item_select.inc
modifiedlib/conf/class/RandomBoxConfig.class.php
modifiedlib/conf/define_y.inc
addedpublic_html/img/info_event/info_event_1006_1.jpg
addedpublic_html/img/info_event/info_event_1006_2.jpg
addedpublic_html/img/info_event/info_event_1007_1.jpg
addedpublic_html/img/info_event/info_event_1007_2.jpg
addedpublic_html/img/info_event/info_event_1008_1.jpg
addedpublic_html/img/info_event/info_event_1008_2.jpg
addedpublic_html/img/info_event/info_event_1009_1.jpg
addedpublic_html/img/info_event/info_event_1009_2.jpg
addedpublic_html/img/info_event/info_event_1010_1.jpg
addedpublic_html/img/info_event/info_event_1010_2.jpg
addedpublic_html/img/info_event/info_event_1011_1.jpg
addedpublic_html/img/info_event/info_event_1011_2.jpg
addedpublic_html/img/info_event/info_event_1012_1.jpg
addedpublic_html/img/info_event/info_event_1012_2.jpg
addedpublic_html/img/info_event/info_event_1013_1.jpg
addedpublic_html/img/info_event/info_event_1013_2.jpg
addedpublic_html/img/info_event/info_event_1014_1.jpg
addedpublic_html/img/info_event/info_event_1014_2.jpg
addedpublic_html/img/info_event/info_event_1015_1.jpg
addedpublic_html/img/info_event/info_event_1015_2.jpg
addedpublic_html/img/info_event/info_event_1016_1.jpg
addedpublic_html/img/info_event/info_event_1016_2.jpg
addedpublic_html/img/info_jrye/3293/info1.jpg
addedpublic_html/img/info_jrye/3293/info2.jpg
addedpublic_html/img/item/120/304.jpg
addedpublic_html/img/item/150/304.jpg
addedpublic_html/img/item/58/304.jpg
addedpublic_html/img/item/60/304.jpg
addedpublic_html/img/item/70/304.jpg
modifiedpublic_html/template/Item.tpl
Patch body is stored as a separate file to keep this page lightweight.

Review comment

#234 by wada at 2024-05-30T16:22:48; lib/conf/class/RandomBoxConfig.class.php / old:None new:190
74db44142494aa9e2d359228669c485454269ec7

@kouho
コインガチャチケットスタンプのみ3個ですか?