meteor-autoform-file icon indicating copy to clipboard operation
meteor-autoform-file copied to clipboard

Fix to UI not refreshed #97 and #100

Open x5engine opened this issue 9 years ago • 14 comments

I hope this get it fixed, I am not sure it's the correct syntax in coffee but the progress bar will work only for files with size more than 2MB...

try and tell me if it fixes it #97 and #100

x5engine avatar Mar 22 '16 17:03 x5engine

the fix almost works. change line 78 in autoform-file.coffee to this: (if doc then doc.isUploaded() else undefined) and doc._id

0x1d avatar Mar 23 '16 19:03 0x1d

@0x1d Thanks/merci Patrick for testing :) I did my best to get this working even if I am not used to coffeejs and you added the last touch for it awesome bro!

Well this should be merged ASAP @yogiben it works and fixes the progress bar issue ;)

Awesome

x5engine avatar Mar 23 '16 21:03 x5engine

i'm glad to help :) thanks for fixing this issue! hope this gets merged soon, along with some other pull requests (firefox open file dialog)

0x1d avatar Mar 23 '16 22:03 0x1d

@yogiben any chance someone integrating these changes? :+1:

RamyElkest avatar Apr 05 '16 22:04 RamyElkest

Yes yes :dancer:

x5engine avatar Apr 05 '16 22:04 x5engine

guess it's time to move to another package :/

RamyElkest avatar May 01 '16 22:05 RamyElkest

I'm tryin' out your package, it seems as though if you upload the images and delete it and try to upload another image the preview is gone.

Neobii avatar May 02 '16 08:05 Neobii

Okay, I'll admit I wasn't using the right branch before, but now using patch-1 I'm getting this error:

51c89b3368c6b.jpg:1 GET http://192.168.177.130:3000/cfs/files/images/Ne3E35Ho9AhQsPAn8/51c89b3368c6…VG9rZW4iOiJRS2d0Tk5mZktxRmtsSl9McnZuaHl5UUk4VmUxaS03enZmTW05QlQ2TkxQIn0%3D 503 (Service Unavailable)

But when I go to the link the image shows up, looks like some kind of race condition.

Neobii avatar May 19 '16 15:05 Neobii

It's actually crashing the server


W20160519-11:58:35.156(-4)? (STDERR) /home/user/.meteor/packages/cfs_gridfs/.0.0.33.xqd10f++os.linux.x86_32+os.linux.x86_64+web.browser+web.cordova/npm/node_modules/mongodb/lib/mongodb/connection/base.js:246
W20160519-11:58:35.157(-4)? (STDERR)         throw message;      
W20160519-11:58:35.157(-4)? (STDERR)               ^
=> Exited with code: 8
W20160519-11:58:35.171(-4)? (STDERR) Error: 573de2ab6bbd6d58200d9372 does not exist
W20160519-11:58:35.171(-4)? (STDERR)     at /home/user/.meteor/packages/cfs_gridfs/.0.0.33.xqd10f++os.linux.x86_32+os.linux.x86_64+web.browser+web.cordova/npm/node_modules/mongodb/lib/mongodb/gridfs/gridstore.js:200:22

Neobii avatar May 19 '16 15:05 Neobii

I'm not sure if it was related, but that error started happening when I was switching around autoform packages. What worked for me was dropping the cfs temp packages.

Neobii avatar May 19 '16 16:05 Neobii

Bump! This should be merged, I've tested this and have had no problems so far!

Neobii avatar May 26 '16 04:05 Neobii

Hey @yogiben, can you merge this please?

krishaamer avatar Jul 23 '16 20:07 krishaamer

Hey @yogiben, any chance this will be available at atmosphere soon?

suheb avatar Jul 30 '16 15:07 suheb