Project

General

Profile

Cleanup #52589

Feature #40907: mgr/dashboard: REST API improvements

mgr/dashboard: clean up controllers

Added by Ernesto Puerta about 1 year ago. Updated 11 months ago.

Status:
Resolved
Priority:
Normal
Category:
General - Back-end
Target version:
% Done:

0%

Tags:
Backport:
pacific
Reviewed:
Affected Versions:
Pull request ID:

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

Follows Dashboard - Bug #52480: mgr/dashboard: make modified API endpoints backward compatible Resolved
Copied to Dashboard - Backport #52870: pacific: mgr/dashboard: clean up controllers Resolved

History

#1 Updated by Ernesto Puerta about 1 year ago

  • Tags set to dashboard_sprint_23

#2 Updated by Ernesto Puerta 12 months ago

  • Parent task changed from #52480 to #40907

#3 Updated by Ernesto Puerta 12 months ago

  • Follows Bug #52480: mgr/dashboard: make modified API endpoints backward compatible added

#4 Updated by Ernesto Puerta 12 months ago

  • Status changed from In Progress to Pending Backport

#5 Updated by Backport Bot 12 months ago

  • Copied to Backport #52870: pacific: mgr/dashboard: clean up controllers added

#6 Updated by Loïc Dachary 11 months 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".

Also available in: Atom PDF