-
Notifications
You must be signed in to change notification settings - Fork 2
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
Implement Boss #53
base: main
Are you sure you want to change the base?
Implement Boss #53
Conversation
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.
Pretty nice! Code is easy to follow and the vacuum attack is a cool mechanic.
The boss health bar works, though maybe later we could locate it at the bottom since it may be confused with the player's health bar. The outlined health bar looks better in my opinion.
The boss was quite difficult to defeat using melee attacks (I used ranged attacks from a distance instead). That might be due to boss melee animations not being implemented yet so we couldn't predict/see its moves, but not a big problem since we could build the boss fight arena so that the player can hide and strategically shoot the boss with ranged attacks.
If you would like to make further changes, please do so! Maybe you could nerf the boss a bit (melee attack damage is high, player could barely approach it especially with the vacuum attacks), but otherwise the boss is very well done. Feel free to merge to main anytime
ClickUp ticket link
Move boss to new scripts
Implementation description
Implements cactus boss scripts into
scenes/demos/Test New Boss
Adds:
Fixes:
Steps to test
What should reviewers focus on?
What kind of health bar is better, outline or no outline?
Any other attacks to add/change/balance?
Checklist