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

Quick Fix #1025

Merged
merged 1 commit into from
Jul 11, 2024
Merged

Quick Fix #1025

merged 1 commit into from
Jul 11, 2024

Conversation

Dhruv-0-Arora
Copy link
Collaborator

Description

Fixed the Manage Assemblies modal not opening. Error coming from #1012 where the modalID was changed because it was originally misspelt. MainHUD was not updated to follow suit.
Also removed duplicate SpawningModal from Synthesis.tsx

@Dhruv-0-Arora Dhruv-0-Arora self-assigned this Jul 11, 2024
@Dhruv-0-Arora Dhruv-0-Arora requested review from HunterBarclay and a team as code owners July 11, 2024 23:02
@Dhruv-0-Arora Dhruv-0-Arora requested review from a-crowell and LucaHaverty and removed request for a team July 11, 2024 23:02
Copy link
Member

@BrandonPacewic BrandonPacewic left a comment

Choose a reason for hiding this comment

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

Nice. 🙃

@BrandonPacewic BrandonPacewic added the hotfix Pull requests that should be resolved before the next snapshot/release label Jul 11, 2024
Copy link
Contributor

@a-crowell a-crowell left a comment

Choose a reason for hiding this comment

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

Whew this was a lot. Great job.

@Dhruv-0-Arora
Copy link
Collaborator Author

Whew this was a lot. Great job.

😓

@HunterBarclay HunterBarclay merged commit 3579db2 into dev Jul 11, 2024
12 checks passed
@HunterBarclay HunterBarclay deleted the dhruv/quick-fix branch July 11, 2024 23:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hotfix Pull requests that should be resolved before the next snapshot/release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants