Skip to content

Fix has pending package migrations#17423

Merged
nikolajlauridsen merged 5 commits intov15/devfrom
v15/hotfix/fix-has-pending-package-migrations
Nov 6, 2024
Merged

Fix has pending package migrations#17423
nikolajlauridsen merged 5 commits intov15/devfrom
v15/hotfix/fix-has-pending-package-migrations

Conversation

@bergmania
Copy link
Member

@bergmania bergmania commented Nov 5, 2024

Description

Fixes #17437, where packages with package migration did report "hasPendingMigrations" when it did not.

The issue seems to happen, when the package Name is different from the package Id.

Tests

  • Install Umbraco.Licenses to Umbraco.Web.UI
  • Boot
  • Go swagger and use the management api as the client also has issues
  • Check the /umbraco/management/api/v1/package/migration-status endpoint and verify there is no pending if the package migrations are executed on boot.

nielslyngsoe and others added 5 commits November 5, 2024 10:08
…s had pending migrations from the management api endpoint, without actually have it.
…-pending-package-migrations

# Conflicts:
#	src/Umbraco.Web.UI.Client
Copy link
Contributor

@nikolajlauridsen nikolajlauridsen left a comment

Choose a reason for hiding this comment

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

Looks good, tests good 👍

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

Comments