[backend] feat(multi-tenancy): xtmhub auto-register, contact-us and clean up PlatformSettings (#5575)#5575
Conversation
…ave lost connectivity
…o avoid issues with one click deploy.
There was a problem hiding this comment.
Pull request overview
Introduces tenant-scoped XTM Hub registration storage and removes legacy XTM Hub registration fields from platform settings, including sending the tenant id during auto-registration.
Changes:
- Persist XTM Hub registration data in
TenantXtmHubRegistration(incl. auto-register + contact-us adjustments). - Extend XTM Hub auto-register payload to include
tenantId. - Remove obsolete XTM Hub registration fields from platform settings + delete legacy keys via Flyway migration.
Reviewed changes
Copilot reviewed 10 out of 11 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| openaev-model/src/main/java/io/openaev/database/model/SettingKeys.java | Removes unused XTM Hub registration-related setting keys. |
| openaev-front/src/utils/api-types.d.ts | Updates generated frontend types to reflect removed platform settings fields. |
| openaev-api/src/test/java/io/openaev/xtmhub/XtmHubServiceTest.java | Updates unit tests for entity-based registration persistence + tenantId payload. |
| openaev-api/src/test/java/io/openaev/rest/XtmHubApiTest.java | Updates integration test setup for contact-us to use tenant registration entity. |
| openaev-api/src/main/java/io/openaev/xtmhub/XtmHubService.java | Writes registration to TenantXtmHubRegistration; reads default-tenant registration for contact-us. |
| openaev-api/src/main/java/io/openaev/xtmhub/XtmHubClient.java | Adds tenantId parameter and includes it in the auto-register JSON body. |
| openaev-api/src/main/java/io/openaev/service/PlatformSettingsService.java | Removes legacy XTM Hub registration read/write logic from platform settings. |
| openaev-api/src/main/java/io/openaev/rest/settings/response/PlatformSettings.java | Removes XTM Hub registration/token fields from API response model. |
| openaev-api/src/main/java/io/openaev/migration/V4_99__Remove_xtmhub_settings_keys.java | Deletes legacy XTM Hub keys from parameters table after migration to tenant entity. |
| openaev-api/src/main/java/io/openaev/api/xtmhub/XtmHubApi.java | Updates RBAC resource type for auto-register endpoint to XTM_HUB_REGISTRATION. |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## release/current #5575 +/- ##
=====================================================
+ Coverage 63.76% 63.83% +0.06%
- Complexity 5825 5828 +3
=====================================================
Files 1145 1146 +1
Lines 34375 34356 -19
Branches 2652 2648 -4
=====================================================
+ Hits 21920 21930 +10
+ Misses 11161 11132 -29
Partials 1294 1294 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
note #5555 (to be merged to master but will end up on release/current imminently) might have an impact on this patch @carinelebas FYI |
Proposed changes
Testing Instructions
Auto-register:
contact-us:
Clean up:
Related issues
Checklist
Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...