BlackBox icon indicating copy to clipboard operation
BlackBox copied to clipboard

Proposal: Use Flysystem for file storage

Open assertchris opened this issue 10 years ago • 7 comments

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

assertchris avatar Jan 10 '15 07:01 assertchris

Ohhh I like that idea!

mnapoli avatar Jan 10 '15 07:01 mnapoli

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

sagikazarmark avatar Jan 11 '15 10:01 sagikazarmark

@sagikazarmark :+1: go ahead :)

mnapoli avatar Jan 12 '15 03:01 mnapoli

Flysystem adapter could eventually replace the existing file storage.

sagikazarmark avatar Jan 12 '15 15:01 sagikazarmark

@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.

sagikazarmark avatar Jan 18 '15 21:01 sagikazarmark

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

mnapoli avatar Jan 18 '15 21:01 mnapoli

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

sagikazarmark avatar Feb 21 '15 15:02 sagikazarmark