Skip to content

Graceful crew/engine errors - #326

Open
KemGus wants to merge 3 commits into
ACE-Project-Team:devfrom
KemGus:feature/graceful-crew-engine-errors
Open

Graceful crew/engine errors#326
KemGus wants to merge 3 commits into
ACE-Project-Team:devfrom
KemGus:feature/graceful-crew-engine-errors

Conversation

@KemGus

@KemGus KemGus commented Aug 2, 2026

Copy link
Copy Markdown
Member

Keeps illegal crew linked instead of dropping them, and surfaces crew/fuel legality issues on the overlay instead of spamming chat.

Does followign checkmarks from the dev board:

  • Engines no longer throw an error in chat when fuel is connected in the wrong order, they just display the situation if you look at the engine and don't activate.
  • Crew no longer throw an error in chat when crew aren't connected, the weapons just display the error and don't activate.

example 1
image


example 2
image

Known issue (if it is):

  1. if warning exists, it doesn't reupdate unless "Active" state changes. Probably is good for optimization
  2. Doesn't show warnings after spawn, unless there is a problem with either component.

@KemGus
KemGus force-pushed the feature/graceful-crew-engine-errors branch from c4a2e84 to 6f520a2 Compare August 2, 2026 23:18
@KemGus
KemGus marked this pull request as ready for review August 2, 2026 23:48
@KemGus

KemGus commented Aug 2, 2026

Copy link
Copy Markdown
Member Author

oops, found an issue here, reverting back to draft.

@KemGus
KemGus marked this pull request as draft August 2, 2026 23:57
dont need to tell all the reasons why that warning exists.
Besides that fixes an issue on smaller resolutions
@KemGus
KemGus marked this pull request as ready for review August 3, 2026 00:06
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.

1 participant