[Environments] Auto reload after Extensions page visit; selectable error text

common/Environments/Models/ExtensionsPageLoaded.cs

@@ -0,0 +1,14 @@
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT License.
+
+using CommunityToolkit.Mvvm.Messaging.Messages;
+
+namespace DevHome.Common.Environments.Models;
+
+public class ExtensionsPageLoaded : ValueChangedMessage<bool>
+{
+    public ExtensionsPageLoaded(bool value)
+        : base(value)
+    {
+    }
+}

tools/Environments/DevHome.Environments/ViewModels/LandingPageViewModel.cs

@@ -9,6 +9,7 @@
 using System.Threading.Tasks;
 using CommunityToolkit.Mvvm.ComponentModel;
 using CommunityToolkit.Mvvm.Input;
+using CommunityToolkit.Mvvm.Messaging;
 using CommunityToolkit.WinUI;
 using CommunityToolkit.WinUI.Behaviors;
 using CommunityToolkit.WinUI.Collections;
@@ -26,7 +27,7 @@ namespace DevHome.Environments.ViewModels;
 /// <summary>
 /// The main view model for the landing page of the Environments tool.
 /// </summary>
-public partial class LandingPageViewModel : ObservableObject, IDisposable
+public partial class LandingPageViewModel : ObservableObject, IDisposable, IRecipient<ExtensionsPageLoaded>
 {
     private readonly ILogger _log = Log.ForContext("SourceContext", nameof(LandingPageViewModel));
 
@@ -48,6 +49,8 @@ public partial class LandingPageViewModel : ObservableObject, IDisposable
 
     private bool _wasSyncButtonClicked;
 
+    private bool _extensionsPageNavigatedTo;
+
     private string _selectedProvider = string.Empty;
 
     public bool IsLoading { get; set; }
@@ -111,6 +114,7 @@ public LandingPageViewModel(
         _lastSyncTime = _stringResource.GetLocalized("MomentsAgo");
         ComputeSystemCardsView = new AdvancedCollectionView(ComputeSystemCards);
         ComputeSystemCardsView.SortDescriptions.Add(new SortDescription("IsCardCreating", SortDirection.Descending));
+        WeakReferenceMessenger.Default.Register<ExtensionsPageLoaded>(this);
     }
 
     public void Initialize(StackedNotificationsBehavior notificationQueue)
@@ -216,7 +220,7 @@ private async Task RunSyncTimmer()
     /// <summary>
     /// Main entry point for loading the view model.
     /// </summary>
-    public async Task LoadModelAsync(bool useDebugValues = false)
+    public async Task LoadModelAsync()
     {
         lock (_lock)
         {
@@ -228,8 +232,10 @@ public async Task LoadModelAsync(bool useDebugValues = false)
             // If the page has already loaded once, then we don't need to re-load the compute systems as that can take a while.
             // The user can click the sync button to refresh the compute systems. However, there may be new operations that have started
             // since the last time the page was loaded. So we need to add those to the view model quickly.
+            // But if the user navigated to the extensions page, we need to reload the compute systems, since they might
+            // have enabled/disabled an extension.
             SetupCreateComputeSystemOperationForUI();
-            if (HasPageLoadedForTheFirstTime && !_wasSyncButtonClicked)
+            if (HasPageLoadedForTheFirstTime && !_wasSyncButtonClicked && !_extensionsPageNavigatedTo)
             {
                 return;
             }
@@ -269,6 +275,7 @@ public async Task LoadModelAsync(bool useDebugValues = false)
         {
             IsLoading = false;
             HasPageLoadedForTheFirstTime = true;
+            _extensionsPageNavigatedTo = false;
         }
     }
 
@@ -552,4 +559,9 @@ private void UpdateCallToActionText()
         CallToActionText = callToActionData.CallToActionText;
         CallToActionHyperLinkButtonText = callToActionData.CallToActionHyperLinkText;
     }
+
+    public void Receive(ExtensionsPageLoaded message)
+    {
+        _extensionsPageNavigatedTo = true;
+    }
 }

tools/Environments/DevHome.Environments/Views/LandingPage.xaml.cs

@@ -24,6 +24,6 @@ public LandingPage()
 
     private async void OnLoaded(object sender, RoutedEventArgs e)
     {
-        await ViewModel.LoadModelAsync(false);
+        await ViewModel.LoadModelAsync();
     }
 }

tools/ExtensionLibrary/DevHome.ExtensionLibrary/ViewModels/ExtensionLibraryViewModel.cs

@@ -8,7 +8,9 @@
 using System.Threading.Tasks;
 using CommunityToolkit.Mvvm.ComponentModel;
 using CommunityToolkit.Mvvm.Input;
+using CommunityToolkit.Mvvm.Messaging;
 using CommunityToolkit.WinUI;
+using DevHome.Common.Environments.Models;
 using DevHome.Common.Extensions;
 using DevHome.Common.Services;
 using Microsoft.UI.Dispatching;
@@ -69,6 +71,10 @@ public async Task LoadedAsync()
     {
         await GetInstalledPackagesAndExtensionsAsync();
         GetAvailablePackages();
+
+        // Send a message to the Environments page to let it know that the Extensions page has been loaded.
+        // And that the Environments page should update its list of environments.
+        WeakReferenceMessenger.Default.Send(new ExtensionsPageLoaded(true));
     }
 
     private async void OnExtensionsChanged(object? sender, EventArgs e)