[pve-devel] [PATCH proxmox_login_manager] ui: Changed "sites" to "hosts".

[pve-devel] [PATCH proxmox_login_manager] ui: Changed "sites" to "hosts".

[Alexander Abraham]

Changed the "Available Sites" text inside the added PVE
hosts view to "Available Hosts" because "Sites" is a bit
odd.

Signed-off-by: Alexander Abraham <a.abraham@proxmox.com>
---
 lib/proxmox_login_selector.dart | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/proxmox_login_selector.dart b/lib/proxmox_login_selector.dart
index fb1218b..7fc0068 100644
--- a/lib/proxmox_login_selector.dart
+++ b/lib/proxmox_login_selector.dart
@@ -127,7 +127,7 @@ class _ProxmoxLoginSelectorState extends State<ProxmoxLoginSelector> {
               const Padding(
                 padding: EdgeInsets.all(12.0),
                 child: Text(
-                  'Available Sites',
+                  'Available Hosts',
                   style: TextStyle(
                     fontSize: 18,
                     fontWeight: FontWeight.bold,
-- 
2.39.5



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Re: [pve-devel] [PATCH proxmox_login_manager] ui: Changed "sites" to "hosts".

[Dominik Csapak]

On 5/28/25 13:04, Alexander Abraham wrote:
> Changed the "Available Sites" text inside the added PVE
> hosts view to "Available Hosts" because "Sites" is a bit
> odd.
> 

I disagree. While it could happen that users add all hosts to their app,
every entry in the app is basically a connection to a cluster, which can contain
multiple hosts.

I know we don't have a fallback connection mechanism for other hosts yet, but
IMO  'Sites' / 'Available Sites' still fit better as we don't list all
hosts from all clusters here.

(The same holds for the other patch for pve_flutter_frontend)

> Signed-off-by: Alexander Abraham <a.abraham@proxmox.com>
> ---
>   lib/proxmox_login_selector.dart | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/lib/proxmox_login_selector.dart b/lib/proxmox_login_selector.dart
> index fb1218b..7fc0068 100644
> --- a/lib/proxmox_login_selector.dart
> +++ b/lib/proxmox_login_selector.dart
> @@ -127,7 +127,7 @@ class _ProxmoxLoginSelectorState extends State<ProxmoxLoginSelector> {
>                 const Padding(
>                   padding: EdgeInsets.all(12.0),
>                   child: Text(
> -                  'Available Sites',
> +                  'Available Hosts',
>                     style: TextStyle(
>                       fontSize: 18,
>                       fontWeight: FontWeight.bold,



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel