Writing some view tests, I noticed we're not changing the status code when we hit an exception. Since this handler is catching all unhandled exceptions, we should really be sending a 500 with success=False.
Hopefully this will get cleaned up once we have some better exception handling in place, but for now this is a quick fix. I need this before I can check in an additional API test.
Writing some view tests, I noticed we're not changing the status code when we hit an exception. Since this handler is catching all unhandled exceptions, we should really be sending a 500 with success=False.
Hopefully this will get cleaned up once we have some better exception handling in place, but for now this is a quick fix. I need this before I can check in an additional API test.
/review @Mimimiel @williamflaherty