Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
sergiutesu opened this issue Aug 23, 2022 · 4 comments
Closed

Comments

@sergiutesu
Copy link

sergiutesu commented Aug 23, 2022

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:

/// <summary>
/// Gets or sets a value indicating whether unattended package migrations are enabled.
/// </summary>
/// <remarks>
/// This is true by default.
/// </remarks>
public bool PackageMigrationsUnattended { get; set; } = true;

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.


This item has been added to our backlog AB#24992

@ronaldbarendse ronaldbarendse changed the title Package Migration issue Package migration state is not correctly displayed/updated in the backoffice Aug 23, 2022
@PriceyCoder
Copy link

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.

@ghost
Copy link

ghost commented Nov 1, 2022

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

image

@kjac
Copy link
Contributor

kjac commented Dec 13, 2022

@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.

@nikolajlauridsen
Copy link
Contributor

Fixed in #13574, thanks for reporting 😄

@nikolajlauridsen nikolajlauridsen added release/10.4.0 release/11.1.0 and removed state/sprint-candidate We're trying to get this in a sprint at HQ in the next few weeks labels Dec 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants