Skip to content

feat(settings): complete #802 settings API integration gaps - #808

Open
vbeni30 wants to merge 1 commit into
devfrom
feat/issue-802-settings-api-integration
Open

feat(settings): complete #802 settings API integration gaps#808
vbeni30 wants to merge 1 commit into
devfrom
feat/issue-802-settings-api-integration

Conversation

@vbeni30

@vbeni30 vbeni30 commented Sep 5, 2026

Copy link
Copy Markdown

Summary

  • Gate settings navigation tabs by backend permissions
  • Surface tag load failures instead of showing silent empty lists
  • Remove the /settings legacy rewrite so the Next.js page is canonical
  • Add frontend tests for the API client, server actions, and settings panels (reads, writes, validation, forbidden)

Test Plan

  • pnpm run frontend:typecheck
  • Vitest settings suite (45 tests)
  • dotnet test SettingsApiController (4 tests)

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