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

Enhanced custom weapons support #183

Merged

Conversation

Petercov
Copy link

New custom weapons system with support for script-defined entity classnames.

Test Content

Does this PR close any issues?

  • No

PR Checklist

  • My PR follows all guidelines in the CONTRIBUTING.md file
  • My PR targets a develop branch OR targets another branch with a specific goal in mind

@Petercov Petercov marked this pull request as ready for review April 19, 2022 07:18
samisalreadytaken pushed a commit to samisalreadytaken/source-sdk-2013 that referenced this pull request Jan 15, 2023
Kick code falls back to entity parents

Approved-by: 1upD
Copy link
Member

@Blixibon Blixibon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me.

Obviously, I'm extremely late on this, and I'd like to apologize for not getting to this PR sooner. The changes in this were substantial enough that I put it off for a future update instead of v7.0 or v7.1, and it unfortunately took me a long time before I was able to begin work on that.

This seems like an amazing system and it fills an important niche in Mapbase's capabilities. Custom weapons are commonly sought after in mapping competitions, and while overriding existing weapons and using the experimental VScript-driven custom weapon were more-or-less passable methods, this takes things to another level. I hope that people can make good use of it.

@Blixibon Blixibon merged commit 6915787 into mapbase-source:develop Oct 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants