Skip to content

Add protobufs for a in-development traffic management module.#849

Merged
thebentern merged 11 commits intomeshtastic:masterfrom
h3lix1:traffic_management
Feb 10, 2026
Merged

Add protobufs for a in-development traffic management module.#849
thebentern merged 11 commits intomeshtastic:masterfrom
h3lix1:traffic_management

Conversation

@h3lix1
Copy link
Contributor

@h3lix1 h3lix1 commented Jan 19, 2026

What does this PR do?

Adds protobufs for changes required to support a traffic management module.

Related Issue

Checklist before merging

  • All top level messages commented
  • All enum members have unique descriptions

Copy link
Contributor Author

@h3lix1 h3lix1 left a comment

Choose a reason for hiding this comment

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

I think this should be ready to go..

@h3lix1 h3lix1 force-pushed the traffic_management branch from 91016c0 to 55a3604 Compare January 31, 2026 17:19
@h3lix1
Copy link
Contributor Author

h3lix1 commented Feb 5, 2026

@GUVWAF if you have a minute, can you review? Thank you.

Copy link
Member

@GUVWAF GUVWAF left a comment

Choose a reason for hiding this comment

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

Looks good to me. Sounds very useful at least for development purposes, don't have any specific comments.

@GUVWAF GUVWAF requested a review from thebentern February 5, 2026 19:14
@thebentern
Copy link
Contributor

@h3lix1 can you take a look at the formatting issues. I think the indention on some of the comments is off
image

@h3lix1
Copy link
Contributor Author

h3lix1 commented Feb 10, 2026

@thebentern I can't take credit for the fomat issue in telemetry.proto but I'll take the one in module_config.proto. Both are fixed. Or, at least it passes buf format --diff on my side.

@thebentern thebentern merged commit e1a6b3a into meshtastic:master Feb 10, 2026
1 of 2 checks passed
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.

3 participants