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

Team 2 towers functionality merging into main #48

Merged
merged 2 commits into from
Aug 25, 2023
Merged

Conversation

The-AhmadAA
Copy link
Contributor

@The-AhmadAA The-AhmadAA commented Aug 25, 2023

Towers functionality being merged into main. The changes being incorporated are:

  • TowerFactory Class created that contains methods for creating the WeaponTower and WallTower entities. (Use weaponTower only at this stage, Walltower is not yet visible) (Refer to Task ticket [Team 2] Tower entity creation #6)
  • Tower Entities have had a CostComponent added for interfacing with Economy once the tower is placed.
  • PopUpMenuInputComponent added for basic input handling (Refer to Task [Team 2] User tower interaction #11) for integration into Interfaces' creation of Tower Popup Menus
  • Weapon Tower animation assets created
  • TowerAnimationController Class created - manages animations of WeaponTower
  • Tower AITaskController gets added to Weapon tower which runs a NPC detection script. This triggers events which are used to start animations.
  • "firingStart" event is triggered when NPC entities are in the line of sight of towers. This event is also intended to be used for triggering the creation of projectiles by ProjectileFactory. (Refer to Task Ticket [Team 2] Towers' Graphics and sound Task #9)

Refer to Feature ticket #5

@The-AhmadAA The-AhmadAA added this to the Sprint 1 milestone Aug 25, 2023
@The-AhmadAA The-AhmadAA self-assigned this Aug 25, 2023
@The-AhmadAA The-AhmadAA linked an issue Aug 25, 2023 that may be closed by this pull request
20 tasks
@The-AhmadAA The-AhmadAA removed a link to an issue Aug 25, 2023
20 tasks
Copy link
Contributor

@MiniSoda17 MiniSoda17 left a comment

Choose a reason for hiding this comment

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

A very detailed and well thought out addition to the game, well done!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

3 participants