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

trial #22

Open
wants to merge 188 commits into
base: backup
Choose a base branch
from
Open

trial #22

wants to merge 188 commits into from

Conversation

kaggarwal2010
Copy link

I wanted to make a rpg game so I wanted to try some of them oot so if you could allow the pull request

iniside and others added 30 commits July 1, 2017 17:46
Ability assets are loadyed async, instanced and then unloaded.
Some UI work, better async loading, fundamentals of new event system for effects which decouples events from effects, making it possible to replicate only minimal info effect back to clients.
…dule which is sample implementation of ability managment
When effects are removed/expired.
Add Removed event for instant effects (for them OnApplied and OnRemoved are called one after another).
…y to WeaponAbility, added functionality to copy values from one attribute set to other
…d windows client, added base menu widgets, moved secret/key into config file
…ttribute extension class to take context into pre/post virtual functions, fixed clang warning, made AI killable, with HealthExtension class
…update MasterBonesMap and causes modular character to be break in editor/game
…to instanced and non instanced cues. Also reworking how EffectSpec and Context are created and passed around various systems, to minimize heap allocations, and as few as copies as possible
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