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

sitl-with-morse.rst:Updated the documentation related to Morse #3465

Merged
merged 1 commit into from May 26, 2021

Conversation

arshPratap
Copy link
Member

Updated the morse documentation to reflect the following changes:

  • Added a note signifying that Morse is not supported(see here)

  • Improvements for the installation section(see here )

  • Minor grammar fixes

Thanks!

@rishabsingh3003
Copy link
Contributor

Coincidently I was going to add a note for the Blender Version requirements when a couple of users asked me.. Can we make it a little bit more highlighted? It's something many people are doing wrong..

@arshPratap
Copy link
Member Author

@rishabsingh3003 thanks for the note! I have made some changes to the Blender Version documentation in corresponding to the suggestions. Do take a look. Thanks!!

Copy link
Contributor

@rishabsingh3003 rishabsingh3003 left a comment

Choose a reason for hiding this comment

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

LGTM!

@Hwurzburg Hwurzburg requested a review from khancyr April 28, 2021 12:52
@Hwurzburg
Copy link
Contributor

merging based on approval

@Hwurzburg Hwurzburg merged commit 6031598 into ArduPilot:master May 26, 2021
@auturgy auturgy mentioned this pull request Aug 7, 2022
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.

None yet

4 participants