Skip to content

Commit

Permalink
Fix has pending package migrations (#17423)
Browse files Browse the repository at this point in the history
* update submodule

* update submodule

* Reintroduce a favicon (#17422)

* Fixes issue where package name was compared to package id, so packages had pending migrations from the management api endpoint, without actually have it.

---------

Co-authored-by: Niels Lyngsø <[email protected]>
  • Loading branch information
bergmania and nielslyngsoe authored Nov 6, 2024
1 parent 21a3c31 commit 0bf2fbc
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -382,16 +382,16 @@ public Task<PagedModel<InstalledPackage>> GetInstalledPackagesFromMigrationPlans
_keyValueService.FindByKeyPrefix(Constants.Conventions.Migrations.KeyValuePrefix);

InstalledPackage[] installedPackages = _packageMigrationPlans
.GroupBy(plan => plan.PackageName)
.GroupBy(plan => (plan.PackageName, plan.PackageId))
.Select(group =>
{
var package = new InstalledPackage
{
PackageName = group.Key,
PackageName = group.Key.PackageName,
};

var currentState = keyValues?
.GetValueOrDefault(Constants.Conventions.Migrations.KeyValuePrefix + package.PackageName);
.GetValueOrDefault(Constants.Conventions.Migrations.KeyValuePrefix + group.Key.PackageId);

package.PackageMigrationPlans = group
.Select(plan => new InstalledPackageMigrationPlans
Expand Down
2 changes: 1 addition & 1 deletion src/Umbraco.Web.UI.Client
Binary file added templates/UmbracoProject/wwwroot/favicon.ico
Binary file not shown.

0 comments on commit 0bf2fbc

Please sign in to comment.