search

django-oscar / django-oscar-stores

Physical store integration for Django Oscar
BSD 3-Clause "New" or "Revised" License
91 stars 58 forks source link

StoreStock and Pricing #157

Open farooqaaa opened 3 years ago

farooqaaa commented 3 years ago

Why doesn't the StoreStock model have a price field? Is it left out intentionally?

Also, I think it would be a good idea to extend Oscar's StockRecord model. The only downside would be that the field name has to be partner even though it'll be referring to a Store. Something like this:

class StoreStock(AbstractStockRecord):
    partner = models.ForeignKey('stores.Store', ...)
solarissmoke commented 3 years ago

Why doesn't the StoreStock model have a price field? Is it left out intentionally?

Yes - this isn't really intended to be a full stock management system for a store. StoreStock is intended just to keep track of whether an item is available at a given store - so that you could, for example, show customers the nearest store where it is available. I think attaching in-store pricing to this is beyond the scope of this package (although you could extend it to do something like that if you wanted to).