Just updated at the weekend. I've run in to issue where update and finalised order handlers are not being hit - has anyone else had the same issue? Able to shed any light on what I am missing?
No issues highlighted in the code as you can see from the screens.
I think you have something that isn't upgraded somewhere as there is no VendrWebComposer for your composer to run after in v2 so something isn't right there.
Maybe clear your solution and rebuild it to make sure no rogue v1 files are hanging around.
Vendr.Web is fine, this is a v2 package. The one that got removed is Vendr.Core.Web.
But VendrWebComposer was removed as part of the v2 upgrade so there is now only VendrComposer which is why it looks like something funky is happening because VS should never have said that was OK, so either it's remembering something it shouldn't, or it's lying 🤷♂️
Order Handlers not working after upgrade
Afternoon,
Umb V 8.18 Vendr 2.1.1
Just updated at the weekend. I've run in to issue where update and finalised order handlers are not being hit - has anyone else had the same issue? Able to shed any light on what I am missing?
No issues highlighted in the code as you can see from the screens.
Hey George,
I think you have something that isn't upgraded somewhere as there is no
VendrWebComposerfor your composer to run after in v2 so something isn't right there.Maybe clear your solution and rebuild it to make sure no rogue v1 files are hanging around.
Matt
Hey Matt,
Removed VendrWebComposer now.
Still nothing happening on update or finalise of order.
Cheers,
George
There isn't a
VendrWebComposerbut there is aVendrComposerwhich you should compost after (sorry, just realised the docs on this were wrong so I've updated them https://vendr.net/docs/core/2.1.0/umbraco-v8/key-concepts/dependency-injection/)Still, if your solution thought that was ok then something is still a little off as it should have given you an error.
You can try adding the
[ComposerAfter(typeof(VendrComposer))]but like I say, I still think something is off in your solution.Hey Matt,
Yeah, it would appear so -
But Vendr.Web is installed on all projects within the solution...
Vendr.Webis fine, this is a v2 package. The one that got removed isVendr.Core.Web.But
VendrWebComposerwas removed as part of the v2 upgrade so there is now onlyVendrComposerwhich is why it looks like something funky is happening because VS should never have said that was OK, so either it's remembering something it shouldn't, or it's lying 🤷♂️is working on a reply...