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

track multiple concurrent requests #105

Merged
merged 3 commits into from
Dec 21, 2017
Merged

Conversation

miyura
Copy link

@miyura miyura commented Dec 12, 2017

No description provided.

@MurhafSousli
Copy link
Owner

MurhafSousli commented Dec 17, 2017

Hi @miyura don't you think it already tracks multiple concurrent requests?

@miyura
Copy link
Author

miyura commented Dec 17, 2017 via email

@miyura
Copy link
Author

miyura commented Dec 19, 2017

Hi @MurhafSousli, Do you have any suggestions to achieve this?

@MurhafSousli
Copy link
Owner

Hi @miyura
As far as I know, the finalize operator on the HttpHandler is triggered once all the current requests are done

return next.handle(req).pipe(finalize(() => {
    // all requests are done
 });

@miyura
Copy link
Author

miyura commented Dec 20, 2017

Hi @MurhafSousli, please try the "Test Http Client" in this stackblitz

https://stackblitz.com/edit/ngx-progressbar-j1wbsp

@MurhafSousli
Copy link
Owner

@miyura You are right, thanks a lot

@MurhafSousli MurhafSousli merged commit 7e75b5a into MurhafSousli:master Dec 21, 2017
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.

2 participants