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

Attack merge attempt 3.1 #28

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Attack merge attempt 3.1 #28

wants to merge 5 commits into from

Conversation

qwet1235
Copy link
Collaborator

@qwet1235 qwet1235 commented Dec 2, 2022

added EnemyConstants and PlayerConstants classes to make it easier to identitfy enemy/player states

@qwet1235
Copy link
Collaborator Author

qwet1235 commented Dec 2, 2022

notes:

  • need to rescale hitbox
  • add attack boxes
  • implement movement + attack into each monster's class

@qwet1235
Copy link
Collaborator Author

qwet1235 commented Dec 4, 2022

  • Added an EnemyManager class that keeps track of all enemies on the game. Need to update with methods for spawning
  • Added hitboxes and attackboxes for all entities. The attacking methods should work but still needs a few tweaks. The game wont run properly cause we still need animations and need to add some other methods besides attacking
  • no clean architecture

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.

3 participants