Skip to content

fix: reset abilities on (re)activation; debounce Godlet Printer materialize (#2802, #2775)#2819

Draft
Bennethxyz wants to merge 2 commits intoFreezingMoon:masterfrom
Bennethxyz:fix/escort-godlet-2802-2775
Draft

fix: reset abilities on (re)activation; debounce Godlet Printer materialize (#2802, #2775)#2819
Bennethxyz wants to merge 2 commits intoFreezingMoon:masterfrom
Bennethxyz:fix/escort-godlet-2802-2775

Conversation

@Bennethxyz
Copy link

This PR addresses #2802 and #2775.

#2802 (Escort Service appears used)

  • Always reset abilities and turn state on activation; previously a waited creature could resume with stale "used" flags.

#2775 (Godlet Printer selected but inactive on double-click)

  • Debounce materialize action: if ability 3 is already active, ignore subsequent clicks to avoid UI getting stuck.

Verification

  • With Scavenger, wait then re-activate: abilities show usable when appropriate.
  • With Dark Priest, double-click materialize: hex selection remains responsive and summoning proceeds.

No asset changes; unit tests pass locally.

@vercel
Copy link

vercel bot commented Nov 6, 2025

Someone is attempting to deploy a commit to the FreezingMoon Team on Vercel.

A member of the Team first needs to authorize it.

@Bennethxyz
Copy link
Author

CI summary: GitHub Actions (build/test/deploy) succeeded. The Vercel check shows an authorization prompt and is expected for external contributors; if required, a maintainer can authorize it, otherwise it can be treated as optional. I’ll monitor and address any feedback.

@vercel
Copy link

vercel bot commented Nov 6, 2025

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

Project Deployment Preview Updated (UTC)
ancientbeast Ready Ready Preview Nov 7, 2025 7:22am

@DreadKnight
Copy link
Member

@Bennethxyz Tested, the Materialize button within dash is not clickable for some reason, hence I can't make any units 🐻

@Bennethxyz Bennethxyz marked this pull request as ready for review November 7, 2025 06:03
@DreadKnight DreadKnight marked this pull request as draft November 7, 2025 07:25
@DreadKnight
Copy link
Member

@Bennethxyz Still broken. Managed to make one unit at very first somehow, but no more after...

@DreadKnight
Copy link
Member

@Bennethxyz Hey, any progress with this one? 🐻 It's one of the few issues remaining for the v0.5 release 🎈

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.

2 participants