PR

【12/14】ダイヤ交換所 #1374

wada opened this at 2021-12-10T11:26:19; closed

Description

opened by wada at 2021-12-10T11:26:19; updated 2026-04-06T10:36:17

概要

【12/24】ダイヤ交換所 #1331

Comment

#15777 by wada at 2021-12-10T11:26:32; updated 2021-12-10T11:26:32

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

wada add_label at 2021-12-10T11:26:42: デベロップへマージ
wada add_label at 2021-12-10T11:26:46: レビュー待ち
wada delete_label at 2021-12-10T11:26:47: イベント

Comment

#15793 by kouho at 2021-12-10T18:37:55; updated 2021-12-10T18:37:55

@wada
動作は問題ないと思いますが2点確認です。
1. イベントの終了日時について
仕様書では23:59:00となっており設定は23:59:59でした。設定側が正で大丈夫でしょうか?

2. configの設定について
lib/conf/class/AccessoryEventConfig.phpはあえて使わずdefine_yにまとめているのでしょうか?
本家では2020年 1月に作られたファイルでしたので使わないなら削除して、
使うならdefine_yを本家と同じ書き方にしたらどうかと思いました。

Comment

#15796 by wada at 2021-12-13T10:35:42; updated 2021-12-13T10:35:42

@kouho
終了日時は23:59:59で大丈夫です。
configの設定は特に理由はないですが、configにまとめた方が良いですね。
動作に影響はないので、今回は対応せず次回開催までに対応します。

Comment

#15798 by kouho at 2021-12-13T10:38:16; updated 2021-12-13T10:38:16

@wada
期間とconfig設定についてかしこまりました。

kouho add_label at 2021-12-13T10:38:18: 皇甫 LGTM

Comment

#15804 by iwamoto at 2021-12-13T11:38:30; updated 2021-12-13T11:38:30

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

iwamoto add_label at 2021-12-13T11:38:36: 岩本 LGTM
wada merge at 2021-12-14T15:12:43: 【12/14】ダイヤ交換所
wada close at 2021-12-14T15:12:43: Close
wada refer at 2021-12-16T10:48:09: 1386:【12/16】リリース

Commit range

base6593089dad9ee3bd401cd062749b08fc0ef6423b
head2e1309d8411477d6ff1f55e6b076f7dbe25ea5c0
base branchjolly/jr-web:develop
head branchjolly/jr-web:acce_gacha

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed3 filesView patch
statusfile
modifiedlib/conf/define_y.inc
addedpublic_html/img/info_event/info_event_325_1.jpg
addedpublic_html/img/info_event/info_event_325_2.jpg
Patch body is stored as a separate file to keep this page lightweight.