mirror/Meshtastic-Android
1
0
Fork 0
mirror of https://github.com/meshtastic/Meshtastic-Android.git synced 2026-05-18 19:56:34 -04:00
Code Issues Packages Projects Releases Wiki Activity

docs: mark round 2 brownfield gap tasks complete

Browse Source 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This commit is contained in:
James Rich
2026-05-11 13:47:55 -05:00
parent af2854a8d3
commit f8b01071db
4 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
specs/007-node-detail-metrics/tasks.md
View File

@@ -297,7 +297,7 @@
- **Priority**: Medium
### NDM-T101: Missing — HostMetricsLogScreen chart+card test coverage
- [ ] Add unit tests for `HostMetricsChart` data model and `formatBytes` edge cases (exact boundaries)
- [x] Add unit tests for `HostMetricsChart` data model and `formatBytes` edge cases (exact boundaries)
- **Rationale**: `formatBytes` is tested but chart data transformation and card selection sync are not.
- **Priority**: Low
@@ -308,7 +308,7 @@
- **Priority**: Medium
### NDM-T103: Missing — Environment NaN guard tests
- [ ] Add tests verifying that `NaN` temperature, humidity, and pressure values are correctly filtered (not rendered, not charted)
- [x] Add tests verifying that `NaN` temperature, humidity, and pressure values are correctly filtered (not rendered, not charted)
- **Rationale**: The code has `isNaN()` guards but no tests validate them.
- **Priority**: Low

2
specs/008-radio-app-settings/tasks.md
View File

@@ -328,7 +328,7 @@
- Target: `commonTest/radio/CleanNodeDatabaseScreenTest.kt`
- Gap: Only ViewModel is tested, not the composable
- [ ] **SET-T073**: Add integration test for profile import → export round-trip verifying `DeviceProfile` protobuf fidelity
- [x] **SET-T073**: Add integration test for profile import → export round-trip verifying `DeviceProfile` protobuf fidelity
- Target: `commonTest/radio/ProfileRoundTripTest.kt`
- Gap: Import and export are tested individually but not end-to-end

2
specs/010-onboarding/tasks.md
View File

@@ -81,7 +81,7 @@
## Gaps — Uncompleted Tasks
- [ ] **OB-T100**: Extract hardcoded notification channel ID `"my_alerts"` to a shared constant or resource
- [x] **OB-T100**: Extract hardcoded notification channel ID `"my_alerts"` to a shared constant or resource
- File: `feature/intro/src/androidMain/kotlin/org/meshtastic/feature/intro/IntroNavGraph.kt` (line 112)
- Rationale: Hardcoded string is fragile; should reference the same constant used when the channel is created.

4
specs/011-wifi-provisioning/tasks.md
View File

@@ -187,12 +187,12 @@
## Identified Gaps (not yet implemented)
### WFP-T023: Expose hidden network provisioning in UI
- [ ] Add a "Hidden Network" toggle or option in `ConnectedContent` that sets `hidden = true` when calling `provisionWifi`
- [x] Add a "Hidden Network" toggle or option in `ConnectedContent` that sets `hidden = true` when calling `provisionWifi`
- [ ] Domain layer already supports `CMD_CONNECT_HIDDEN` (2) — only UI wiring needed
- **Priority**: Low — niche use case
### WFP-T024: Add retry mechanism for BLE scan timeout
- [ ] When BLE scan times out (10s), offer a "Retry" button instead of requiring the user to navigate back and re-enter
- [x] When BLE scan times out (10s), offer a "Retry" button instead of requiring the user to navigate back and re-enter
- [ ] Consider exponential backoff or a manual retry count limit
- **Priority**: Medium — improves UX for unreliable BLE environments