Skip to content

Navigation Menu

Sign in
Sign up

FlowFolder - #90

Open
dolymood wants to merge 9 commits into
flowjs:master from
dolymood:folder
Open

FlowFolder #90
dolymood wants to merge 9 commits into
flowjs:master from
dolymood:folder

Conversation

@dolymood

@dolymood dolymood commented Apr 7, 2015

Copy link
Copy Markdown
Contributor

When we upload a folder, may be we want to show the folder rather than the files in it.

So i created a new branch in https://github.com/dolymood/flow.js/tree/folder.I added FlowFolder constructor and the parsedFiles property to Flow object.

Maybe it is a good idea.

Or support an option like showFolder in the Flow options.

AidasK commented Apr 21, 2015

Copy link
Copy Markdown
Member

This looks like a big feature and well done, but I don't get the idea why it is needed.

Copy link
Copy Markdown
Contributor Author

A folder can be treated as an entry.
You can show a folder item in your page, then you can pause, resume or cancel the files by handle the folder item. This will be look like a "real" root directory which contains some files and have some file siblings.

Or the FolowFolder will be a plugin of flow.js and i will modify my files. 😄

evilaliv3 force-pushed the master branch 2 times, most recently from 360bba6 to b9e5421 Compare March 22, 2016 19:25

Copy link
Copy Markdown

Is this project dead or is the author ignoring any contributions? This is well over a year old and would be a really nice feature :(

AidasK commented May 31, 2016

Copy link
Copy Markdown
Member

I am not sure we need this feature. This pull request contains so many code changes, it would require some decent testing. For now, only two people requested this, so it is a low priority. It would be better to have this as a plugin

johaven commented Jul 9, 2017
edited
Loading

Copy link
Copy Markdown

It would be nice to have this feature, i often need to send folder with a lot of files (more than 1000 files) and it's difficult to follow the upload status of all these files.
I have no solution to track the files uploaded from one specific folder (i must use global flow progress which is no representative of my folder's files ...).
I must track error per file, connect to signal complete and filter files which are a parts of my folder ...
Display side, i use a subclass to freeze variables that will be modified during the next upload (FlowObj uploaded size, remaining time ...).
If we want to track the sending per folder it's currently very complex ! :)

Would there be a solution to create a new flow object dedicated to a folder?

AidasK commented Jul 10, 2017
edited
Loading

Copy link
Copy Markdown
Member

Currently I don't have time to merge this feature. I gave a quick peek and I think it needs some performance improvements. And what about angular? It might not be compatible. As @dolymood has mentioned, these changes can be moved to a separate plugin. I could merge it as a separate file. It does not need to be tested or to be perfect.

@AidasK AidasK reopened this Jul 10, 2017

johaven commented Jul 10, 2017

Copy link
Copy Markdown

Just one question a little bit off subject, I'm trying to see if I can create a second instance of the flow to dedicate it to a folder when it is detected by the first instance.
Apparently this is not possible, and I do not understand why! The Flow instance is a singleton?
Example:

import * as Flow from '@flowjs/flow.js'
const a = new Flow({...})
const b = new Flow({...})
b.custom_attribute = "test"
console.log(a.custom_attribute)
-> "test"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /