Inventory summary report

[tangollama]

@jkleinsc, I think this is ready to go.

[jkleinsc]

@tangollama Can you remove the commented out console.logs? Also, looks like there are some compile warnings you need to fix. See:
https://travis-ci.org/HospitalRun/hospitalrun-frontend/builds/79578002

[tangollama]

Yes I candor the console logs. As for the warnings, I really wasn't sure how to address those but I'll give it a go.

Typed or dictated imperfectly on my iPhone (717.385.9970)

On Sep 10, 2015, at 4:11 PM, John Kleinschmidt notifications@github.com wrote:

@tangollama Can you remove the commented out console.logs? Also, looks like there are some compile warnings you need to fix. See:
https://travis-ci.org/HospitalRun/hospitalrun-frontend/builds/79578002

—
Reply to this email directly or view it on GitHub.

[jkleinsc]

@tangollama the proper way to handle the errors about reject not being used was to use reject properly, not remove it completely. Regardless, I am going to merge this pull request and fix the issue myself.

[tangollama]

Ok thx. I'm happy to get appropriate instruction on that. I couldn't seem to find a context where it worked properly.

On Sep 11, 2015, at 9:35 AM, John Kleinschmidt notifications@github.com wrote:

@tangollama the proper way to handle the errors about reject not being used were to use reject properly, not remove it completely. Regardless, I am going to merge this pull request and fix the issue myself.

—
Reply to this email directly or view it on GitHub.

[jkleinsc]

@tangollama see 73a0138 for how I updated the code to use reject. Basically the concept is we bubble up the errors and display an error message at the top level of the promise chain.