Skip to content

Added status field to field access messages#7

Merged
raffael0 merged 7 commits intomainfrom
field_responses
Feb 28, 2026
Merged

Added status field to field access messages#7
raffael0 merged 7 commits intomainfrom
field_responses

Conversation

@raffael0
Copy link
Member

@raffael0 raffael0 commented Feb 26, 2026

This PR fixes the issue that we previously did not define what the server should respond, when it does not have corresponding field registered. To fix this I added a status field to the response messages

Additionally i've added a ci script, so that you can run and fix formatting/linting issues locally. See: ci-rust.sh

Here is the document diff:
LiquidCAN-diffmain.pdf

@raffael0 raffael0 force-pushed the field_responses branch 2 times, most recently from c3b4d20 to 038e0f0 Compare February 27, 2026 23:04
Copy link
Contributor

@fweichselbaum fweichselbaum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@raffael0 raffael0 merged commit c66b713 into main Feb 28, 2026
1 check passed
raffael0 added a commit that referenced this pull request Feb 28, 2026
Updated the version number for the #7 Pull Request
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.

3 participants