detecting when files have uploaded and other events #891

Open
opened 2017-09-22 20:50:14 +00:00 by figuerres · 6 comments
figuerres commented 2017-09-22 20:50:14 +00:00 (Migrated from github.com)

it seems like this could use some events to notify the calling code of the status changes.

kind of like @Output() onVoted = new EventEmitter();
https://angular.io/guide/component-interaction

i am not sure how to best contribute the code for this, i think i can take the code an make a localversion but then no one else gets the benefit.

so that in a component it can get an event in a function to handle things like file x was uploaded take it out of the queue display and update my grid of files on the server.

is anyone working on this kind of update ?
does anyone have a fork where they did it ?

it seems like this could use some events to notify the calling code of the status changes. kind of like @Output() onVoted = new EventEmitter<boolean>(); https://angular.io/guide/component-interaction i am not sure how to best contribute the code for this, i think i can take the code an make a localversion but then no one else gets the benefit. so that in a component it can get an event in a function to handle things like file x was uploaded take it out of the queue display and update my grid of files on the server. is anyone working on this kind of update ? does anyone have a fork where they did it ?
adrianfaciu commented 2017-10-01 10:02:03 +00:00 (Migrated from github.com)

If you have a local version of this a PR would be awesome 😄

If you have a local version of this a PR would be awesome 😄
AckerApple commented 2017-10-02 18:53:23 +00:00 (Migrated from github.com)

I have finished addressing this in v0.2.0 of angular-file

Recommend moving to my fork for the requested functionality

I have finished addressing this in v0.2.0 of [angular-file](https://www.npmjs.com/package/angular-file) Recommend moving to my fork for the requested functionality
adrianfaciu commented 2017-10-02 19:28:35 +00:00 (Migrated from github.com)

@AckerApple there are still a lot of people using this in their applications, already in production. At least some of them would like to keep using it, as it might not be that easy for them to upgrade to something else.

@AckerApple there are still a lot of people using this in their applications, already in production. At least some of them would like to keep using it, as it might not be that easy for them to upgrade to something else.
AckerApple commented 2017-10-02 19:34:06 +00:00 (Migrated from github.com)

@figuerres reported this same issue on my fork: https://github.com/AckerApple/angular-file/issues/2

Of course people have this package in production and may want to keep using it. Doesn't mean I shouldn't be helpful and mention the functionality they seek is available elsewhere.

If it bothers you that I'm all up in your package, it wouldn't be the first time I've heard that. I have not been blocked by a package, nor am I trolling... Doing what I think is helpful.

@figuerres reported this same issue on my fork: https://github.com/AckerApple/angular-file/issues/2 Of course people have this package in production and may want to keep using it. Doesn't mean I shouldn't be helpful and mention the functionality they seek is available elsewhere. If it bothers you that I'm all up in your package, it wouldn't be the first time I've heard that. I have not been blocked by a package, nor am I trolling... Doing what I think is helpful.
adrianfaciu commented 2017-10-02 20:02:09 +00:00 (Migrated from github.com)

Did not suggest you are trolling or bothering someone, just pointed out a scenario for some people, including myself. I'm not directly related to valour-software, but since I'm using this package I'm trying to give a hand to keep it more updated 😄

Did not suggest you are trolling or bothering someone, just pointed out a scenario for some people, including myself. I'm not directly related to valour-software, but since I'm using this package I'm trying to give a hand to keep it more updated 😄
AckerApple commented 2017-10-02 20:03:17 +00:00 (Migrated from github.com)

Yo extra thanks for maintaining this package. Big fan.

Understood.

Yo extra thanks for maintaining this package. Big fan. Understood.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: dc/ng2-file-upload#891