Skip to content
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

Add attack vectors to martialarts_fictional.json(Five Deadly Venoms + Bionic Combatives) #60138

Merged
merged 1 commit into from
Aug 17, 2022

Conversation

TheShadowFerret
Copy link
Contributor

@TheShadowFerret TheShadowFerret commented Aug 14, 2022

Summary

Bugfixes "Five Deadly Venoms and Bionic Combatives have attack vectors"

Purpose of change

The Five Deadly Venoms and Biojutsu's attack techniques don't function due to a lack of an attack vector.

Fixes #60122.

Describe the solution

Add "appropriate" attack vector

*See additional context

Describe alternatives you've considered

Besides the FINGERS bug, a hand vector for Lizard Wall Dive.

Testing

Attack a debug monster while next to a wall(Lizard Style) using every style, see every non-disarm technique trigger. Spawn NPCs, disarm them with Bionic Combatives & Centipede Kung Fu.

Additional context

Ping @Hymore246 as maintainer of MythMA

FINGERS is supposed to be a valid attack vector per #53934 & would be used for the spearhand strike, but it doesn't register so I had to use HAND instead.

@github-actions github-actions bot added <Bugfix> This is a fix for a bug (or closes open issue) [JSON] Changes (can be) made in JSON Martial Arts Arts, Techniques, weapons and anything touching martial arts. astyled astyled PR, label is assigned by github actions labels Aug 14, 2022
@TheShadowFerret TheShadowFerret changed the title Add attack vectors to martialarts_fictional.json Add attack vectors to martialarts_fictional.json(Five Deadly Venoms + Bionic Combatives) Aug 14, 2022
@github-actions github-actions bot added json-styled JSON lint passed, label assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Aug 14, 2022
@dseguin dseguin merged commit b3977fe into CleverRaven:master Aug 17, 2022
@TheShadowFerret TheShadowFerret deleted the MissingVectors branch August 20, 2022 14:36
Hirmuolio pushed a commit to Hirmuolio/Cataclysm-DDA that referenced this pull request Aug 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions <Bugfix> This is a fix for a bug (or closes open issue) [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions Martial Arts Arts, Techniques, weapons and anything touching martial arts.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"Stinger Kick" in Scorpion Kung Fu not working
2 participants