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

Use NET_ENABLE instead of NET_ENABLED for consistency with other _ENABLE parameters #26562

Merged
merged 4 commits into from
Mar 20, 2024

Conversation

timtuxworth
Copy link
Contributor

The new networking capability in 4.5.0 has used NET_ENABLED. With rare exceptions, NET_ENABLE seems to be the standard. This fixes that.

There is no parameter conversion since this code has never been officially released.

Copy link
Contributor

@magicrub magicrub left a comment

Choose a reason for hiding this comment

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

You missed one, I got it for you!

@rmackay9 rmackay9 added DevCallEU WikiNeeded needs wiki update labels Mar 20, 2024
Copy link
Collaborator

@Ryanf55 Ryanf55 left a comment

Choose a reason for hiding this comment

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

Yay. Catch em before beta!

@tridge tridge merged commit d3125fe into ArduPilot:master Mar 20, 2024
91 checks passed
@Hwurzburg
Copy link
Collaborator

@timtuxworth you wouldn't be interested in fixing the wiki build error(s) that this PR creates, would you?

@timtuxworth
Copy link
Contributor Author

timtuxworth commented Mar 20, 2024

@timtuxworth you wouldn't be interested in fixing the wiki build error(s) that this PR creates, would you?

Does this do it @Hwurzburg ? ArduPilot/ardupilot_wiki#5833.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Networking WikiNeeded needs wiki update
Projects
Status: 4.5.0-beta4
Development

Successfully merging this pull request may close these issues.

7 participants