Skip to content

[Studio] fix: connect K8s certificate page to APIs#440

Merged
lizhimins merged 1 commit into
apache:rocketmq-studiofrom
btlqql:agent/studio-k8s-cert-api
Jul 17, 2026
Merged

[Studio] fix: connect K8s certificate page to APIs#440
lizhimins merged 1 commit into
apache:rocketmq-studiofrom
btlqql:agent/studio-k8s-cert-api

Conversation

@btlqql

@btlqql btlqql commented Jul 16, 2026

Copy link
Copy Markdown

Summary

  • load K8s certificates through the service layer instead of importing page-level mock data
  • connect create, update, and delete actions to the existing backend endpoints
  • add loading, submitting, validation, SAN parsing, and request error states
  • keep mock-mode certificate CRUD consistent through an in-memory store
  • return created, updated, and renewed records from the API wrappers
  • add API contract coverage for list, create, update, and delete operations

Root cause

The certificate page initialized itself directly from mockK8sCerts and mutated only React state. Although the API and server controller already implemented certificate CRUD, production UI actions never invoked them, and the add button remained a placeholder.

Impact

Production users now see server-provided certificate data and certificate changes persist through the backend. Mock mode remains useful for local development, including create, edit, and delete flows.

Validation

  • cd web && npm test — 10 tests passed
  • cd web && npm run build — passed
  • cd web && npm run lint — 0 errors; 3 pre-existing warnings in unrelated files
  • pre-commit ESLint and Prettier checks — passed

Fixes #439

@lizhimins lizhimins changed the title [Studio][fix] Connect K8s certificate page to backend APIs [Studio] fix: connect K8s certificate page to APIs Jul 17, 2026
@lizhimins
lizhimins merged commit a002477 into apache:rocketmq-studio Jul 17, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants