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

fix issue #1930 #1935

Merged
merged 1 commit into from
Mar 1, 2024
Merged

fix issue #1930 #1935

merged 1 commit into from
Mar 1, 2024

Conversation

elgarbe
Copy link
Contributor

@elgarbe elgarbe commented Mar 1, 2024

move /conn parameters to /sys and /time to fix #1930

Copy link
Member

@vooon vooon left a comment

Choose a reason for hiding this comment

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

LGTM, Thanks!

@vooon vooon merged commit ef8de4c into mavlink:ros2 Mar 1, 2024
0 of 3 checks passed
@vooon vooon added this to the Version 2.7 milestone Mar 1, 2024
@elgarbe elgarbe deleted the fix-#1930 branch March 2, 2024 00:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Default config files wrongly set parameters for sys and time nodes
2 participants