Skip to content

refactor(front): extract GatewayInstanceModal domain model#1075

Merged
vitormattos merged 1 commit into
masterfrom
refactor/gateway-instance-modal-model
May 11, 2026
Merged

refactor(front): extract GatewayInstanceModal domain model#1075
vitormattos merged 1 commit into
masterfrom
refactor/gateway-instance-modal-model

Conversation

@vitormattos
Copy link
Copy Markdown
Collaborator

Summary

  • extract heavy GatewayInstanceModal domain logic into a dedicated pure module:
    • src/services/gatewayInstanceModalModel.ts
  • keep GatewayInstanceModal.vue focused on UI wiring and events
  • centralize and unit-test logic for:
    • gateway id resolution from hints/edit state
    • provider catalog normalization and selection resolution
    • current/visible fields and validation fields
    • catalog selection state sync
    • form validation (required, provider selection, integer min/max)

Commit

  • cc295335 refactor(front): extract GatewayInstanceModal domain model

Validation

  • npm run test -- src/tests/components/GatewayInstanceModal.spec.ts src/tests/services/gatewayInstanceModalModel.spec.ts src/tests/views/AdminSettings.spec.ts
    • 51 passed, 0 failed
  • npm run test -- src/tests/components/GatewayInstanceCard.spec.ts src/tests/components/GatewayInstanceModal.spec.ts src/tests/components/GatewayTestModal.spec.ts src/tests/views/AdminSettings.spec.ts src/tests/services/gatewayInstanceModalModel.spec.ts
    • 89 passed, 0 failed

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
@vitormattos vitormattos merged commit 1360600 into master May 11, 2026
37 of 38 checks passed
@vitormattos vitormattos deleted the refactor/gateway-instance-modal-model branch May 11, 2026 22:46
@vitormattos
Copy link
Copy Markdown
Collaborator Author

/backport to stable33

@vitormattos
Copy link
Copy Markdown
Collaborator Author

/backport to stable32

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