PR

【2/24】覇戦リーグ本戦 #2621

kouho opened this at 2023-02-21T14:28:03; closed

Description

opened by kouho at 2023-02-21T14:28:03; updated 2023-02-24T09:58:44

概要

https://docs.google.com/spreadsheets/d/1AmGUCMG0q0g4xLvrL5LgfP6j3X2fzGiwidCTGy4wdIg/edit#gid=1710863681

テスト方法、確認した内容

  • 本戦報酬は変更がありません。
  • 報酬や情報・開催期間に誤りがないか

TODO

  • 次回の予選からは予選時に1デバッグで本戦まで確認する流れにする

Comment

#25961 by kouho at 2023-02-21T14:28:44; updated 2023-02-21T14:28:44

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

Comment

#25975 by wada at 2023-02-21T17:59:34; updated 2023-02-21T17:59:34

@kouho
守護獣イベントNoの更新は予選の時だけで大丈夫ですよ。

Comment

#25976 by kouho at 2023-02-21T18:09:36; updated 2023-02-21T18:09:36

@wada
jrye.event_scheduleで本戦と予選が分けて入っていましたのでイベントNoを更新するようにしていました。
コメントにある守護獣リリース後どうなるかを把握していないのですがもしかして後々まずいのでしょうか?

Comment

#25977 by wada at 2023-02-21T18:18:40; updated 2023-02-21T18:18:40

@kouho
jrye.event_scheduleで分けていたのは、以前は同じイベントNoだったので区別するために、予選と本戦を分けていました。

イベントNoは何番でも大丈夫ですよ。
守護獣でしか使っていなくて、守護獣は予選・本戦同じイベントNoなので、なにか理由があるのか気になってコメントしました。

wada add_label at 2023-02-21T18:18:51: 和田 LGTM

Comment

#25979 by kouho at 2023-02-21T19:03:21; updated 2023-02-21T19:03:21

@wada
> jrye.event_scheduleで分けていたのは、以前は同じイベントNoだったので区別するために、予選と本戦を分けていました。

>イベントNoは何番でも大丈夫ですよ。
守護獣でしか使っていなくて、守護獣は予選・本戦同じイベントNoなので、なにか理由があるのか気になってコメントしました。

イベントNoは何番でも大丈夫なこと了解です。予選と同じものにしました。

kouho merge at 2023-02-24T09:58:44: 【2/24】覇戦リーグ本戦
kouho close at 2023-02-24T09:58:44: Close

Commit range

based1d4f631b69d11d178b3a0d5aca79e997b753fcd
head2a7a8baafc6b9e3747f876272f293b304f41a444
base branchjolly/jr-web:develop
head branchjolly/jr-web:jackpot_29

View commits

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