Skip to content

Withdrawn - #112

Closed
b-Tomas wants to merge 1 commit into
mainfrom
b-Tomas/mission-status-error-lowercase
Closed

Withdrawn#112
b-Tomas wants to merge 1 commit into
mainfrom
b-Tomas/mission-status-error-lowercase

Conversation

@b-Tomas

@b-Tomas b-Tomas commented Aug 16, 2026

Copy link
Copy Markdown
Member

Withdrawn.

The platform matches mission status literals exactly, including case:
"OK" | "warning" | "error". MISSION_STATUS_ERROR published "Error",
which matches nothing, so executor missions that failed never got error
status treatment. The MiR and Gausium connectors each define their own
lowercase literals to work around this.

Adds MISSION_STATUS_WARNING to complete the vocabulary.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@b-Tomas b-Tomas changed the title fix: lowercase mission status "error" to match the platform Withdrawn Aug 16, 2026
@b-Tomas b-Tomas closed this Aug 16, 2026
@b-Tomas
b-Tomas deleted the b-Tomas/mission-status-error-lowercase branch August 16, 2026 18:43
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