Skip to content

feat(xo6): add the VM name to actions that open a modal - #10310

Open
AlinaD-2912 wants to merge 6 commits into
masterfrom
xo6/add-the-vm-name-to-vm-actions
Open

feat(xo6): add the VM name to actions that open a modal#10310
AlinaD-2912 wants to merge 6 commits into
masterfrom
xo6/add-the-vm-name-to-vm-actions

Conversation

@AlinaD-2912

@AlinaD-2912 AlinaD-2912 commented Aug 26, 2026

Copy link
Copy Markdown
Collaborator

Description

XO-2897

Add the VM name to actions that open a modal, such as :

  • delete
  • force reboot
  • force shutdown
  • reboot
  • shutdown

Screenshots

Before After
Delete image image
Force reboot image image
Force shutdown image image
Reboot image image
Shutdown image image

Checklist

  • Commit
    • Title follows commit conventions
    • Reference the relevant issue (Fixes #007, See xoa-support#42, See https://...)
    • If bug fix, add Introduced by
  • Changelog
    • If visible by XOA users, add changelog entry
    • Update "Packages to release" in CHANGELOG.unreleased.md
  • PR
    • If UI changes, add screenshots
    • If not finished or not tested, open as Draft

Review process

If you are an external contributor, you can skip this part. Simply create the pull request, and we'll get back to you as soon as possible.

This 2-passes review process aims to:

  • develop skills of junior reviewers
  • limit the workload for senior reviewers
  • limit the number of unnecessary changes by the author
  1. The author creates a PR.
  2. Review process:
    1. The author assigns the junior reviewer.
    2. The junior reviewer conducts their review:
      • Resolves their comments if they are addressed.
      • Adds comments if necessary or approves the PR.
    3. The junior reviewer assigns the senior reviewer.
    4. The senior reviewer conducts their review:
      • If there are no unresolved comments on the PR → merge.
      • Otherwise, we continue with 3.
  3. The author responds to comments and/or makes corrections, and we go back to 2.

Notes:

  1. The author can request a review at any time, even if the PR is still a Draft.
  2. In theory, there should not be more than one reviewer at a time.
  3. The author should not make any changes:
    • When a reviewer is assigned.
    • Between the junior and senior reviews.
  4. If the PR relates to a change in the openAPI specification, a member of the DevOps team must also participate in the review.

@AlinaD-2912 AlinaD-2912 self-assigned this Aug 26, 2026
@AlinaD-2912
AlinaD-2912 force-pushed the xo6/add-the-vm-name-to-vm-actions branch from 0fe4d13 to ae46b36 Compare August 26, 2026 09:52
@AlinaD-2912
AlinaD-2912 marked this pull request as ready for review August 26, 2026 11:02
@plane-sync-vates

Copy link
Copy Markdown

Linked to Plane Work Item(s)

This comment was auto-generated by Plane

@AlinaD-2912
AlinaD-2912 requested a review from Elise-FZI August 26, 2026 11:21

@Elise-FZI Elise-FZI left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you should update screenshots

Comment thread CHANGELOG.unreleased.md Outdated
Comment thread @xen-orchestra/web-core/lib/locales/en.json Outdated
Comment thread @xen-orchestra/web-core/lib/locales/fr.json Outdated
Comment thread @xen-orchestra/web-core/lib/components/modal/VtsDeleteModal.vue
Comment thread @xen-orchestra/web-core/lib/components/modal/VtsActionModal.vue Outdated
Comment thread CHANGELOG.unreleased.md Outdated
@Elise-FZI
Elise-FZI requested a review from OlivierFL August 31, 2026 13:33

@OlivierFL OlivierFL left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nothing to add

@AlinaD-2912
AlinaD-2912 force-pushed the xo6/add-the-vm-name-to-vm-actions branch 5 times, most recently from c74cdf7 to e228e48 Compare September 1, 2026 12:46
Comment thread @xen-orchestra/web-core/lib/locales/fr.json Outdated
@AlinaD-2912
AlinaD-2912 force-pushed the xo6/add-the-vm-name-to-vm-actions branch from 9953b66 to a3f196e Compare September 2, 2026 08:13
# Conflicts:
#	@xen-orchestra/web-core/lib/locales/en.json
#	@xen-orchestra/web-core/lib/locales/fr.json

# Conflicts:
#	CHANGELOG.unreleased.md

# Conflicts:
#	CHANGELOG.unreleased.md
# Conflicts:
#	CHANGELOG.unreleased.md
@AlinaD-2912
AlinaD-2912 force-pushed the xo6/add-the-vm-name-to-vm-actions branch from a3f196e to 5ce1c15 Compare September 4, 2026 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants