Add configurable board settings to Docker container #263
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add environment variables for board configuration (BOARD_NAME, BOARD_LAYOUT, BOARD_SIZE, BOARD_SET, BOARD_ANGLE) - Update Python webservice to use environment variables for redirect URL construction - Add comprehensive documentation for board configuration options - Include guidance on finding board settings from BoardSesh URLs 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
## Code Cleanup & Bug Fixes: - Remove unused imports (ssl, threading, FileResponse, StaticFiles) from main.py - Remove unused STATIC_PATH variable - Fix critical bug in bluetooth_controller.py (missing led_controller parameter) - Remove unused boardsesh_client.py file (232 lines of dead code) - Replace hardcoded "kilter" board names with environment variables - Improve error handling with specific exceptions instead of bare except clauses - Clean up unused database columns from sessions table - Add database cleanup routines for old cache and session data ## Board Configuration Improvements: - Update README with correct URL slug format for board configurations - Document proper Kilter board sets: bolt, screw, screw_bolt - Document proper Tension board sets: plastic, wood, plastic_wood - Document Tension layout naming: two-mirror, two-spray for 2.0 layouts - Add comprehensive examples with correct environment variable usage - Explain how users can find their configuration from BoardSesh URLs ## Technical Improvements: - Replace placeholder API endpoint with proper health check - Add automatic cleanup of orphaned queue items on startup - Improve WebSocket connection error handling - Add proper logging for cache operations - All Python files now pass syntax validation 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Background
The board controller Docker container previously had a hardcoded redirect URL pointing to a specific Kilter board configuration. This made it difficult for users with different board setups (Tension boards, different sizes, etc.) to use the controller effectively.
Changes
docker-compose.ymlwith sensible defaultsmain.pyto construct redirect URLs from environment variablesTest plan
🤖 Generated with Claude Code