Umbraco-CMS icon indicating copy to clipboard operation
Umbraco-CMS copied to clipboard

Package migration state is not correctly displayed/updated in the backoffice

Open sergiutesu opened this issue 2 years ago • 1 comments

Which exact Umbraco version are you using? For example: 9.0.1 - don't just write v9

10.1.0

Bug summary

After running the package migration successfully, the button still shows as there are pending changes.

More about the Package Migration Settings here: https://our.umbraco.com/documentation/Reference/Configuration/PackageMigrationSettings/

Specifics

The default for PackageMigrationsUnattended is true and I don't have any other setting in my project to change it.

The setting doesn't have a default value in the schema: it's missing the DefaultValue attribute on the property: https://github.com/umbraco/Umbraco-CMS/blob/e626fca2432582f052cb13654eedd9e60ef8723f/src/Umbraco.Core/Configuration/Models/UnattendedSettings.cs#L38-L44

Steps to reproduce

  1. Install a completely new Umbraco v10.1.0 locally & spin it up
  2. Install Forms (I used NuGet package manager for that)
  3. Go to the packages section and click on "Run pending package migration" image

It will say it is successful. image

Going back to the packages section the button will still show as pending. image

Expected result / actual result

The button should be disabled or not there since there is nothing pending.

sergiutesu avatar Aug 23 '22 12:08 sergiutesu

I have also had this issue since at least v10.0.1. It is still happening with Umbraco v10.2 and UmbracoForms 10.1.2 It occurred with both UmbracoForms and uSync.

PriceyCoder avatar Sep 19 '22 22:09 PriceyCoder

I also see this for the packages I have installed, even in 10.3.2.

image

ghost avatar Nov 01 '22 13:11 ghost

@sergiutesu thank you for raising this issue - and I'm sorry it took us so long to respond to it.

I have reproduced it and submitted a fix in #13574 😄

For future reference, this is a regression issue caused by this commit.

kjac avatar Dec 13 '22 17:12 kjac

Fixed in #13574, thanks for reporting 😄

nikolajlauridsen avatar Dec 14 '22 10:12 nikolajlauridsen