Cleanup #52589
Feature #40907: mgr/dashboard: REST API improvements
mgr/dashboard: clean up controllers
Status:
Resolved
Priority:
Normal
Assignee:
Category:
General - Back-end
Target version:
% Done:
0%
Tags:
Backport:
pacific
Reviewed:
Affected Versions:
Pull request ID:
Tags:
Description
Clean up existing controller infrastructure and replace it with best practices:
- Remove extremely large files (e.g.: controllers/__init__.py) and use separate files/sub-modules instead.
- Remove use of global vars (they complicate testing/mocking, debugging, etc.).
- Rename class names according to existing MVC patterns (e.g.: Routes Python module, django/fastapi): Models, Controllers, Endpoints, Routers.
Related issues
History
#1 Updated by Ernesto Puerta over 2 years ago
- Tags set to dashboard_sprint_23
#2 Updated by Ernesto Puerta over 2 years ago
- Parent task changed from #52480 to #40907
#3 Updated by Ernesto Puerta over 2 years ago
- Follows Bug #52480: mgr/dashboard: make modified API endpoints backward compatible added
#4 Updated by Ernesto Puerta over 2 years ago
- Status changed from In Progress to Pending Backport
#5 Updated by Backport Bot over 2 years ago
- Copied to Backport #52870: pacific: mgr/dashboard: clean up controllers added
#6 Updated by Loïc Dachary over 2 years ago
- Status changed from Pending Backport to Resolved
While running with --resolve-parent, the script "backport-create-issue" noticed that all backports of this issue are in status "Resolved" or "Rejected".