-
Notifications
You must be signed in to change notification settings - Fork 172
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
Implements all the pickup scripts #1050
Conversation
50ae176
to
6cc0a1e
Compare
Testsed to make sure they work Tested to make sure existing script works still killing it immedialtely is live accurate, the timer was not accurate
6cc0a1e
to
37fe935
Compare
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.
Why seperate the various coin spawners into a different class? All that seems different here is just the value of coins, which could just be an argument to the constructor of your base SpecialCoinSpawner class
@EmosewaMC brought this up, but I had already done it this way lol |
add speed buff
note, armor and speed buff don't work due to something in the skill. the skills are casts and the effects play but repair and speedboost are not given. out of scope for this PR. but noting for me to look at later |
Has thos broken the imagination pickups on the ag monument? Those were working fine before |
Ah armor and speedboost specifically nvm |
split speed out to use target as caster fix armor skill
Tested to make sure they work
Tested to make sure existing script works still
killing it immediately is live accurate, the timer was not accurate