[backend/frontend] feat(settings): introduce tenant-scoped settings (#4864)

[Romuald Lemesle (RomuDeuxfois)]

Proposed changes

• Move global settings under platform settings
• Let dashboard settings onto dedicated tenant settings

[codecov]

Codecov Report

❌ Patch coverage is 8.82353% with 62 lines in your changes missing coverage. Please review.
✅ Project coverage is 63.50%. Comparing base (fc1a1ae) to head (4355610).
⚠️ Report is 1 commits behind head on release/current.

Files with missing lines	Patch %	Lines
...penaev/service/settings/TenantSettingsService.java	0.00%	29 Missing ⚠️
...custom_dashboard/CustomDashboardTenantService.java	0.00%	22 Missing ⚠️
...va/io/openaev/rest/settings/TenantSettingsApi.java	0.00%	9 Missing ⚠️
.../rest/settings/form/TenantSettingsUpdateInput.java	0.00%	1 Missing ⚠️
...v/rest/settings/response/TenantSettingsOutput.java	0.00%	1 Missing ⚠️

Additional details and impacted files

@@                  Coverage Diff                  @@
##             release/current    #5361      +/-   ##
=====================================================
- Coverage              63.53%   63.50%   -0.04%     
- Complexity              5640     5642       +2     
=====================================================
  Files                   1109     1115       +6     
  Lines                  33279    33306      +27     
  Branches                2542     2547       +5     
=====================================================
+ Hits                   21144    21150       +6     
- Misses                 10919    10940      +21     
  Partials                1216     1216              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[corinnekrych (corinnekrych)]

Romuald Lemesle (@RomuDeuxfois) I'm trying to test your PR but i might be missing some context/understanding.
1/ I log in as platofm admin
2/ create a tenant A
3/ go to Platform | Parameters see:

4/ go to Settings | Parameters

Small bug: {t('OpenAEV platform')} is missing in 3/

Question:
1/ how do i get dashboard setting displayed?
2/ is the left menu

• Settings menu => for tenant user: we could have a read only view of platform settings
• Platform menu => platform user

[Romuald Lemesle (RomuDeuxfois)]

corinnekrych (@corinnekrych) PR ready for a secound round

[corinnekrych (corinnekrych)]

So here we have a special case where /api/settings could be used for either tenant-scope or platform-scope therefore

openaev/openaev-front/src/utils/tenant-url-helper.ts

Lines 124 to 134 in 418fa1e

	const TENANT_EXEMPT_PREFIXES = [
	'/api/me',
	'/api/login',
	'/api/auth',
	'/api/reset',
	'/api/settings',
	'/api/tenants',
	'/api/logs',
	'/api/images',
	'/api/platform-groups',
	'/api/platform-roles',

won't work...
Suggestion: I wonder if a separate name api/tenant-settings would be cleaner, since we're also adding anew spring REST endpoint

[corinnekrych (corinnekrych)]

Romuald Lemesle (@RomuDeuxfois) here is the test i did:
1/ create tenant A
2/ create a new dashboard "TOTO"
3/ go to Settings | Parameter, set home dashboard to TOTO
4/ go to Home => blank screen with "select dashboard"
Looking at network tab api/settings/home-dashboard does not return any response

is it expected?

[Romuald Lemesle (RomuDeuxfois)]

corinnekrych (@corinnekrych) ready for the second round