-
-
Notifications
You must be signed in to change notification settings - Fork 1
ワイルドフレアの変更と1.21.4移植 #976
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: releases/1-21-4
Are you sure you want to change the base?
ワイルドフレアの変更と1.21.4移植 #976
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
NePonpon
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
自分から半径2マス以内のところに撃った矢を着弾させたら別の方向に矢が爆発する
着弾してMotionが0にでもなって向きを正確にとれていないとか?
ショットガンみたいなものだと思ってるから、その場にまとまってるほうがそれっぽいかな。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
チケット URL
GH-972
対応内容・対応背景・妥協点
ワイルドフレアを1.21.4に移植した。
ダメージが扱いにくかったのでより調整・理解しやすく変更した。
やったこと
・サウンドソースの変更
・ダメージが付与された後1秒間のダメージを受けないインターバル期間を設けた。これによりほとんどの状態では1ヒットしかしなくなる。
・デバフ付与の追加
やってないこと
ダメージの調整はスキルダメージ全般の調整と同じ時期に行う予定のため、見送り。
テスト
正常に矢が拡散したほか、ダメージを一度のみ与えることができた。
レビュー観点
・矢の拡散に問題はないか?
・ダメージの付与に問題はないか?
補足
前提PR: #975
チェックリスト (ガイドライン)