Skip to content

block creation or sending of duplicate channels. #3913

Merged
jamesarich merged 2 commits into
meshtastic:mainfrom
DaneEvans:fix/3903_dupe_channels
Dec 6, 2025
Merged

block creation or sending of duplicate channels. #3913
jamesarich merged 2 commits into
meshtastic:mainfrom
DaneEvans:fix/3903_dupe_channels

Conversation

@DaneEvans

Copy link
Copy Markdown
Collaborator

don't create or send duplicate channels by any means from the android app.

closes #3903

@github-actions github-actions Bot added the bugfix PR tag label Dec 6, 2025
@codecov

codecov Bot commented Dec 6, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 53 lines in your changes missing coverage. Please review.
✅ Project coverage is 0.53%. Comparing base (dbc5fd7) to head (3be1c1e).
⚠️ Report is 18 commits behind head on main.

Files with missing lines Patch % Lines
...ture/settings/radio/channel/ChannelConfigScreen.kt 0.00% 37 Missing ⚠️
...otlin/org/meshtastic/core/model/util/ChannelSet.kt 0.00% 8 Missing ⚠️
...n/org/meshtastic/core/ui/qr/ScannedQrCodeDialog.kt 0.00% 6 Missing ⚠️
...ain/java/com/geeksville/mesh/ui/sharing/Channel.kt 0.00% 2 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##            main   #3913      +/-   ##
========================================
- Coverage   0.53%   0.53%   -0.01%     
========================================
  Files        392     392              
  Lines      22940   22984      +44     
  Branches    2894    2903       +9     
========================================
  Hits         122     122              
- Misses     22797   22841      +44     
  Partials      21      21              

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

@jamesarich jamesarich added this pull request to the merge queue Dec 6, 2025
Merged via the queue into meshtastic:main with commit 7db7f61 Dec 6, 2025
6 checks passed
jamesarich added a commit that referenced this pull request Dec 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix PR tag

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Multiple matching channels are allowed with different names

2 participants