Skip to content

Conversation

jwhitak
Copy link
Collaborator

@jwhitak jwhitak commented Sep 15, 2025

What this does

This PR solves an issue caused by opening multiple input windows at the same time. If you spammed the select module button as a borg, while you would receive the tools/sprite/etc of the first choice you make, the last choice in the flood of input windows is what you would count-as for purposes of borg module upgrades. This PR fixes this bug.

This PR also changes the wording of an incompatible upgrade, now clearly stating that the upgrade is incompatible, not just "it will not fit".

Finally, as a bonus because I don't want to make another PR, this fixes Borgs/MoMMis from being able to open TGUI-based computer consoles. This was broken by #37919 for two months and nobody reported it on github.

Why it's good

Fixes #38394.

How it was tested

image image image

Changelog

🆑

  • bugfix: Borgs can no longer change their internal borg type multiple times by spamming the set module button.
  • bugfix: Borgs and MoMMi can remotely view tgui-based windows again.

@jwhitak jwhitak added Bug / Fix This is either a bug or a fix for a bug. 100% tested I promise I tested it every possible way / HAHA HE PROMISED HE TESTED IT AND DIDN'T labels Sep 15, 2025
@jwhitak jwhitak changed the title Fixes Borgs with Invalid Second Input Module Misc Borg Bugfixes Sep 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
100% tested I promise I tested it every possible way / HAHA HE PROMISED HE TESTED IT AND DIDN'T Bug / Fix This is either a bug or a fix for a bug.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Secborg not taking sec upgrade board
1 participant