argonaut icon indicating copy to clipboard operation
argonaut copied to clipboard

DecodeResult should preserve its history in both failure and success cases

Open chrislewis opened this issue 9 years ago • 4 comments

This will require changing the current structure of DecodeResult to DecodeResult[A](history: CursorHistory, result: String \/ A).

chrislewis avatar Aug 24 '14 21:08 chrislewis

Is there something I can do to assist with this? Lack of preserved history led to 10 days of debugging for us as any issues during decode only ever resulted in a generic None.get exception, something like this would have reduced that down to 4 hours worth of exception checking. For posterity I'd like to help implement this if not already done to save future users from falling into this rut our team did.

tylerjharden avatar Mar 18 '16 20:03 tylerjharden

@tylerjharden Please feel free to look into implementing it, I can provide assistance if necessary.

seanparsons avatar Mar 18 '16 21:03 seanparsons

Argonaut started off that way, but all sorts of useful properties are subsequently lost. On 19/03/2016 8:27 AM, "Sean Parsons" [email protected] wrote:

@tylerjharden https://github.com/tylerjharden Please feel free to look into implementing it, I can provide assistance if necessary.

— You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub https://github.com/argonaut-io/argonaut/issues/134#issuecomment-198547509

tonymorris avatar Mar 18 '16 22:03 tonymorris

Finding myself wanting this a couple times now, I'm curious if anyone can shed some light on exactly which properties are lost? Failing that maybe I can try implementing it myself and seeing what walls I run into..

adelbertc avatar Aug 30 '16 06:08 adelbertc