Commit Graph

5 Commits

Author SHA1 Message Date
28e8425a08 fix(luci-app-secubox-admin): remove luci-app-secubox dependency to fix installation conflict
Removed unnecessary dependency on luci-app-secubox which was causing file
conflicts during installation. The admin package doesn't actually use any
modules from luci-app-secubox - it only uses its own secubox-admin.* modules.

This fixes the installation error:
  "Package luci-app-secubox wants to install file /etc/config/secubox
   But that file is already provided by package secubox-core"

Changes:
- Makefile: Removed +luci-app-secubox from LUCI_DEPENDS
- Package now only depends on: +luci-base +rpcd +secubox-core
- Incremented PKG_RELEASE: 7 → 8
- Updated DEPLOY_UPDATES.md with v1.0.0-8 details

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
2026-01-04 15:15:05 +01:00
17065bf776 fix(luci-app-secubox-admin): add graceful RPC fallback to all views
Fixed "No related RPC reply" errors across all admin views by wrapping
ALL RPC calls in L.resolveDefault() with appropriate fallback values.

This allows the frontend to load gracefully even when the backend RPCD
methods are not yet deployed, showing empty data instead of crashing.

Changes:
- health.js: Wrapped getHealth() → L.resolveDefault(getHealth(), {})
- logs.js: Wrapped getLogs() → L.resolveDefault(getLogs(), { logs: '' })
- settings.js: Wrapped getApps() and getModules() with fallbacks
- apps.js: Wrapped getApps() and getModules() (checkUpdates already wrapped)
- dashboard.js: Wrapped all 4 RPC calls (getApps, getModules, getHealth, getAlerts)
- Incremented PKG_RELEASE: 6 → 7
- Updated DEPLOY_UPDATES.md with v1.0.0-7 details

All admin pages now load successfully regardless of backend deployment status.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
2026-01-04 15:06:45 +01:00
1c5d8eb29f fix(luci-app-secubox-admin): fix WidgetRenderer constructor error
Fixed TypeError "WidgetRenderer is not a constructor" in dashboard.js
by removing the 'new' keyword. LuCI's baseclass.extend() creates callable
classes that should not be instantiated with 'new'.

Changes:
- dashboard.js: Changed from 'new WidgetRenderer({...})' to 'WidgetRenderer({...})'
- Added comprehensive try-catch error handling with fallback error display
- Incremented PKG_RELEASE: 5 → 6
- Updated DEPLOY_UPDATES.md with v1.0.0-6 details

This fix allows the widget system to initialize properly on the dashboard.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
2026-01-04 14:59:10 +01:00
753d50db70 docs: Update deployment guide with v1.0.0-5 and RPC troubleshooting
- Updated package version to 1.0.0-5
- Added troubleshooting for 'No related RPC reply' errors
- Explained graceful degradation behavior
- Clarified deployment order requirements
2026-01-04 14:44:49 +01:00
adfeed60e2 docs: Add automated deployment script and documentation
Add comprehensive deployment automation for SecuBox AppStore updates:

- deploy-to-router.sh: Automated build and deploy script
  - Rebuilds packages
  - Copies to router
  - Installs with --force-reinstall
  - Restarts RPCD
  - Verifies installation
  - Tests RPC methods

- DEPLOY_UPDATES.md: Complete deployment documentation
  - Quick deploy instructions
  - Verification steps
  - Troubleshooting guide
  - Package version tracking

Usage:
  ./deploy-to-router.sh

Or with custom router IP:
  ROUTER_IP=192.168.1.1 ./deploy-to-router.sh
2026-01-04 14:42:34 +01:00