PR

【5/10】転生解放 #1795

wada opened this at 2022-04-28T18:51:59; closed

Description

opened by wada at 2022-04-28T18:51:59; updated 2022-05-17T14:17:26

概要

https://docs.google.com/spreadsheets/d/18PPB3afQ_amw-WwlDOgfeQAXqwjs30HW3zf1bX7Mui0/edit#gid=788225172
【5/17】転生機能 #1725

ニャッカのみ転生設定

転生カード情報から育成中カード詳細に移動するとボタンが反応しないは対応中

DB更新内容

jolly/jrye-db#49

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

master_card_limit_breakthrough.start_timeを調整
issue内のチェック項目

TODO

issue内で未チェックの内容
仕様書の作業シート内の5/10 和田以外の項目は、随時対応

表記周りの改善は全体の開発確認時に対応します。

wada change_title at 2022-05-02T10:25:39: [WIP]転生解放 【5/10】転生解放

Comment

#18809 by wada at 2022-05-02T10:33:38; updated 2022-05-02T10:33:38

@kouho
レビューお願いします。
対応箇所はissue内のチェック項目か仕様書を確認お願いします。

wada add_label at 2022-05-02T10:33:43: レビュー待ち

Comment

#18885 by kouho at 2022-05-09T11:29:39; updated 2022-05-09T11:29:39

@wada
master_composition_bonus.csvで覚醒ボーナスの設定が、
ニャッカ船長ではなくニャッカが+40にされているように見えますので確認お願いいたします。

Comment

#18887 by kouho at 2022-05-09T11:42:27; updated 2022-05-09T11:42:27

@wada
master_card_limit_breakthroughのstart_timeが現在時刻より後でS_GetLimitBreakAfterCardAllの結果がnullだとデッキ編集画面でエラーとなりました。
恐らく予め仕込むものと思いますのでS_GetLimitBreakAfterCardAll関数でselect結果が空の場合nullではなく、空配列を返した方が良いと思います。

Comment

#18888 by kouho at 2022-05-09T12:04:38; updated 2022-05-09T12:04:38

@wada
転生の書の必要数が本家の10倍という仕様なら予めmaster_limit_break_item_set.csvでニャッカ以外の転生の書も10倍にしておきませんか?毎度修正するより事故防止になるかと思います。


![スクリーンショット 2022-05-09 12](https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/1652065384224i9bIZO2ohb)

Comment

#18894 by kouho at 2022-05-09T14:54:06; updated 2022-05-09T14:55:17

@wada
デッキ編集で転生素材が足りない時のテキストです。
表記周りについてですが改善というより対応必要な項目だと思います。

修正案

  • テキスト修正

「ガチャ酒場」や「デイリークエスト」「イベントの報酬」で「イベントの報酬」など

  • リンク削除

「ガチャ酒場」「デイリークエスト」

!スクリーンショット 2022-05-09 14

Comment

#18901 by wada at 2022-05-09T17:45:49; updated 2022-05-09T17:45:49

@kouho

> master_composition_bonus.csvで覚醒ボーナスの設定が、

修正しました。

> 恐らく予め仕込むものと思いますのでS_GetLimitBreakAfterCardAll関数でselect結果が空の場合

変更箇所が多そうなので、$_df['LIMIT_BREAK_ENABLE']16:01公開にすることで、この現象が起きないようにしました。
リリースまでに対応できそうであれば、戻り値修正します。

> 転生の書の必要数が本家の10倍という仕様なら予めmaster_limit_break_item_set.csvでニャッカ

事故防止と確認しやすさのため、ニャッカ以外のデータを削除しました。
次回以降の転生情報は、他の転生情報と同時に設定します。

> デッキ編集で転生素材が足りない時のテキストです。
該当箇所を削除しました。

Comment

#18908 by kouho at 2022-05-10T14:11:49; updated 2022-05-10T14:11:49

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

kouho add_label at 2022-05-10T14:11:53: 皇甫 LGTM
wada merge at 2022-05-10T17:57:58: 【5/10】転生解放
wada close at 2022-05-10T17:57:58: Close
wada refer at 2022-05-16T15:21:30: 1827:【5/16】転生機能リリース
wada refer at 2022-05-17T14:17:26: 1827:【5/17】転生機能リリース

Commit range

baseeae2e6dea9fbcc214452dab0045fb8745fd8e893
head4e8ce55b847ec757073fad87cde24a2497c231c3
base branchjolly/jr-web:develop
head branchjolly/jr-web:limit_break

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed13 filesView patch
statusfile
modifiedlib/conf/define.inc
modifiedlib/conf/define_y.inc
modifiedlib/models/Limit_break.php
modifiedlib/public/LimitBreak-List.inc
addedpublic_html/img/common/btn_limit_break.png
modifiedpublic_html/index.php
modifiedpublic_html/template/Card-AlbumDetail.tpl
modifiedpublic_html/template/Card-Detail.tpl
modifiedpublic_html/template/Composition-Check.tpl
modifiedpublic_html/template/Deck-CardDetail.tpl
modifiedpublic_html/template/LimitBreak-Check.tpl
modifiedpublic_html/template/LimitBreak-GrowthUserCardList.tpl
modifiedpublic_html/template/Main.tpl
Patch body is stored as a separate file to keep this page lightweight.