search

cardiacsociety / web-services

MappCPD API and other services
0 stars 1 forks source link

Handle errors from go routine that generate excel reports #51

Closed mikedonnici closed 5 years ago

mikedonnici commented 5 years ago

Errors are only logged to stdout.

Better to send an error back through a channel and maybe display on the report itself.

This would be particularly useful if there was an error with a single row that was non-critical.

mikedonnici commented 5 years ago

This is a bit curly so another option might be to add an error tab to the actual spreadsheet and stick each logged error into a row on that tab, maybe with the primary id in the first column, and the message in the second.

mikedonnici commented 5 years ago

Added error tab to excel workbook which will list all the errors found and the relevant record id.

Only added to member and journal report at this stage.