Gábor Pór

Results 8 comments of Gábor Pór

> This way you'd have SKUs like "SHIRT-M-BLUE" or "SHIRT-L-BLACK". This part has me a little confused at the moment. `PriceVariantsProduct` content items come with a singular `ProductPart` that has...

Thanks, getting closer to understanding the issue at hand! > At this point it would be good to convert the base SKU to upper-case during publish just to be clean...

Having added the new methods to **IProductService**, the test-related classes (`DummyProductService` and `FakeProductService`) that are derived from the interface require an implementation of said methods as well. Should these have...

@DAud-IcI, a few questions: 1. Is it only the Product content type that should have an inventory part? 2. SKUs are normally strings — are they meant to be used...

6. Should any of the inventory part's fields be displayed when viewing the product on the user-facing site? ![image](https://user-images.githubusercontent.com/51411356/202751194-68fa0bfd-115a-49de-8f5b-2b9a5af79d69.png) 7. When should inventory be decreased — upon being added to...

> Do you have a workaround, perhaps? As far as I have seen, this issue can be avoided by ensuring in the recipe that the content item creation steps run...

What I've found is the logic to remove the container upon tenant deletion is [already there](https://github.com/OrchardCMS/OrchardCore/blob/main/src/OrchardCore.Modules/OrchardCore.Media.Azure/Services/MediaBlobContainerTenantEvents.cs#L57), but it's not run because the corresponding settings are bugged (i.e. setting `RemoveContainer` to...