Introduce Tenant Name to Tenant Management API#290
Introduce Tenant Name to Tenant Management API#290SujanSanjula96 merged 7 commits intowso2:masterfrom
Conversation
| String.format(ERROR_CODE_DOMAIN_NOT_FOUND.getMessage(), domain)); | ||
| } | ||
| String tenantName = tenantManager.getTenantNameByID(tenant.getId()); | ||
| tenant.setName(StringUtils.isNotBlank(tenantName) ? tenantName : tenant.getDomain()); |
There was a problem hiding this comment.
You can use the "domain"
There was a problem hiding this comment.
We should return the name in the tenant.
There was a problem hiding this comment.
There's an intermediate getTenant() call here, which occurs between the tenant entry creation and organization creation. This call fails if we were to add the logic to fetch the name with the tenant.
As a precaution, it was decided to proceed with the current implementation.
Co-authored-by: Malith Dilshan <79596630+Malith-19@users.noreply.github.com>
|
PR builder started |
|
PR builder completed |
|
PR builder started |
|
PR builder completed |
4 participants
Purpose
$subject
Merge in order: