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.

Sign up for GitHub

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

Jump to bottom

common.xml: Add MAV_RESULT_COMMAND_LONG_ONLY and MAV_RESULT_COMMAND_I… #340

Merged
merged 2 commits into from
Nov 29, 2023
Merged

common.xml: Add MAV_RESULT_COMMAND_LONG_ONLY and MAV_RESULT_COMMAND_I… #340

merged 2 commits into from
Nov 29, 2023

Commits on Nov 29, 2023

  1. common.xml: Add MAV_RESULT_COMMAND_LONG_ONLY and MAV_RESULT_COMMAND_I… 

    …NT_ONLY to MAV_RESULT (mavlink#1982)

* common.xml: Add MAV_RESULT_COMMAND_LONG_ONLY value to MAV_RESULT enum

* common.xml: Add MAV_RESULT_COMMAND_INT_ONLY value to MAV_RESULT enum
    @nexton-winjeel @peterbarker
    nexton-winjeel authored and peterbarker committed Nov 29, 2023
    Configuration menu
    Copy the full SHA
    0982e5c View commit details
    Browse the repository at this point in the history

  2. common.xml: remove validity requirement for COMMAND_INT_ONLY (mavlink… 

    …#2059)

... and COMMAND_LONG_ONLY

It doesn't really matter why the command is being rejected, just that it is.

Adding constraints about why an autopilot might send the message reduces the usefulness of this.
    @peterbarker
    peterbarker committed Nov 29, 2023
    Configuration menu
    Copy the full SHA
    cb2ed9f View commit details
    Browse the repository at this point in the history