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

Update of mspX documentation for inav for the final state of the 8.0 release. #234

Merged
merged 12 commits into from
Dec 10, 2024

Conversation

b14ckyy
Copy link
Contributor

@b14ckyy b14ckyy commented Dec 9, 2024

No description provided.

@b14ckyy
Copy link
Contributor Author

b14ckyy commented Dec 9, 2024

btw should I call it MspX like in the previous doc version or mspX like its shown in the Config Lua?

Edit: I could also add screenshots for a MSP+CRSF config to make that clearer if you want. I just realized that.

@jlpoltrack
Copy link
Collaborator

btw should I call it MspX like in the previous doc version or mspX like its shown in the Config Lua?

Let's go with 'MspX'

@jlpoltrack
Copy link
Collaborator

A fair amount of comments but I guess my question is how much around INAV 7 needs to be there? Is the expectation that INAV 7 will be in use for a long time?

Update from JLPs feedback
@b14ckyy
Copy link
Contributor Author

b14ckyy commented Dec 9, 2024

A fair amount of comments but I guess my question is how much around INAV 7 needs to be there? Is the expectation that INAV 7 will be in use for a long time?

Made all the changes as requested beside 2 smaller additions.
INAV 7 will probably be around as a proven stable version until we are a few patches and/or a few months in INAV 8.x
So at least for the next 4-6 months we will have plenty of pilots not updating. Even today we have a lot still flying 6.x versions.

further tweaks by JLPs suggestion
@b14ckyy
Copy link
Contributor Author

b14ckyy commented Dec 9, 2024

btw: I think I will not add a config example for INAV 7. If someone wants to set up CRSF for RC, that user should just refer to the INAV Documentation for normal CRSF receiver.

@jlpoltrack
Copy link
Collaborator

Okay, just few more small comments - otherwise I am happy with this.

probably last tweaks
@b14ckyy
Copy link
Contributor Author

b14ckyy commented Dec 9, 2024

alright, good to go then. Thanks for your help.

@jlpoltrack jlpoltrack merged commit 1f96658 into olliw42:master Dec 10, 2024
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.

2 participants