PR

【12/4】天下無双の船ボーナスとスケジュールの整合性チェック追加 #3254

kouho opened this at 2023-12-01T15:28:11; closed

Description

opened by kouho at 2023-12-01T15:28:11; updated 2023-12-11T17:08:33

概要

天下無双の告知内で天下無双の開催期間と船ボーナスの設定日数の整合性チェックをしています。
不整合の場合はエラーを投げてみています。

また合わせて告知内に表示する船ボーナスの日付をprod_startを参照するように修正しました。

Comment

#32192 by kouho at 2023-12-01T15:29:12; updated 2023-12-01T15:29:12

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

Comment

#32238 by wada at 2023-12-04T15:58:32; updated 2023-12-04T15:58:32

@kouho
間違えていても正しく表示するようにできますか?

wada add_label at 2023-12-04T15:58:34: コメントあり

Comment

#32242 by kouho at 2023-12-04T16:51:08; updated 2023-12-04T16:51:08

@wada
イベント期間の設定が船ボーナスの設定期間より短い場合に期間設定に合わせて表示するように修正しました。

Comment

#32255 by wada at 2023-12-04T18:20:38; updated 2023-12-04T18:20:38

@kouho
エラー発生時にどのような表示になりますか?
確認してもエラー表示になりませんでした。

Comment

#32256 by wada at 2023-12-05T10:32:59; updated 2023-12-05T10:32:59

@kouho
エラー出力だとデバッガは確認できないので、お知らせ上にエラーテキストを表示することは出来ませんか?

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

Comment

#32260 by kouho at 2023-12-05T11:03:12; updated 2023-12-05T11:03:12

@wada
エラーログではなく、お知らせ上にエラーテキストを表示するように修正しました。

Comment

#32270 by wada at 2023-12-05T15:52:15; updated 2023-12-05T15:52:15

@kouho
ありがとうございます。
船ボーナスが少ないケースもエラーテキストがほしいです。

Comment

#32361 by kouho at 2023-12-08T19:11:05; updated 2023-12-08T19:11:05

@wada
船ボーナスが少ないケースにもエラーメッセージを表示して表示内容をわかりやすくどう不整合が起きているのか明記するように修正しました。

Comment

#32383 by wada at 2023-12-11T16:03:26; updated 2023-12-11T16:03:26

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

wada delete_label at 2023-12-11T16:03:31: コメントあり
wada add_label at 2023-12-11T16:03:33: 和田 LGTM
kouho merge at 2023-12-11T17:08:33: 【12/4】天下無双の船ボーナスとスケジュールの整合性チェック追加
kouho close at 2023-12-11T17:08:33: Close

Commit range

base7d77081342620ffe185d7cad4cca4d685d76b3fd
head3d56a8b07faa2d9ff686605a9ee15a809616049c
base branchjolly/jr-web:master
head branchjolly/jr-web:battle_event_schedule_check

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed2 filesView patch
statusfile
modifiedlib/conf/class/BattleEventConfig.class.php
modifiedpublic_html/template/parts/parts_battle_schedule_info_template.tpl
Patch body is stored as a separate file to keep this page lightweight.