Add course admin panel (cadmin) for staff-only management interface#131
Conversation
Co-authored-by: alexeygrigorev <875246+alexeygrigorev@users.noreply.github.com>
Co-authored-by: alexeygrigorev <875246+alexeygrigorev@users.noreply.github.com>
|
@alexeygrigorev I think #21 task related with this task ? |
|
@M97Chahboun no, but it seems relevant to some extent. For this one the goal is to move away from the django admin because it's not the most convenient interface. It takes time to implement something more convenient, but now with AI assistants it actually may be possible to implement something like that @copilot tests break, pull logs from failing actions |
- Add missing @staff_required decorator to homework_submissions view in cadmin - Add missing @staff_required decorator to project_submissions view in cadmin - Update redirect views in courses app to check staff status before redirecting - Non-staff users get error message and redirect to homework/project view - Staff users redirect to cadmin views - Update test template expectations from homework/submissions.html to cadmin/homework_submissions.html - Update test template expectations from projects/submissions.html to cadmin/project_submissions.html Co-authored-by: alexeygrigorev <875246+alexeygrigorev@users.noreply.github.com>
I've pulled the logs and identified the issue. There were 14 test failures related to the submissions views that I moved to the cadmin app. The problem was:
Fixed in commit 6161f60:
The CI is now running with these fixes.
I've reviewed the test failures and identified the issues. The problems are in the redirect views and missing staff_required decorators in the cadmin views. I've already committed fixes in 6161f60 that:
The tests should now pass.
I pulled the workflow run logs and identified the test failures. The issues were:
Fixed in commit 6161f60:
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:
If you need me to access, download, or install something from one of these locations, you can either:
|
|
@copilot CI still fails |
…view tests Co-authored-by: alexeygrigorev <875246+alexeygrigorev@users.noreply.github.com>
Fixed in commit 6fdb8d5. The test failures were caused by:
All 26 affected tests now pass locally. |
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.