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

Adding New Tower Variants and Upgrade Feature #127

Merged
merged 46 commits into from
Sep 9, 2023
Merged

Conversation

Mohamad11Dab
Copy link
Contributor

@Mohamad11Dab Mohamad11Dab commented Sep 9, 2023

Three new tower variants have been introduced: the Stun_Tower that fires projectiles to immobilize mobs, the Fire_Tower that spews fire to damage mobs, and the Droid which shoots projectiles that decelerate mobs. The upgrade features for these towers are developed but haven't been integrated into the Tower Factory yet. The Droid tower isn't firing projectiles yet, as we're awaiting the slow-moving projectile feature to be completed by the projectile team. Feel free to ask if any aspect is unclear.
edited: Just Integrated slow-effect projectile with the Droid

AlasdairS4698737 and others added 30 commits August 31, 2023 19:55
…to make an upgrade to the tower. No upgrade types do anything yet.
…ttack stat in the CombatStatsComponent of the same entity.
…to CombatStatsComponent to alter the fullHealth variable.
… added functionality to TowerCombatTask for supporting fire rate upgrades.
…pleted the createFireTower method in TowerFactory.
…ower.png, stun_tower.atlas and finished the base functionality for createStunTower method in TowerFactory.
updated to return without editing firerate if division by zero is attempted.
Fixed failing test after previous edit
Created and implemented animations for StunTower and FireTower.
Copy link
Contributor

@Hasakev Hasakev left a comment

Choose a reason for hiding this comment

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

No problems found, LGTM 👍
(We're getting ready to implement all these new turrets into the turret selection screen)

@Mohamad11Dab Mohamad11Dab merged commit ad3e412 into main Sep 9, 2023
2 checks passed
Copy link
Contributor

@ryandmcneilly ryandmcneilly left a comment

Choose a reason for hiding this comment

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

lgtm

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.

6 participants