Closed
Bug 1026934
Opened 11 years ago
Closed 11 years ago
Handle errors the same way accross all services.
Categories
(Hello (Loop) :: Server, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: alexis+bugs, Assigned: rhubscher)
Details
(Whiteboard: [qa+])
Attachments
(2 files)
https://github.com/mozilla-services/msisdn-gateway/issues/53 reuses the way we handle errors for other mozilla's services, we should use this for loop as well.
|
Assignee | |
Updated•11 years ago
|
Assignee: nobody → rhubscher
|
|
Assignee | |
Comment 2•11 years ago
|
||
Attachment #8451686 -
Flags: review?(alexis+bugs)
|
|
Assignee | |
Updated•11 years ago
|
Status: NEW → ASSIGNED
|
|
Reporter | |
Comment 3•11 years ago
|
||
This is okay for the code: https://github.com/mozilla-services/loop-server/commit/d6b530c388a92333ac5c2b6e868dffc1cdab75b4
But misses a doc update.
|
|
Reporter | |
Updated•11 years ago
|
Attachment #8451686 -
Flags: review?(alexis+bugs) → review+
|
|
Assignee | |
Comment 4•11 years ago
|
||
Attachment #8452321 -
Flags: review?(alexis+bugs)
|
|
Reporter | |
Comment 5•11 years ago
|
||
Comment on attachment 8452321 [details] [review]
Link to github PR
r+ with nits.
Attachment #8452321 -
Flags: review?(alexis+bugs) → review+
|
|
Assignee | |
Comment 6•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•