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

Blob #26

Merged
merged 4 commits into from
Nov 15, 2019
Merged

Blob #26

merged 4 commits into from
Nov 15, 2019

Conversation

taufiq
Copy link
Contributor

@taufiq taufiq commented Nov 15, 2019

This PR will allow one to retrieve/upload larger images using Git blobs. (#18)

The commit edac009 addresses the retrieving issue.
The commit 54a41a3 addresses the uploading issue.

An error is thrown when using `res.render()` in the error handler
function. The error returned is `no default engine was specified and no
extension was provided`. The solution was to use `res.json()`
A size limit of 1 MB is imposed for the file retrieval from the Github "Contents" API.  Usually, the files in the Isomer repo don't hit that size limit except for images where they tend to be bigger than 1MB.

To get around this, the blobs API  supports file sizes of of up to 100MB.
You can now send json payloads of upto 5mb
@taufiq taufiq requested a review from kwajiehao November 15, 2019 08:11
Copy link
Contributor

@kwajiehao kwajiehao left a comment

Choose a reason for hiding this comment

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

AG LGTM GG

@kwajiehao kwajiehao merged commit a9f1f2a into master Nov 15, 2019
@kwajiehao kwajiehao deleted the blob branch November 15, 2019 08:12
harishv7 pushed a commit that referenced this pull request Feb 17, 2023
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