Looks good but I've suggested a code simplification. No need to re-review.
https://codereview.appspot.com/6858099/diff/5001/app/app.js File app/app.js (right):
https://codereview.appspot.com/6858099/diff/5001/app/app.js#newcode325 app/app.js:325: .some() returns true or false so I think you could just do:
var updateNeeded = this.db.services.some(function(service) { return (self.serviceEndpoints[service.get('id')] === undefined)});
https://codereview.appspot.com/6858099/
« Back to merge proposal
Looks good but I've suggested a code simplification. No need to
re-review.
https:/ /codereview. appspot. com/6858099/ diff/5001/ app/app. js
File app/app.js (right):
https:/ /codereview. appspot. com/6858099/ diff/5001/ app/app. js#newcode325
app/app.js:325:
.some() returns true or false so I think you could just do:
var updateNeeded = this.db. services. some(function( service) { dpoints[ service. get('id' )] ===
return (self.serviceEn
undefined)});
https:/ /codereview. appspot. com/6858099/