Skip to content

Extend schema to allow for presigned URLs for uploading files#108

Open
singingwolfboy wants to merge 13 commits intographile:mainfrom
singingwolfboy:upload-via-signed-url-extend-schema
Open

Extend schema to allow for presigned URLs for uploading files#108
singingwolfboy wants to merge 13 commits intographile:mainfrom
singingwolfboy:upload-via-signed-url-extend-schema

Conversation

@singingwolfboy
Copy link
Collaborator

@singingwolfboy singingwolfboy commented Feb 12, 2020

Follow-up to #107. This extends our GraphQL schema to expose a createUploadUrl mutation, which allows an API client to get a pre-signed URL for uploading a file to S3.

Copy link
Member

@benjie benjie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few comments, sorry about the delay - I've been busy on other projects.

@benjie
Copy link
Member

benjie commented Apr 20, 2020

CI is failing because aws-sdk was not added as a dependency of the relevant package.

Copy link
Member

@benjie benjie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for improving this PR 👍 I've left some more feedback.

@benjie
Copy link
Member

benjie commented May 15, 2020

I tried to get this into a mergeable state in #167 but I ran out of time.

@benjie benjie changed the base branch from master to main June 24, 2020 11:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants