-
-
Notifications
You must be signed in to change notification settings - Fork 4
Add BLE log viewer screen with expandable Maintenance section to SmartSpin2k Config App #127
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
Conversation
|
@copilot I think there was an internet outage and it's back now. YOu can try this again |
|
@copilot Lets try this again. There was an internet interruption which stopped your progress last time. Also, I notice in your summary, you have a toggle to enable/disable logs. Instead of using a button, that command should be automatically sent to enable upon entering the screen and automatically disabled when leaving the screen. doudar's avatar |
|
Lets try this again. There was an internet interruption which stopped your progress last time. Also, I notice in your summary, you have a toggle to enable/disable logs. Instead of using a button, that command should be automatically sent to enable upon entering the screen and automatically disabled when leaving the screen. doudar's avatar |
Co-authored-by: doudar <[email protected]>
|
@copilot Installing the linux dependencies to test build seems to be causing you issues. Please skip that step. |
Understood. I've completed the implementation without building. Here's what the BLE Log screen looks like: Main Device Screen - Added new "BLE Logs" navigation card: BLE Log Screen with logs: Key features implemented:
Commit: 6904f47 |
|
@copilot Instead of read current, lets add a save and send buttons. |
Co-authored-by: doudar <[email protected]>
Updated the BLE Log screen with Save and Send buttons: New Control Panel: Functionality:
All three buttons are disabled when no logs are present. Commit: 2bea487 |
|
@copilot This is looking good! Now, I think we should put "Update Firmware" and BLE Logs (lets rename that item "View Logs") into a combined option called "Maintenance" that expands showing the "Update Firmware and "View Logs" when clicked on the main device screen. |
Co-authored-by: doudar <[email protected]>
Created an expandable Maintenance section on the main device screen: Main Device Screen - Collapsed: Main Device Screen - Expanded: Changes:
Commit: 6f276b4 |
Implementation Summary
This PR adds a dedicated BLE Log Viewer screen that integrates with the SmartSpin2k firmware's new BLE log streaming capability (see doudar/SmartSpin2k#714).
Key Features
Automatic Log Streaming:
User Interface:
Demo Mode:
Technical Details
ble_logs_2026-01-12T17-13-25.txt)share_pluspackage for sharing logs via system share sheetThe implementation is minimal and focused, reusing existing functions and patterns to keep the change set small.
Original prompt
This pull request was created from Copilot chat.
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.