Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Display files in case detail #319

Merged

Conversation

Pascal-Delange
Copy link
Contributor

@Pascal-Delange Pascal-Delange commented Dec 19, 2023

Copy link

linear bot commented Dec 19, 2023

MAR-379 Upload a file & list files on a case

  • User clicks on "Add a file" here
  • Modal show up, with
    • title : "Add a file"
    • text : drop any file here to attach it to the case. The following extensions are supported: …
    • a drop zone (similar to .csv uploads) + button
  • On submit, show a spinner
  • When the file is uploaded, add it to the list of files (added here)
    • A new event is added to the timeline "File Added"
    • TODO : Segment event
  • When clicking on the file name, simply open it in another tab
    • URL should be protected, short-lived
    • To access the URL, user needs read access on the case itself
  • supported extensions
    • jpg, png, pdf, zip, doc, docx, xls, xlsx, jpeg

Remaining TODO

  • POST upload files on case
  • store them in GCS or in local temp folder
  • front UI to upload file (modal)
  • back-end input validation (file type, size)
  • read API to download file (signed url)
  • Add new env variable to cloud run jobs
  • create buckets
  • front UI to list and download files on a case
  • front UI to download file from events

@Pascal-Delange Pascal-Delange force-pushed the pascal/mar-379-upload-a-file-list-files-on-a-case branch from 8280a26 to 7e0f800 Compare December 19, 2023 14:43
@Pascal-Delange Pascal-Delange force-pushed the pascal/mar-379-upload-a-file-list-files-on-a-case branch from 7e0f800 to 7bfe826 Compare December 19, 2023 14:44
@Pascal-Delange Pascal-Delange marked this pull request as ready for review December 19, 2023 14:44
@Pascal-Delange Pascal-Delange requested a review from a team December 19, 2023 15:06
@Pascal-Delange Pascal-Delange merged commit 3b700da into main Dec 19, 2023
@Pascal-Delange Pascal-Delange deleted the pascal/mar-379-upload-a-file-list-files-on-a-case branch December 19, 2023 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants