Skip to content

Default to zerohop advert with 'advert' CLI-Command #1714

Closed
mesher-de wants to merge 1 commit intomeshcore-dev:devfrom
mesher-de:default-to-0-hop-advert-in-CLI
Closed

Default to zerohop advert with 'advert' CLI-Command #1714
mesher-de wants to merge 1 commit intomeshcore-dev:devfrom
mesher-de:default-to-0-hop-advert-in-CLI

Conversation

@mesher-de
Copy link

This is an alternative approach to #1490

The goal is to reduce mesh-load cause by too many repeater adverts.

This PR modifies the logic of the advert command to send a zerohop advert by default and introduces a new advert.flood command for sending a flood advert.

@liamcottle
Copy link
Member

Hey, thanks for the time/effort putting in the PR :)

To be honest, would prefer to leave advert cli command as flood, otherwise now you have a command that does different things across different firmware versions.

This also means the app would need to determine which firmware version you're running so it knows to use advert.flood instead of advert. Otherwise this changes what that buttons does in the app, which may be unexpected to the user.

Would prefer to have a new command advert.zerohop that sends a zero hop advert. I haven't had a chance to review that linked PR but looks like it does something along those lines.

@mesher-de
Copy link
Author

Hey, thanks for the time/effort putting in the PR :)

To be honest, would prefer to leave advert cli command as flood, otherwise now you have a command that does different things across different firmware versions.

This also means the app would need to determine which firmware version you're running so it knows to use advert.flood instead of advert. Otherwise this changes what that buttons does in the app, which may be unexpected to the user.

Would prefer to have a new command advert.zerohop that sends a zero hop advert. I haven't had a chance to review that linked PR but looks like it does something along those lines.

Hi Liam,

Thanks for taking the time to review this!

Will update the linked #1490 to use advert.zerohop as you suggested and close this one.

@mesher-de mesher-de closed this Feb 17, 2026
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

Comments