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.

Sign up for GitHub

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

Jump to bottom

Build Tower bug fix #234

Merged
merged 2 commits into from
Oct 6, 2023
Merged

Build Tower bug fix #234

merged 2 commits into from
Oct 6, 2023

Conversation

Mohamad11Dab
Copy link
Contributor

Just Fixed a bug concerning tower placement.

@Mohamad11Dab Mohamad11Dab added bug Something isn't working enhancement New feature or request team 2 sprint 4 labels Oct 6, 2023
@Mohamad11Dab Mohamad11Dab self-assigned this Oct 6, 2023
@Mohamad11Dab Mohamad11Dab added this to the Sprint 4 milestone Oct 6, 2023
Hasakev
Hasakev approved these changes Oct 6, 2023
View reviewed changes
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.

Hi, This look good Mo. Every row placement seems to work except the first row, (i don't know if this is intended, other than that it's good to go!
image

@Mohamad11Dab
Copy link
Contributor Author

@Hasakev it is intended because the first column has all the gap scanners that spawns the engineers

@Hasakev Hasakev merged commit 7d8efa9 into main Oct 6, 2023
2 checks passed
@Mohamad11Dab Mohamad11Dab deleted the Mohamad-Maps branch October 17, 2023 05:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Hasakev Hasakev Hasakev approved these changes

@The-AhmadAA The-AhmadAA Awaiting requested review from The-AhmadAA

Assignees

@Mohamad11Dab Mohamad11Dab

Labels
bug Something isn't working enhancement New feature or request sprint 4 team 2
Projects
2023 Studio 3 Project
Status: Done
Milestone
Sprint 4
Development

Successfully merging this pull request may close these issues.
2 participants
@Mohamad11Dab @Hasakev