generated from mwc/lab_server
Add assessment
This commit is contained in:
parent
8e9bdd0991
commit
7d1dbb6323
|
@ -0,0 +1,22 @@
|
|||
# Server Lab Assessment
|
||||
|
||||
Pat,
|
||||
|
||||
Nice work! Everything looks good. I see in your commit message that you
|
||||
were trying to handle exceptions properly, but unfortunately this is a
|
||||
problem with Banjo's API views. For example, if you run:
|
||||
|
||||
$ http get http://127.0.0.1:5000/couplets/rhyme word=neverly
|
||||
|
||||
You'll get back a 400 error, as expected. The API view doesn't handle
|
||||
the exception like it should:
|
||||
|
||||
$ http get http://127.0.0.1:5000/api/couplets/rhyme word=neverly
|
||||
|
||||
And returns a 500 error. I'll fix this in the next release--sorry if
|
||||
it caused you some difficulty.
|
||||
|
||||
Your responses to questions in `notes.md` are thoughtful--I agree with
|
||||
everything :)
|
||||
|
||||
Chris
|
Loading…
Reference in New Issue