bito-code-review[bot] commented on PR #35794:
URL: https://github.com/apache/superset/pull/35794#issuecomment-3431889314
<details open>
<summary><strong>Interaction Diagram by <a
href="https://bito.ai#sequence_diagram";>Bito</a></strong></summary>
```mermaid
sequenceDiagram
participant APP as React App
participant TC as ThemeController<br/>🔄 Updated | ●●○ Medium
participant API as makeApi (SupersetClient)<br/>🟩 Added | ●●● High
participant BE as Theme REST API
participant DB as Database
Note over TC, API: Authentication upgrade<br/>fetch() → SupersetClient
APP->>TC: createDashboardThemeProvider(themeId)
TC->>API: makeApi GET /api/v1/theme/{id}
API->>BE: Authenticated HTTP request
BE->>DB: Query theme data
DB-->>BE: Return theme config
BE-->>API: JSON response with json_data
API-->>TC: Parsed theme config
TC-->>APP: Dashboard theme provider
```
Critical path: React App->ThemeController->makeApi
(SupersetClient)->Theme REST API->Database
> **Note:** ThemeController methods now use SupersetClient's makeApi
instead of direct fetch() calls for theme API requests. This ensures proper
authentication headers and error handling are applied to all theme-related API
calls.
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
