Skip to content

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

Open
x5engine wants to merge 2 commits into
yogiben:masterfrom
x5engine:patch-1
Open

Fix to UI not refreshed #97 and #100#103
x5engine wants to merge 2 commits into
yogiben:masterfrom
x5engine:patch-1

Conversation

@x5engine

Copy link
Copy Markdown

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

0x1d commented Mar 23, 2016

Copy link
Copy Markdown

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

0x1d commented Mar 23, 2016

Copy link
Copy Markdown

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

@yogiben any chance someone integrating these changes? 👍

@x5engine

x5engine commented Apr 5, 2016

Copy link
Copy Markdown
Author

Yes yes 💃

@RamyElkest

Copy link
Copy Markdown

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

@Neobii

Neobii commented May 2, 2016

Copy link
Copy Markdown

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

Neobii commented May 19, 2016

Copy link
Copy Markdown

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

Neobii commented May 19, 2016

Copy link
Copy Markdown

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

Neobii commented May 19, 2016

Copy link
Copy Markdown

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

Neobii commented May 26, 2016

Copy link
Copy Markdown

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

@krishaamer

Copy link
Copy Markdown

Hey @yogiben, can you merge this please?

@suheb

suheb commented Jul 30, 2016

Copy link
Copy Markdown

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

@x5engine

Copy link
Copy Markdown
Author

Hey guys why update on this? Still waiting 😂

@x5engine

Copy link
Copy Markdown
Author

Yo guys any news on this? What's the ETA?

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