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

Handle MAV_CMD_NAV_TAKEOFF as both command-long and command-int #25299

Merged
merged 7 commits into from
Oct 24, 2023

Conversation

peterbarker
Copy link
Contributor

@peterbarker peterbarker commented Oct 18, 2023

Built on top of #25294 as a bugfix. merged now

@peterbarker
Copy link
Contributor Author

peterbarker commented Oct 18, 2023

Also on top of #25233 which removes unimplemented code now merged

@peterbarker peterbarker force-pushed the pr/nav-takeoff-stuff branch 2 times, most recently from 045d1cc to 244db73 Compare October 18, 2023 08:49
@peterbarker peterbarker force-pushed the pr/nav-takeoff-stuff branch 2 times, most recently from f79d59e to 8326341 Compare October 18, 2023 23:24
@tridge tridge requested review from rmackay9 and IamPete1 October 24, 2023 00:12
@tridge
Copy link
Contributor

tridge commented Oct 24, 2023

@IamPete1 needs your approval

Copy link
Member

@IamPete1 IamPete1 left a comment

Choose a reason for hiding this comment

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

Copter changes look good.

@peterbarker peterbarker merged commit 146485f into ArduPilot:master Oct 24, 2023
@peterbarker peterbarker deleted the pr/nav-takeoff-stuff branch October 24, 2023 02:01
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.

4 participants