cohen990 / VoxPop

The voice of the people in pictures!
0 stars 0 forks source link

Images are too damn big #32

Closed cohen990 closed 9 years ago

cohen990 commented 9 years ago

If an image larger than ~50mb table storage cannot deal with it.

2 possible situations

  1. do not store base64 encoded image in the table (store as a byte stream instead)
  2. store images in blob storage and save a link to them in the table.
AJizzleBizzle commented 9 years ago

I don't think blob is quite sorted yet - still getting a Base64string in Table rather than a blob URI - plus get unhandled error for images >1mb. Maybe we didn't merge sundays work? I'm a bit confused

cohen990 commented 9 years ago

Is it possible that you forgot to do the remote update and pull upstream master?

cohen990 commented 9 years ago

Oh your right it's not in there. I'll take a look when I get to work

AJizzleBizzle commented 9 years ago

Awesome dude

cohen990 commented 9 years ago

you have a branch on your fork called blobstorage can you open a PR from there?

AJizzleBizzle commented 9 years ago

Er yh I think I've just done it

AJizzleBizzle commented 9 years ago

Fixed in a PR #37