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

reimplement stick mission load as toggle; update RC control images an… #7204

Merged
merged 1 commit into from
Jun 28, 2021
Merged

reimplement stick mission load as toggle; update RC control images an… #7204

merged 1 commit into from
Jun 28, 2021

Conversation

stronnag
Copy link
Collaborator

Completes #7193

  • Re-implements stick toggle function for mission load
  • Updates RC control stick diagrams
  • Update Control documentation

Should also fix iNavFlight/inav-configurator#1270

@stronnag stronnag added the Release Notes Add this when a PR needs to be mentioned in the release notes label Jun 28, 2021
@stronnag stronnag added this to the 3.0.1 milestone Jun 28, 2021
@stronnag
Copy link
Collaborator Author

Suggested release note text:

Fixes regression in MSP mission loading from 3.0.0

  • MSP_LOAD_MISSION will unconditionally load a mission from EEPROM to RAM
  • The Mission load stick command is fully documented as a toggle:
    • If no mission is loaded to RAM, the EEPROM mission is loaded
    • If a mission is in RAM, it is cleared.
    • Successful loading is indicated by the ACTION_SUCCESS beep, otherwise the ACTION_FAIL beep is played.
      The "mission unload" toggle addresses the situation where nav_wp_load_on_boot is ON and a mission with a "distance exceeded" WP1 has been inadvertently automatically loaded.

@stronnag stronnag merged commit 562fc00 into release_3.0.1 Jun 28, 2021
@stronnag stronnag deleted the stick_toggle_mission_load_and_document branch June 28, 2021 14:48
@RobinMosedale RobinMosedale mentioned this pull request Jul 6, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Release Notes Add this when a PR needs to be mentioned in the release notes
Projects
None yet
Milestone
3.0.1
Development

Successfully merging this pull request may close these issues.
1 participant
@stronnag