Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pr move param transaction protocol #279

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

MatejFranceskin
Copy link

These messages will be removed from upstream development and need to be moved to auterion specific xml.

auterion.xml messages were reordered by message id.

@MatejFranceskin MatejFranceskin force-pushed the pr-move-param-transaction-protocol branch from 2e46646 to 9ccf420 Compare December 11, 2024 08:20
@MatejFranceskin MatejFranceskin force-pushed the pr-move-param-transaction-protocol branch from 9ccf420 to 5bbefbe Compare December 11, 2024 08:25
Copy link

@potaito potaito left a comment

Choose a reason for hiding this comment

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

Awesome, thank you. Looks good

@hamishwillee
Copy link
Collaborator

Thanks very much! The corresponding mavlink PR mavlink#2169 has merged so this can merge whenever you've pull that into the repo.

Copy link
Collaborator

@hamishwillee hamishwillee left a comment

Choose a reason for hiding this comment

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

This is doing lots of other stuff I don't know much about, but the param stuff is good!

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