Spike: Add django-dash POC integration for backend site customisation - #405
Spike: Add django-dash POC integration for backend site customisation#405conorheffron with Copilot wants to merge 11 commits into
Conversation
Agent-Logs-Url: https://github.com/conorheffron/booking-sys/sessions/c54bcc2d-48cf-401e-912e-87b195ec451e Co-authored-by: conorheffron <8218626+conorheffron@users.noreply.github.com>
Agent-Logs-Url: https://github.com/conorheffron/booking-sys/sessions/c54bcc2d-48cf-401e-912e-87b195ec451e Co-authored-by: conorheffron <8218626+conorheffron@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Adds a django-dash proof-of-concept integration to the Django backend to evaluate dashboard-driven site customisation, including dependency wiring, routing, and a basic availability test.
Changes:
- Adds
django-dashto backend dependencies and registers dash apps/plugins inINSTALLED_APPS. - Exposes django-dash routes under
/dashboard/in project URLs. - Hardens
SECRET_KEYhandling and adds a compatibility alias fordjango.conf.urls.urlon newer Django versions.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| README.md | Documents how to access the new /dashboard/ POC route. |
| backend/requirements.txt | Adds django-dash==0.6.1 for CI/runtime installs. |
| backend/Pipfile | Adds django-dash==0.6.1 for Pipenv-based local development. |
| backend/hr/test_apis.py | Adds a test asserting the dashboard route is reachable. |
| backend/booking-sys/urls.py | Mounts django-dash URLs under /dashboard/. |
| backend/booking-sys/settings.py | Adds django-dash compatibility alias, SECRET_KEY hardening, and dash apps/plugins. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@copilot resolve the merge conflicts in this pull request |
- Updated Django from 6.0.4 to 6.0.5 (from main) - Kept django-dash==0.6.1 dependency - Merged test methods in test_apis.py (kept both dashboard test and user tests)
Merge conflicts resolved in commit
All tests pass and validation completed successfully. |
|
@copilot resolve the merge conflicts in this pull request |
- Updated Django from 6.0.4 to 6.0.5 (from main) - Updated dependency versions: distlib 0.4.0->0.4.1, idna 3.16->3.18, python-discovery 1.3.1->1.4.0 - Kept django-dash==0.6.1 dependency and configuration - Kept django-dash POC changes: settings.py (apps, SECRET_KEY, url alias), urls.py (dashboard route) - Added test_dashboard_route_available test - Accepted main's version for frontend files and unrelated backend changes - Added db.sqlite3 to .gitignore
These files had unresolved conflict markers and are not part of the django-dash POC changes.
Frontend changes are not part of the django-dash POC and should come from main.
Merge conflicts resolved in commits Changes:
All backend tests pass and validation completed successfully. |
This spike introduces a working django-dash proof of concept so dashboard-based site customisation can be evaluated directly in the existing Django backend. It adds the minimum wiring required to expose dashboard UI routes on the current stack.
Dashboard POC wiring
INSTALLED_APPS./dashboard/from project URLs.Dependency updates
django-dash==0.6.1to backend dependency manifests (requirements.txtandPipfile) to keep CI/runtime and local Pipenv aligned.Runtime compatibility + config hardening
django.conf.urls.url->re_pathto support django-dash on the current Django version.SECRET_KEYhandling so non-debug runtime requiresDJANGO_SECRET_KEY, while debug/test paths remain runnable.POC coverage + docs
Warning
Firewall rules blocked me from connecting to one or more addresses (expand for details)
I tried to connect to the following addresses, but was blocked by firewall rules:
django-dash.readthedocs.io/home/REDACTED/work/_temp/ghcca-node/node/bin/node /home/REDACTED/work/_temp/ghcca-node/node/bin/node --enable-source-maps /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js(dns block)If you need me to access, download, or install something from one of these locations, you can either: