Skip to content

fixed the bug for selecting double a materialize unit#2788

Closed
tusharshah21 wants to merge 2 commits intoFreezingMoon:masterfrom
tusharshah21:bug-2775
Closed

fixed the bug for selecting double a materialize unit#2788
tusharshah21 wants to merge 2 commits intoFreezingMoon:masterfrom
tusharshah21:bug-2775

Conversation

@tusharshah21
Copy link
Copy Markdown
Contributor

Fix: Dark Priest materialize button double-click freeze

Added check to prevent double-clicking materialize button from freezing game.
Cancels ability properly when clicked twice.

Fixes #2775

Before:

ancient.beasst.mp4

After:

bug.fix.ancient.beast.mp4

wallet address: 0x2b33E4D2bD2f34310956dCb462d58413d3dCcdf8 (Metamask)

@vercel
Copy link
Copy Markdown

vercel Bot commented Sep 20, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Updated (UTC)
ancientbeast Ready Ready Preview Sep 22, 2025 5:47am

@DreadKnight
Copy link
Copy Markdown
Member

DreadKnight commented Sep 21, 2025

@tusharshah21 Video looks good, anyway, I've tested this and it's broken:
when I select a unit for materialization, the ability actually gets canceled.

@DreadKnight DreadKnight marked this pull request as draft September 21, 2025 00:20
@DreadKnight
Copy link
Copy Markdown
Member

@tusharshah21 You fixed the previous problem, but the initial bug you're trying to fix still occurs 🐻

@tusharshah21
Copy link
Copy Markdown
Contributor Author

okays can you drop a little more description mate I might have missed something or still ain't able to understand the issue completely till now then

@DreadKnight
Copy link
Copy Markdown
Member

okays can you drop a little more description mate I might have missed something or still ain't able to understand the issue completely till now then

You don't test on GitHub. Vids were on point.

@DreadKnight
Copy link
Copy Markdown
Member

@tusharshah21 Hey, any progress with this? You can use the "View deployment" buttons showing the new deployments after a few minutes a patch was pushed. Latest one leads here -> https://ancientbeast-3lvv9noy8-freezingmoon.vercel.app/ test it out 🐻

@tusharshah21
Copy link
Copy Markdown
Contributor Author

Hi mate, looking into it now! was out of station recently!

@DreadKnight
Copy link
Copy Markdown
Member

@tusharshah21 Any progress with this? 🐻

@tusharshah21
Copy link
Copy Markdown
Contributor Author

Hey, this issue is covered already I think by someone else, if not could continue on it. Let me know mate

@tusharshah21
Copy link
Copy Markdown
Contributor Author

Else we could close this PR if it is covered

@DreadKnight
Copy link
Copy Markdown
Member

Hey, this issue is covered already I think by someone else, if not could continue on it. Let me know mate

@tusharshah21 That someone else might have been an AI and that PR didn't actually worked, or else I wouldn't have been asking 😆

I asked someone else to poke at it as well, so whomever gets the job done first, should be an easy enough issue 🐻

@tusharshah21
Copy link
Copy Markdown
Contributor Author

alrighty mate updating you soon

@DreadKnight DreadKnight closed this Apr 4, 2026
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.

Godlet Printer selected but inactive [bounty: 44 XTR]

2 participants