search

mnapoli / BlackBox

[Experimental] Versatile storage library
http://mnapoli.fr/BlackBox/
14 stars 0 forks source link

Proposal: Use Flysystem for file storage #7

Open assertchris opened 9 years ago

assertchris commented 9 years ago

Would be able to support more types of filesystems, like S3 and Dropbox...

mnapoli commented 9 years ago

Ohhh I like that idea!

sagikazarmark commented 9 years ago

If you let me, I would go for it to learn the package architecture. What do you think?

mnapoli commented 9 years ago

@sagikazarmark :+1: go ahead :)

sagikazarmark commented 9 years ago

Flysystem adapter could eventually replace the existing file storage.

sagikazarmark commented 9 years ago

@mnapoli do you think we could replace the entire file implementation with a flysystem one? IMO in 2015 adding an extra dependency is not that big problem. Also, we would end up having one backend instead of two.

mnapoli commented 9 years ago

I'd say at first let's add a new one, and then we'll see if we remove the File one.

sagikazarmark commented 9 years ago

I had to wait for things to be cleared, started to work on this again.