Skip to content

Conversation

@DaneEvans
Copy link
Collaborator

Requires meshtastic/firmware#8523

Adds a button to node details to request neighbours from a node.

@github-actions github-actions bot added the enhancement New feature or request label Nov 15, 2025
@codecov
Copy link

codecov bot commented Dec 6, 2025

Codecov Report

❌ Patch coverage is 0% with 142 lines in your changes missing coverage. Please review.
✅ Project coverage is 0.52%. Comparing base (fd02f72) to head (353eff0).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...in/java/com/geeksville/mesh/service/MeshService.kt 0.00% 68 Missing ⚠️
app/src/main/java/com/geeksville/mesh/ui/Main.kt 0.00% 61 Missing ⚠️
...stic/feature/node/component/RemoteDeviceActions.kt 0.00% 5 Missing ⚠️
...htastic/feature/node/detail/NodeDetailViewModel.kt 0.00% 5 Missing ⚠️
...src/main/java/com/geeksville/mesh/model/UIState.kt 0.00% 2 Missing ⚠️
.../org/meshtastic/feature/node/component/NodeMenu.kt 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##            main   #3709      +/-   ##
========================================
- Coverage   0.53%   0.52%   -0.01%     
========================================
  Files        392     392              
  Lines      22940   23082     +142     
  Branches    2894    2920      +26     
========================================
  Hits         122     122              
- Misses     22797   22939     +142     
  Partials      21      21              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

}
}

override fun requestNeighbourInfo(requestId: Int, destNum: Int) = toRemoteExceptions {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Neighbour or Neighbor - idrc, but we should align on one. 😂

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Unfortunately I think the US got to protos first.

Copy link
Collaborator

@jamesarich jamesarich left a comment

Choose a reason for hiding this comment

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

Aside from or vs our inconsistence - this is looking pretty good 👍

@DaneEvans
Copy link
Collaborator Author

It also pops up on every Neighbour packet, or when one arrives while the app is closed.

I need to fix that. Was hoping to test it on new FW, but then the new FW didn't play nice.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants