PR

【確認でき次第】覚醒者選択画面の判定に転生機能のリリースチェックを入れる #1288

kouho opened this at 2021-11-05T14:14:04; closed

Description

opened by kouho at 2021-11-05T14:14:04; updated 2025-12-26T11:11:29

概要

開発環境で覚醒者を選ぶを押すと502 BatGatewayとなる問題の修正のためのPRです。
転生カードがあるかどうかの判定が影響しているようでしたので転生機能のリリース判定を入れました。

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

  • 覚醒者選択画面を確認して問題がないか

TODO

開発への反映は確認でき次第で、
本番への反映は次のイベントリリースの時にでも一緒にしようと思います。

kouho add_label at 2021-11-05T14:14:06: デベロップへマージ
kouho add_label at 2021-11-05T14:14:08: レビュー待ち
kouho assign at 2021-11-05T14:14:13: Not assigned:kouho

Comment

#15013 by kouho at 2021-11-05T14:14:29; updated 2021-11-05T14:14:29

@iwamoto @wada
確認お願いいたします。

Comment

#15044 by wada at 2021-11-08T11:04:57; updated 2021-11-08T11:04:57

@kouho
関数の先頭でglobal宣言お願いします。

Comment

#15049 by kouho at 2021-11-08T13:54:05; updated 2021-11-08T13:54:05

@wada
確認ありがとうございます。global宣言を追加しました。

Comment

#15050 by wada at 2021-11-08T14:31:09; updated 2021-11-08T14:31:09

@kouho
セミコロンで区切られているので、カンマに変更お願いします。

-       global $_dbc;
+       global $_df; $_dbc;

Comment

#15051 by kouho at 2021-11-08T14:43:10; updated 2021-11-08T14:43:10

@wada
失礼しました。。。修正しました。

Comment

#15052 by wada at 2021-11-08T14:50:15; updated 2021-11-08T14:50:15

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

wada add_label at 2021-11-08T14:50:18: 和田 LGTM

Comment

#15054 by iwamoto at 2021-11-08T15:44:41; updated 2021-11-08T15:44:41

@kouho
確認しました。大丈夫かと思います。

iwamoto add_label at 2021-11-08T15:44:43: 岩本 LGTM
kouho merge at 2021-11-08T16:18:18: 【確認でき次第】覚醒者選択画面の判定に転生機能のリリースチェックを入れる
kouho close at 2021-11-08T16:18:18: Close

Commit range

base5665d05010d8854f178f160ae2fc9d3533c7321b
head6f58d6f7985d4553f4f76281ebffb8d0c5889020
base branchjolly/jr-web:develop
head branchjolly/jr-web:fix_composition_error_in_dev

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/SubFunc.inc
Patch body is stored as a separate file to keep this page lightweight.