Hmm, my guess is the save method signature has cha...
# package-development
m
Hmm, my guess is the save method signature has changed and so it's not being found. We might need to recompile Vendr Checkout against v10
yea, it looks like
IContentService.Save
method had it's
userId
converted to a nullable int and so it's caused a method signature change. It looks like I have a workaround in Vendr core for a similar issue so I'll have to copy it over.
Actually, no I beilieve it's this commit that is the issue https://github.com/umbraco/Umbraco-CMS/commit/0151c435f162b3cdbbc2a95a16f225df619a2caa#diff-64d1e24396b0d5c6f78de95d5e823a87ef4142d21097ad33c3766c509e387a3f This introduces a new base interface and the Save method got moved to this and so anything that was compiled on an Umbraco version prior to this will fail as it expects the method to be defined on the previous interface.
Anywho, I've pushed an update to our unstable feed at https://nuget.outfield.digital/unstable/vendr/v3/index.json so if you want to give the v2.1.2-beta0001 that is on there a try and see if that resolves the issue for you?
w
My colleague ended up downloading the Vendr.Checkout source code and saw that it targets .net4.7/8 and .net5 whereas Umbraco 10 targets .net6. He installed it in our Application and removed the references to use our solution references and it worked fine, so I don't know if that impacts the code change you describe above.
m
yea, I mean that will work. The dual targeting and targeting .NET5 shouldn't matter as it's trying to support v8, v9 and v10
By recompiling it against v10 you resolve the issue anyway as now it knows to use the newer base interface
My fix just ensures it maintains compatability with the other Umbraco versions
w
My colleagues gonna give this a blast and report back!