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

Updated new Taekwondo techniques to be style specific #37680

Merged
merged 1 commit into from
Feb 4, 2020
Merged

Updated new Taekwondo techniques to be style specific #37680

merged 1 commit into from
Feb 4, 2020

Conversation

Hymore246
Copy link
Contributor

Summary

SUMMARY: Bugfixes "Updated new Taekwondo techniques to be style specific"

Purpose of change

Created two new techniques for Taekwondo instead of using the techniques from other styles.

Describe the solution

In regards to #37362, I've updated the techniques added to the Taekwondo style to be unique to Taekwondo. It is bad practice to use techniques or buffs designed for one martial arts style in a second martial arts style because if the technique or buff is modified or removed to balance one style, the second style is also effected. This is why so many styles have techniques that are identical. Leaving the techniques as is can cause unwanted bugs and additional work for content creators in the future.

@ZhilkinSerg ZhilkinSerg added <Bugfix> This is a fix for a bug (or closes open issue) [JSON] Changes (can be) made in JSON Melee Melee weapons, tactics, techniques, reach attack labels Feb 4, 2020
@ZhilkinSerg ZhilkinSerg merged commit c132958 into CleverRaven:master Feb 4, 2020
@sayke
Copy link
Contributor

sayke commented Feb 4, 2020

Got it, didn't know they were supposed to be unique to each style! Thanks

@Hymore246 Hymore246 deleted the TKD-specific-techs branch February 4, 2020 21:33
@Hymore246
Copy link
Contributor Author

Got it, didn't know they were supposed to be unique to each style! Thanks

I'm working on a best practices guide to help contributers. It should help explain the do's and don'ts and game balance concepts when dealing with the martial arts system.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bugfix> This is a fix for a bug (or closes open issue) [JSON] Changes (can be) made in JSON Melee Melee weapons, tactics, techniques, reach attack
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants