mat/python-comwatt-client
Watch
1
0
Fork
You've already forked python-comwatt-client
 0
Code Issues 47 Pull requests Projects Releases Packages Wiki Activity Actions

📝 fix: remove non-existent gateway diagnostic endpoint #60

Merged
mat merged 1 commit from remove-gateway-diagnostic-doc into main 2026-09-06 17:23:41 +00:00
Conversation 0 Commits 1 Files changed 2 -2
mat commented 2026-09-06 17:22:55 +00:00
Owner
Copy link

What

Removes GET /api/gateways/{gatewayId}/diagnostic from the gateway endpoint list in docs/energy.comwatt.com/devices.md, and drops the corresponding roadmap item.

Why (re-verified live 2026-09-06)

  • GET /api/gateways/359/diagnostic (numeric id) → 404 No static resource api/gateways/359/diagnostic.
  • GET /api/gateways/BXCCBB77D9/diagnostic (uid) → same 404
  • Control GET /api/gateways/359 → 200 (session and id valid)

The route does not exist server-side. The current SPA bundle (main.155e4f16.js, newer than the one the roadmap note was written against) does contain api/gateways/${id}/diagnostic as a fetcher (module export kJ), but it is dead code: fetch_gateway_diagnostic_operation occurs once, built via the comma operator and discarded, wired to no component.

Docs-only change — no client code or tests reference the endpoint. 133 tests pass.

## What Removes `GET /api/gateways/{gatewayId}/diagnostic` from the gateway endpoint list in `docs/energy.comwatt.com/devices.md`, and drops the corresponding roadmap item. ## Why (re-verified live 2026-09-06) - `GET /api/gateways/359/diagnostic` (numeric id) → **404** `No static resource api/gateways/359/diagnostic.` - `GET /api/gateways/BXCCBB77D9/diagnostic` (uid) → same 404 - Control `GET /api/gateways/359` → 200 (session and id valid) The route does not exist server-side. The current SPA bundle (`main.155e4f16.js`, newer than the one the roadmap note was written against) does contain `api/gateways/${id}/diagnostic` as a fetcher (module export `kJ`), but it is dead code: `fetch_gateway_diagnostic_operation` occurs once, built via the comma operator and discarded, wired to no component. Docs-only change — no client code or tests reference the endpoint. 133 tests pass.
📝 fix: remove non-existent gateway diagnostic endpoint (#60)
All checks were successful
Tests / forgejo-pytest (push) Successful in 43s
Details
Tests / forgejo-pytest (pull_request) Successful in 42s
Details
827c9e33d9 
GET /api/gateways/{gatewayId}/diagnostic returns 404 'No static resource'
live (re-verified 2026-09-06 on the probe account, numeric id and uid
alike, while GET /api/gateways/{id} answers 200). The current SPA bundle
(main.155e4f16.js) does contain the call but only as dead code — the
fetch_gateway_diagnostic_operation is built and discarded, wired to no
component. Drop the line from devices.md and the roadmap item.
mat changed title from WIP: 📝 fix: remove non-existent gateway diagnostic endpoint to 📝 fix: remove non-existent gateway diagnostic endpoint 2026-09-06 17:23:37 +00:00
mat merged commit c666c91b02 into main 2026-09-06 17:23:41 +00:00
mat deleted branch remove-gateway-diagnostic-doc 2026-09-06 17:23:42 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
mat/python-comwatt-client!60
No description provided.