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

Fix to UI not refreshed #97 and #100 #103

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

x5engine
Copy link

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

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
@0x1d
Copy link

0x1d commented Mar 23, 2016

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

@x5engine
Copy link
Author

@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

@0x1d
Copy link

0x1d commented Mar 23, 2016

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)

@RamyElkest
Copy link

@yogiben any chance someone integrating these changes? 👍

@x5engine
Copy link
Author

x5engine commented Apr 5, 2016

Yes yes 💃

@RamyElkest
Copy link

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

@Neobii
Copy link

Neobii commented May 2, 2016

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
Copy link

Neobii commented May 19, 2016

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
Copy link

Neobii commented May 19, 2016

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
Copy link

Neobii commented May 19, 2016

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
Copy link

Neobii commented May 26, 2016

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

@krishaamer
Copy link

Hey @yogiben, can you merge this please?

@suheb
Copy link

suheb commented Jul 30, 2016

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

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.

6 participants