Skip to content

Fix add qr acting like replace instead of add#3823

Merged
jamesarich merged 2 commits intomeshtastic:mainfrom
RCGV1:fix-add-qr
Nov 26, 2025
Merged

Fix add qr acting like replace instead of add#3823
jamesarich merged 2 commits intomeshtastic:mainfrom
RCGV1:fix-add-qr

Conversation

@RCGV1
Copy link
Copy Markdown
Member

@RCGV1 RCGV1 commented Nov 26, 2025

Closes #3464 ??

There was a major bug that made a add channel actually replace all channels instead. The only thing that worked correctly was add would not override the lora settings.

This is what happened when I would share just one channel from iOS to Android
image

image

@github-actions github-actions bot added the bugfix PR tag label Nov 26, 2025
@DaneEvans
Copy link
Copy Markdown
Collaborator

does 3464 match what you were seeing ? My memory is that it was doing nothing, but I would have used an Android QR code, so maybe they're different bugs.

@jamesarich jamesarich enabled auto-merge November 26, 2025 13:36
@codecov
Copy link
Copy Markdown

codecov bot commented Nov 26, 2025

Codecov Report

❌ Patch coverage is 0% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 0.54%. Comparing base (fb2aebf) to head (a322a17).
⚠️ Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
...n/org/meshtastic/core/ui/qr/ScannedQrCodeDialog.kt 0.00% 10 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##            main   #3823      +/-   ##
========================================
- Coverage   0.54%   0.54%   -0.01%     
========================================
  Files        386     386              
  Lines      22525   22531       +6     
  Branches    2828    2829       +1     
========================================
  Hits         122     122              
- Misses     22382   22388       +6     
  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 Nov 26, 2025
Merged via the queue into meshtastic:main with commit 7fd1f19 Nov 26, 2025
6 checks passed
@jamesarich jamesarich added this to the 2.7.8 milestone Nov 26, 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]: Add channels from QR not playing nicely

3 participants