Upgrading Umbraco.Commerce from 13.1.4 to 13.1.10 causes error related to remainingAmount

[stefanie27]

Describe the bug
We have to upgrade to hotfix 13.1.10 to make sure we're not experiencing issue with AddToCart related to .NET 8.0.10.
Upgrade worked fine and site ran as normal for a while. After a while (Locally after rebuilding the site) following error occured:

SqlException: The object 'remainingAmount_default' is dependent on column 'remainingAmount'.
ALTER TABLE ALTER COLUMN remainingAmount failed because one or more objects access this column.
Umbraco.Commerce.Persistence.Migrations.DbUpUmbracoCommerceMigrationsRunner.MigrateUp()

Screenshots

Additional context
We were able to recreate issue again and again by multiple DEVs locally and on UAT server, by restoring previous version DB and let Umbraco Upgrade run

Umbraco Commerce version:
v13.1.10

[mattbrailsford]

Couple of questions

1. What database type is this? SQLite or sql server?
2. Is this originally a Vendr upgrade? Or has it always been Umbraco Commerce?

[mattbrailsford]

Ok, I think I might actually see the issue. For some reason your default constraint is called remainingAmount_default when really it should be named DF_umbracoCommerceGiftCard_remainingAmount.

I think if you drop the remainingAmount_default constraint and re-run the upgrade it should run, and it should also then recreate the constraint with the right name.

I'm not quite sure why it's got the wrong name, but hopefully this will get you upgraded.

[stefanie27]

Hi @mattbrailsford , Thanks for getting back to me so quickly.
It's SQL Server and original site was on Vendr. I will try, what you suggested. Thanks.

[stefanie27]

Hi @mattbrailsford ,

Just confirming dropping the constraint worked and new constraint has been added.

Regards,
Stefanie.

ALTER TABLE [dbo].[umbracoCommerceGiftCard]
DROP CONSTRAINT remainingAmount_default;

[mattbrailsford]

Excellent!

Ok, then in that case I will just call this issue closed and hopefully if anyone else has this same issue, then this should be enough to get them upgraded.

Thank you for reporting 👍

[NickNedfinity]

Can this issue also be added to the docs for upgrading from vendr to commerce?
I ask this because this issue is not being shown in the google search results.

[mattbrailsford]

@NickNedfinity I've submitted a docs PR here to add a "Common Issues" section to the migration guide umbraco/UmbracoDocs#7338