Skip to content

Update supported node version to >= 4.0 - #1248

Merged
kornelski merged 1 commit into
forwardemail:masterfrom
jwgoh:update/engine
Jul 11, 2017
Merged

Update supported node version to >= 4.0#1248
kornelski merged 1 commit into
forwardemail:masterfrom
jwgoh:update/engine

Conversation

@jwgoh

@jwgoh jwgoh commented Jul 11, 2017

Copy link
Copy Markdown
Contributor

@kornelski
kornelski merged commit 7e19e77 into forwardemail:master Jul 11, 2017
@jwgoh
jwgoh deleted the update/engine branch July 12, 2017 03:17
@caicai0

caicai0 commented Aug 2, 2017

Copy link
Copy Markdown

I used it for loading.
var loadingUrl = [];
function load(url){
loadingUrl.push(url);
superagent.get(url).end(function(err,res){
loadingUrl.remove(url);
//other code
});
}

when I used this load() function too many times for diffrent urls.I find that there are away some urls in loadingUrl.I do not know why

@kornelski

Copy link
Copy Markdown
Contributor

@caicai0 Try using constructs like Promise.all() or use async/await.

However, please do not post in these comments, as they are about a pull request, which is totally unrelated to your problem.

@forwardemail forwardemail locked and limited conversation to collaborators Aug 2, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants