PR

天下無双の賞金首冒険時の次の報酬表示修正 #697

kouho opened this at 2021-03-26T19:28:33; closed

Description

opened by kouho at 2021-03-26T19:28:33; updated 2024-04-03T10:48:37

天下無双の賞金首冒険時の次のランキング報酬の表示部分が敷居を直書きされてました。
今回敷居が増えたため追加が必要だったため修正しています。

今後もランキング報酬の敷居をさらに細分化すると同様に修正が必要になる点が気になりますが、
既存のやり方で対応しています。

Comment

#10017 by kouho at 2021-03-26T19:29:06; updated 2021-03-26T19:29:06

@wada @iwamoto
度々すみませんが、確認お願いいたします。

Comment

#10027 by iwamoto at 2021-03-29T12:38:28; updated 2021-03-29T12:38:28

@kouho
問題ないと思います。

iwamoto add_label at 2021-03-29T12:38:35: 岩本 LGTM

Comment

#10038 by wada at 2021-03-29T14:04:02; updated 2021-03-29T14:04:02

next_rank_date_item_nameを使用していないようなので、削除しても良いです。
ランキングデータの取得を順位ごとにインデックスで取得しているので、今後報酬が増減しても対応できるようにした方が良いと思います。

#こんな感じ?
foreach ($config as $index => $lower_point) {
	if ($my_rank <= $lower_point) {
		#データ格納
	}
}
kouho commit at 2021-03-29T19:16:29: 天下無双次の報酬表示修正 #697 64709ea7b79d53bfea7757cf9e75ee8fd70c9b19
kouho commit at 2021-03-29T19:16:29: 天下無双 次のランキング報酬表示修正 & 不要部分削除 & ランキングの敷居を直書きをやめてループさせる #697 0e2925e76d972b5bc8e3ad2bde781bc923686a85

Comment

#10063 by kouho at 2021-03-29T19:18:11; updated 2021-03-29T19:18:11

@wada
確認ありがとうございます。指摘点を踏まえて修正しました。
各敷居ごとの確認は前回の本番イベントデータをコピーして、自分のポイントを以下の数値にして確認しました。

5 230000Pt
10 150000Pt
20 90000Pt
50 41000Pt
100 30000Pt
200 25000Pt
300 21000pt
400 19000Pt
500 17000Pt
700 15000Pt

Comment

#10064 by wada at 2021-03-29T19:33:48; updated 2021-03-29T19:33:48

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

wada add_label at 2021-03-29T19:33:51: 和田 LGTM

Comment

#10066 by kouho at 2021-03-30T10:01:40; updated 2021-03-30T10:01:40

@wada
確認ありがとうございます。

kouho merge at 2021-03-30T10:01:52: 天下無双の賞金首冒険時の次の報酬表示修正
kouho close at 2021-03-30T10:01:52: Close
iwamoto refer at 2021-03-31T10:17:31: 707:【3/31】リリース
kouho delete_label at 2021-04-27T16:46:29: レビュー待ち

Commit range

base0453e150221c10da39f769abe4deeb91a7876012
head0e2925e76d972b5bc8e3ad2bde781bc923686a85
base branchjolly/jr-web:develop
head branchjolly/jr-web:event_battle_8_679

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
modifiedpublic_html/template/Mission.tpl
Patch body is stored as a separate file to keep this page lightweight.