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 5] UI Upgrade Changes, Bug Fixes, Code Smells, Drops #290

Merged
merged 14 commits into from
Oct 17, 2023

Conversation

Hasakev
Copy link
Contributor

@Hasakev Hasakev commented Oct 16, 2023

Hi, here are some small changes I've added:

  • Made the Upgrade UI Input Component more Balance Team friendly (added in constants etc.)
  • Made Upgrades use Crystals instead of Scrap (repair still uses Scraps)
  • Moved the Upgrades screen to a static location on the left.
  • Re-added drops to mobs and bosses
  • Removed Xlint error when game launches
  • Minor Code Smells and Bugs from SonarCloud

@Hasakev Hasakev linked an issue Oct 16, 2023 that may be closed by this pull request
@Hasakev Hasakev self-assigned this Oct 16, 2023
@Hasakev Hasakev requested a review from karthikeya-v October 16, 2023 15:23
@Hasakev Hasakev changed the title [Team 5] UI Upgrade Changes, [Team 5] UI Upgrade Changes, Bug Fixes, Code Smells, Drops Oct 16, 2023
@Hasakev Hasakev requested a review from JSLLW October 17, 2023 00:16
@Gaganx0
Copy link
Contributor

Gaganx0 commented Oct 17, 2023

please solve conflicts

@Hasakev Hasakev merged commit e11bcfa into main Oct 17, 2023
2 checks passed
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.

Tower Upgrade Pop-up is Partially Off-screen
3 participants