lighthouse icon indicating copy to clipboard operation
lighthouse copied to clipboard
verified publisher icon nuwave

Add @upload directive and tests, allows a file to be passed as an arg…

Open eNzyOfficial opened this issue 3 years ago • 3 comments

…ument, uploaded to disk and replaced with the uploaded file path

  • [ ] Added or updated tests
  • [ ] Documented user facing changes
  • [ ] Updated CHANGELOG.md

Resolves #2094

Changes

Added @upload directive. This will take in an UploadedFile, store it to a disk and return the resulting filepath.

If disk is omitted, it defaults to config value filesystems.default If path is ommited, it defaults to /

eNzyOfficial avatar Apr 18 '22 05:04 eNzyOfficial

Please review the tests, I used existing tests as examples when creating my own, so I could have missed something.

eNzyOfficial avatar Apr 18 '22 05:04 eNzyOfficial

Requested changes have been made.

eNzyOfficial avatar Apr 20 '22 08:04 eNzyOfficial

Static analysis is now passing, requested changes have been made.

eNzyOfficial avatar May 06 '22 03:05 eNzyOfficial

Sorry it's taken so long to get around to this. I've made the changes requested above.

eNzyOfficial avatar Sep 12 '22 04:09 eNzyOfficial

I've modified the directive to include the public argument, this allows the ability to set if the visibility should be public/private.

eNzyOfficial avatar Sep 12 '22 05:09 eNzyOfficial