PR

【4/11】「祟る逆賊・ギィ」スキル #1713

wada opened this at 2022-04-07T17:35:04; closed

Description

opened by wada at 2022-04-07T17:35:04; updated 2022-04-19T11:46:40

概要

【4月】ガチャ #1657
https://docs.google.com/spreadsheets/d/1MLrGmdymf5anE_9rN1fB1Z1vQvjC3_31r6_stVjj-MM/edit#gid=141274177

新スキル「怨毒」追加、GK設定

ブランチ

web: gacha_step_0420
db: gacha_step_0420

ブランチ名をガチャにしてますが、ガチャ自体の更新はこのブランチには含めていません。

カードスキルIDについて

本家IDを見てIDを振るのは手間なので、達眼スキルと同じ以下のルールで設定

sprintf("1%03d%02d", $skill_group, $id);

2桁を超えると破綻するが一旦保留

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

スキルは、「502:猛毒」と同じ効果

TODO

ガチャ画像は別pr
ガチャ実装は管理画面側で登録した後、確認

wada change_title at 2022-04-08T10:51:12: [WIP]ステップアップガチャ「祟る逆賊・ギィ」 [WIP]「祟る逆賊・ギィ」スキル
wada change_title at 2022-04-08T11:06:40: [WIP]「祟る逆賊・ギィ」スキル 【4/11】「祟る逆賊・ギィ」スキル
wada add_label at 2022-04-08T11:06:43: レビュー待ち

Comment

#18350 by wada at 2022-04-08T11:11:42; updated 2022-04-08T11:11:42

@kouho
レビューお願いします。4/20ギィガチャの新規スキル用prです。
今回のガチャ設定は、開発環境の管理画面から登録した後、確認してもらおうと思います。

Comment

#18351 by kouho at 2022-04-08T11:34:28; updated 2022-04-08T11:34:28

@wada
dbブランチプッシュできていなさそうなので確認お願いいたします

Comment

#18352 by wada at 2022-04-08T11:43:48; updated 2022-04-08T11:43:48

@kouho
すいません、dbブランチプッシュしました。

Comment

#18353 by kouho at 2022-04-08T12:05:34; updated 2022-04-08T12:05:34

@wada
ありがとうございます。追加スキル確認しました。
スキル動作は問題ないと思います。今回は関係がない部分ですが、SkillFunc.in 55324行目に潜在能力系スキルを判定する箇所があります。こちらに条件加えておきますか?それとも必要な時に加えますか?

また、既存カードのスキル変更なので忘れない内に強化アイテムも修正している方が安全かもしれません。
ID 1159 呪刻の鎧【祟る逆賊・ギィ】スキル「呪怨」が「再生」に変化

Comment

#18357 by wada at 2022-04-08T15:08:19; updated 2022-04-08T15:08:19

@kouho
確認ありがとうございます。
スキル判定について、処理ごとに修正するのが手間なのでグループ判定用の関数追加しました。

カードのスキルが変更されていなかったのでこちらも修正しました。
強化アイテムは工数を使いたくないので、公開時に変更対応します。

Comment

#18359 by kouho at 2022-04-08T15:59:36; updated 2022-04-08T15:59:36

@wada
対応ありがとうございます。確認しました。
すみません、見落としていたのですが怨毒発動時のスキル名が表示されませんでした。
Subfunc.inc 47254行目の猛毒と同様の対応が入りそうです。
また、スキル名を単に表示しようとするとスキル発動時のアイコンがおかしくなるので確認していただけませんか?

スキル追加以外についての質問です。
カードステータスについては管理画面から登録後に確認すればOKでしょうか?

Comment

#18360 by wada at 2022-04-08T16:05:21; updated 2022-04-08T16:05:21

@kouho
スキルの発動はどこで確認しましたか?

prの作り方が悪かったですね、スキル以外は全て別対応です。

Comment

#18361 by kouho at 2022-04-08T16:12:11; updated 2022-04-08T16:13:16

@wada
スキル以外は全て別対応かしこまりました。

スキルの発動はゲーム内の模擬戦ですね。ローカルブランチのgacha_step_0420でバトル用のデッキにギィを入れて、同じ海賊団メンバーの島ページからバトルをして確認しました。

Comment

#18362 by wada at 2022-04-08T16:15:51; updated 2022-04-08T16:15:51

@kouho
$skill_nameは特別な名前を付けたいときに使う処理なので、今回は設定していません。
(本家の更新では同名の設定は削除されています)
一旦、S_GetMasterSkill内のキャッシュを使用せずに確認してみてください。

Comment

#18363 by kouho at 2022-04-08T16:21:39; updated 2022-04-08T16:21:39

@wada
ありがとうございます。確かにS_GetMasterSkill内のキャッシュを使わなければ問題なく表示されました。
新スキル追加に関しては問題ないかと思います。

kouho add_label at 2022-04-08T16:21:46: 皇甫 LGTM

Comment

#18367 by wada at 2022-04-08T16:33:56; updated 2022-04-08T16:33:56

了解です。ありがとうございます。

wada merge at 2022-04-11T10:04:52: 【4/11】「祟る逆賊・ギィ」スキル
wada close at 2022-04-11T10:04:52: Close
wada refer at 2022-04-19T11:46:40: 1748:【4/20 12:00】ガチャ系リリース

Commit range

base263a10b3eee261d485ede398ac0f6717becf2b4b
head23a422e1ed1a686eb5fdf39f390dd98c9a50a46c
base branchjolly/jr-web:develop
head branchjolly/jr-web:gacha_step_0420

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed3 filesView patch
statusfile
modifiedlib/SkillFunc.inc
modifiedlib/conf/define_y.inc
modifiedpublic_html/js/JOLLYROGER_battle.js
Patch body is stored as a separate file to keep this page lightweight.