-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Mute: Nodes #8088
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
Closed
Closed
Mute: Nodes #8088
Changes from all commits
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
ef29b22
Disable OLED and Ext notifications for nodes which have been muted
ford-jones 1a8aa0e
Regen protos
ford-jones 122dadb
Merge branch 'develop' into 7943-mute-node
ford-jones 3bffd09
Merge branch 'develop' into 7943-mute-node
ford-jones d36b919
trunk fmt, regen protos
ford-jones 5bd253a
Merge branch 'develop' into 7943-mute-node
ford-jones 1d9ab65
Regen protos
ford-jones 02e8da4
Merge branch '7943-mute-node' of https://github.com/ford-jones/firmwa…
ford-jones 9946002
Use correct proto branch
ford-jones 55ec0ed
Update proto branch
ford-jones b2f5e9d
Add is_muted flag to NodeInfo
ford-jones 8d0cd48
Resolve merge conflict
ford-jones 1e1d52c
Merge branch 'develop' into 7943-mute-node
ford-jones c7ed036
trunk fmt
ford-jones e9fc4b3
Merge branch 'develop' into 7943-mute-node
ford-jones f004cff
Merge branch 'develop' into 7943-mute-node
ford-jones 1271754
Merge branch 'develop' into 7943-mute-node
ford-jones 33c61b8
Merge branch 'develop' into 7943-mute-node
ford-jones 470d0ca
Resolve merge conflict
ford-jones b5cffb6
Trunk fmt
ford-jones fad34f0
Merge branch 'develop' into 7943-mute-node
ford-jones 83db71c
Fix: Resolved merge conflicts
ford-jones 1e2dbdf
Merge branch 'develop' into 7943-mute-node
ford-jones 9323471
Clean: Run trunk fmt
ford-jones fe4bee4
Merge branch '7943-mute-node' of https://github.com/ford-jones/firmwa…
ford-jones 13cd148
Fix: Resolve DebugRenderer conflicts
ford-jones 6375ab3
Fix: Resolve NodeDB conflicts
ford-jones 718add9
Merge branch 'develop' into 7943-mute-node
ford-jones 9f1e092
Fix: Resolve UIRenderer conflicts
ford-jones 2375aec
Merge branch '7943-mute-node' of https://github.com/ford-jones/firmwa…
ford-jones 90cca25
Fix: Linting
ford-jones File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,50 +1,39 @@ | ||
| { | ||
| "build": { | ||
| "arduino": { | ||
| "ldscript": "esp32s3_out.ld", | ||
| "memory_type": "qio_opi" | ||
| }, | ||
| "core": "esp32", | ||
| "extra_flags": [ | ||
| "-DBOARD_HAS_PSRAM", | ||
| "-DLILYGO_TBEAM_1W", | ||
| "-DARDUINO_USB_CDC_ON_BOOT=1", | ||
| "-DARDUINO_USB_MODE=0", | ||
| "-DARDUINO_RUNNING_CORE=1", | ||
| "-DARDUINO_EVENT_RUNNING_CORE=1" | ||
| ], | ||
| "f_cpu": "240000000L", | ||
| "f_flash": "80000000L", | ||
| "flash_mode": "qio", | ||
| "psram_type": "opi", | ||
| "hwids": [ | ||
| [ | ||
| "0x303A", | ||
| "0x1001" | ||
| ] | ||
| ], | ||
| "mcu": "esp32s3", | ||
| "variant": "t-beam-1w" | ||
| "build": { | ||
| "arduino": { | ||
| "ldscript": "esp32s3_out.ld", | ||
| "memory_type": "qio_opi" | ||
| }, | ||
| "connectivity": [ | ||
| "wifi", | ||
| "bluetooth", | ||
| "lora" | ||
| "core": "esp32", | ||
| "extra_flags": [ | ||
| "-DBOARD_HAS_PSRAM", | ||
| "-DLILYGO_TBEAM_1W", | ||
| "-DARDUINO_USB_CDC_ON_BOOT=1", | ||
| "-DARDUINO_USB_MODE=0", | ||
| "-DARDUINO_RUNNING_CORE=1", | ||
| "-DARDUINO_EVENT_RUNNING_CORE=1" | ||
| ], | ||
| "debug": { | ||
| "openocd_target": "esp32s3.cfg" | ||
| }, | ||
| "frameworks": [ | ||
| "arduino" | ||
| ], | ||
| "name": "LilyGo TBeam-1W", | ||
| "upload": { | ||
| "flash_size": "16MB", | ||
| "maximum_ram_size": 327680, | ||
| "maximum_size": 16777216, | ||
| "require_upload_port": true, | ||
| "speed": 921600 | ||
| }, | ||
| "url": "http://www.lilygo.cn/", | ||
| "vendor": "LilyGo" | ||
| } | ||
| "f_cpu": "240000000L", | ||
| "f_flash": "80000000L", | ||
| "flash_mode": "qio", | ||
| "psram_type": "opi", | ||
| "hwids": [["0x303A", "0x1001"]], | ||
| "mcu": "esp32s3", | ||
| "variant": "t-beam-1w" | ||
| }, | ||
| "connectivity": ["wifi", "bluetooth", "lora"], | ||
| "debug": { | ||
| "openocd_target": "esp32s3.cfg" | ||
| }, | ||
| "frameworks": ["arduino"], | ||
| "name": "LilyGo TBeam-1W", | ||
| "upload": { | ||
| "flash_size": "16MB", | ||
| "maximum_ram_size": 327680, | ||
| "maximum_size": 16777216, | ||
| "require_upload_port": true, | ||
| "speed": 921600 | ||
| }, | ||
| "url": "http://www.lilygo.cn/", | ||
| "vendor": "LilyGo" | ||
| } |
Submodule protobufs
updated
9 files
| +1 −5 | buf.yaml | |
| +12 −30 | meshtastic/admin.proto | |
| +0 −1 | meshtastic/atak.proto | |
| +0 −1 | meshtastic/channel.proto | |
| +1 −8 | meshtastic/config.proto | |
| +6 −1 | meshtastic/deviceonly.proto | |
| +0 −5 | meshtastic/mesh.options | |
| +9 −112 | meshtastic/mesh.proto | |
| +0 −8 | meshtastic/portnums.proto |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If the protobufs have been updated, we shouldn't see the changes in here. Someone probably needs to do a protobuf merge from master or something ...
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey! Not sure I'm understanding correctly.
These protobuf additions (
set_muted_nodeandremove_muted_node) were added by me and pertain to this pull request. This stuff here is the result of runningregen-protos.sh.Is that what you're referring to?
The protobuf changes are here: meshtastic/protobufs#779
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
generally the protobuf changes are merged into the repo separately through an automated process, so individual patches shouldn't need to carry the changes...
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I hear ya, that's why I opened the proto PR. I'm just a little confused as this file we're discussing is one of the
generated/meshtastic/*.pb.h. Should these not be checked in? Otherwise this wouldn't build or pass the CI.Happy to jump in a call to get on the same page when you're next available 🙂
Sorry if I'm totally misunderstanding.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
New, you're right - since the protobuf was released they should be here, and so these changes would just magically disappear . But it's not, so the automation is not working.