Skip to content

Conversation

@stephdl
Copy link
Contributor

@stephdl stephdl commented Nov 13, 2025

Introduce the Applications Center view for managing and installing applications, including navigation updates and new modals for instance management and restart confirmation. Add translations for the new features and ensure a cohesive user experience.

NethServer/dev#7663

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces a new "Applications Center" view that provides centralized management of installed application instances across the cluster. The implementation separates application management (Applications Center) from application installation (Software Center), creating a clearer distinction between browsing available apps and managing installed instances.

Key Changes:

  • Added new /applications-center route with a dedicated view component for managing installed applications
  • Refactored modal components (SetInstanceLabelModal, RestartModuleModal) to promote code reuse
  • Updated navigation to include both Software Center (for installation) and Applications Center (for management)

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 9 comments.

Show a summary per file
File Description
core/ui/src/views/ApplicationsCenter.vue New view component implementing the Applications Center with features for listing, filtering, updating, cloning, moving, restarting, and uninstalling application instances
core/ui/src/views/SoftwareCenterAppInstances.vue Refactored to use extracted modal components and removed inline modal definitions
core/ui/src/components/software-center/SetInstanceLabelModal.vue New reusable modal component for editing instance labels
core/ui/src/components/software-center/RestartModuleModal.vue New reusable modal component for confirming instance restarts
core/ui/src/router/index.js Added route configuration for the new Applications Center view
core/ui/src/components/shell/SideMenuContent.vue Updated side menu with separate entries for Software Center and Applications Center with appropriate icons
core/ui/src/views/ClusterStatus.vue Updated navigation link to point to Applications Center
core/ui/public/i18n/en/translation.json Added translation keys for the new Applications Center feature

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 6 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

github-actions bot pushed a commit that referenced this pull request Nov 13, 2025
github-actions bot pushed a commit that referenced this pull request Nov 21, 2025
github-actions bot pushed a commit that referenced this pull request Nov 21, 2025
github-actions bot pushed a commit that referenced this pull request Nov 21, 2025
github-actions bot pushed a commit that referenced this pull request Nov 24, 2025
github-actions bot pushed a commit that referenced this pull request Nov 24, 2025
github-actions bot pushed a commit that referenced this pull request Nov 24, 2025
github-actions bot pushed a commit that referenced this pull request Nov 24, 2025
github-actions bot pushed a commit that referenced this pull request Nov 24, 2025
@stephdl stephdl requested a review from andre8244 November 24, 2025 14:03
github-actions bot pushed a commit that referenced this pull request Nov 25, 2025
Copy link
Contributor

@andre8244 andre8244 left a comment

Choose a reason for hiding this comment

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

Minor i18n fixes then we're good

github-actions bot pushed a commit that referenced this pull request Nov 25, 2025
@stephdl stephdl merged commit 69ad96d into main Nov 25, 2025
3 checks passed
@stephdl stephdl deleted the sdl-7663 branch November 25, 2025 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants