search

ava-innersource / Liquid-Application-Framework-1.0-deprecated

Liquid is a framework to speed up the development of microservices
MIT License
25 stars 14 forks source link

Liquid.Repository.MediaStorageConfiguration isn't generalizable #121

Open bruno-brant opened 4 years ago

bruno-brant commented 4 years ago

The configuration of a Media Storage is specific to a storage provider - Azure - it expects a ConnectionString and a Container, terms that are specific to Azure Blob Storage and not used by AWS, Google and others.

We should move this class from Liquid.Repository to near its consumers, inside Liquid.OnAzure and Liquid.OnAWS, etc. The name of the configuration should also reflect which storage it's from, of course.

bruno-brant commented 4 years ago

We need to remember to make sure this isn't a breaking change.