Add configuration to delete Tenant Application artifacts #435

Merged
tcdlpds merged 2 commits from ashvini/device-mgt-core:Imp#11122 into master 3 months ago
* Backend Implementation to select whether app artifacts should be deleted in the tenant Deletion process * Ticket: https://roadmap.entgra.net/issues/11122
ashvini changed title from Add configuration to delete Tenant Application artifacts to WIP: Add configuration to delete Tenant Application artifacts 3 months ago
ashvini force-pushed Imp#11122 from 48069dd351 to 0d33a9e674 3 months ago
ashvini force-pushed Imp#11122 from 0d33a9e674 to 80c5e72fa0 3 months ago
ashvini force-pushed Imp#11122 from 80c5e72fa0 to b4d0896c95 3 months ago
ashvini changed title from WIP: Add configuration to delete Tenant Application artifacts to Add configuration to delete Tenant Application artifacts 3 months ago
tcdlpds requested changes 3 months ago
*/
void deleteApplicationDataOfTenant(int tenantId) throws ApplicationManagementException;
void deleteApplicationDataByTenantDomain(String tenantDomain) throws ApplicationManagementException;
Owner

Add Java Doc comments for all these methods

Add Java Doc comments for all these methods
log.error(msg);
return Response.status(Response.Status.UNAUTHORIZED).entity(msg).build();
} else {
if(deleteAppArtifacts){
Owner

Fix formatting issue

Fix formatting issue
tcdlpds marked this conversation as resolved
ashvini force-pushed Imp#11122 from a6f6e9ae84 to e01f1f8437 3 months ago
ashvini changed title from Add configuration to delete Tenant Application artifacts to WIP: Add configuration to delete Tenant Application artifacts 3 months ago
ashvini force-pushed Imp#11122 from e01f1f8437 to 9e13ca057b 3 months ago
ashvini force-pushed Imp#11122 from 9e13ca057b to e0ec8da653 3 months ago
ashvini force-pushed Imp#11122 from e0ec8da653 to a9797199e1 3 months ago
ashvini force-pushed Imp#11122 from a9797199e1 to 711bcf18f1 3 months ago
ashvini changed title from WIP: Add configuration to delete Tenant Application artifacts to Add configuration to delete Tenant Application artifacts 3 months ago
rajitha requested changes 3 months ago
componentContext.getBundleContext().registerService(DeviceStatusManagementService.class.getName(),
deviceStatusManagementService, null);
TenantMgtDataHolder.getInstance().setDeviceStatusManagementService(deviceStatusManagementService);
TenantManagerAdminService tenantManagerAdminService = new TenantManagerAdminServiceImpl();
Collaborator

Why there are two services for tenant management admin service?

Why there are two services for tenant management admin service?
tcdlpds marked this conversation as resolved
ashvini force-pushed Imp#11122 from 711bcf18f1 to 9698f93bd6 3 months ago
ashvini force-pushed Imp#11122 from 9698f93bd6 to 8c23437247 3 months ago
ashvini changed title from Add configuration to delete Tenant Application artifacts to WIP: Add configuration to delete Tenant Application artifacts 3 months ago
ashvini force-pushed Imp#11122 from 8c23437247 to 2df6575470 3 months ago
tcdlpds requested changes 3 months ago
int tenantId;
try {
tenantId = DataHolder.getInstance().getTenantManagerAdminService().getTenantId(tenantDomain);
APIUtil.getApplicationStorageManager().deleteAppFolderOfTenant(tenantId);
Owner

Can't we get this from dataholder?

Can't we get this from dataholder?
tcdlpds marked this conversation as resolved
tcdlpds requested changes 3 months ago
ashvini force-pushed Imp#11122 from 2df6575470 to e1a47c68fb 3 months ago
ashvini changed title from WIP: Add configuration to delete Tenant Application artifacts to Add configuration to delete Tenant Application artifacts 3 months ago
tcdlpds merged commit 2ac2b0fc37 into master 3 months ago

Reviewers

rajitha requested changes 3 months ago
tcdlpds requested changes 3 months ago
The pull request has been merged as 2ac2b0fc37.
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: community/device-mgt-core#435
Loading…
There is no content yet.