Modify default S3 path strategy - Githubissues

agritheory / cloud_storage

S3 protocol storage for Frappe Applications - compatible with S3, Backblaze and DO Spaces

https://agritheory.com/documentation/cloud_storage

Other

13 stars 7 forks source link

Modify default S3 path strategy #37

Open agritheory opened 1 year ago

agritheory commented 1 year ago

The use case for making a date/doctype/docname flavor of S3 path has multiple issues:

Verbose
Makes less sense in the context of the file association features
Makes paths more guessable (employee/employee_id/contract.pdf)
Nobody is or really should be using the S3/Spaces UI to browse files

Switch to using the name of the File record

Allow configuration of file path with a hookable function in case there's a customer reason to customize this (for example an FTP integration)