UI Add comprehensive domain deletion confirmation dialog (Feature Request #11497)

[Imvedansh]

Implements a confirmation modal for domain deletion that shows detailed impact before proceeding, making it consistent with account deletion

Description

This PR implements a comprehensive domain deletion confirmation dialog, making the domain deletion process consistent with account deletion and providing better warnings to users about the impact of their actions.

Fixes #11497

New Component: DomainDeleteConfirm

• Created a modal component that shows detailed impact before domain deletion
• Displays a table with account-wise VM statistics:
  • Account name
  • Total VMs
  • Running VMs
  • Stopped VMs
• Shows prominent warning about permanent deletion of accounts and VMs
• Requires user to type the exact domain name to enable deletion
• Fetches live data using listAccounts and listVirtualMachines APIs

Types of changes

• Breaking change (fix or feature that would cause existing functionality to change)
• New feature (non-breaking change which adds functionality)
• Bug fix (non-breaking change which fixes an issue)
• Enhancement (improves an existing feature and functionality)
• Cleanup (Code refactoring and cleanup, that may add test cases)
• Build/CI
• Test (unit or integration test code)

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

• Major
• Minor

Bug Severity

• BLOCKER
• Critical
• Major
• Minor
• Trivial

Screenshots (if appropriate):

How Has This Been Tested?

1. Navigate to Domains section
2. Select a domain with multiple accounts and VMs
3. Click "Delete Domain" action
4. Verify modal shows:
   • Warning message about permanent deletion
   • Table with accurate account and VM counts
   • Confirmation input field
5. Try clicking "Delete Domain" without typing name → button should be disabled
6. Type incorrect domain name → button should remain disabled
7. Type correct domain name → button should enable
8. Click "Delete Domain" → domain should be deleted and UI should update

How did you try to break this feature and the system with this change?

• Domain with no accounts
• Domain with accounts but no VMs
• Domain with mix of running and stopped VMs
• Deleting domain while viewing it (should navigate away)
• Whitespace handling in domain name input

[Imvedansh]

@blueorangutan ui

[blueorangutan]

@Imvedansh a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

[blueorangutan]

UI build: ✔️
Live QA URL: https://qa.cloudstack.cloud/simulator/pr/12380 (QA-JID-838)

[Imvedansh]

@DaanHoogland WDYT?

[DaanHoogland]

clgtm

[DaanHoogland]

@Imvedansh , I moved it to 4.20 as this is the oldest supported LTS.

[DaanHoogland]

@blueorangutan ui

[blueorangutan]

@DaanHoogland a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 15.17%. Comparing base (a208db5) to head (6e9eb22).
⚠️ Report is 683 commits behind head on 4.20.

Additional details and impacted files

@@             Coverage Diff              @@
##              4.20   #12380       +/-   ##
============================================
+ Coverage     4.28%   15.17%   +10.88%     
- Complexity       0    11366    +11366     
============================================
  Files          372     5416     +5044     
  Lines        29745   476130   +446385     
  Branches      5229    58139    +52910     
============================================
+ Hits          1275    72243    +70968     
- Misses       28324   395796   +367472     
- Partials       146     8091     +7945     

Flag	Coverage Δ
uitests	4.28% <ø> (-0.01%)	⬇️
unittests	15.89% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.

[blueorangutan]

UI build: ✔️
Live QA URL: https://qa.cloudstack.cloud/simulator/pr/12380 (QA-JID-839)

[Imvedansh]

@DaanHoogland WDYT?

I was working on

@Imvedansh , I moved it to 4.20 as this is the oldest supported LTS.

Yeah , was thinking same.
I was thinking of 4.22 or for now 4.20 is fine?

[shwstppr]

Need update for async deleteDomain (try deleting a domain which has an account)

[shwstppr]

@Imvedansh deleteDomain is an async API. You would mostly always get a 200 response as it would return the jobid. So you'll have to poll that job ID instead showing success immediately

[Imvedansh]

Ahh, alrightyy.
I ll shoot changes shortly