Skip to content

[Studio] fix: connect general settings to APIs#458

Open
Loyal-Young wants to merge 1 commit into
apache:rocketmq-studiofrom
Loyal-Young:agent/studio-general-settings-api
Open

[Studio] fix: connect general settings to APIs#458
Loyal-Young wants to merge 1 commit into
apache:rocketmq-studiofrom
Loyal-Young:agent/studio-general-settings-api

Conversation

@Loyal-Young

Copy link
Copy Markdown

Summary

  • load the general-settings form from /api/settings/general and persist changes through /api/settings/general/save
  • align the frontend settings type with GeneralSettingsVO
  • show loading, saving, and request-error feedback; add API contract coverage

Validation

  • npm.cmd test -- generalSettings.test.ts
  • npm.cmd run lint (existing warnings only)
  • tsc -p tsconfig.app.json --noEmit
  • npx.cmd vite build

@Loyal-Young
Loyal-Young marked this pull request as ready for review July 18, 2026 14:10
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.

1 participant