Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DecodeResult should preserve its history in both failure and success cases #134

Open
chrislewis opened this issue Aug 24, 2014 · 4 comments

Comments

@chrislewis
Copy link

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

@tylerjharden
Copy link

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.

@seanparsons
Copy link
Member

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

@tonymorris
Copy link
Member

Argonaut started off that way, but all sorts of useful properties are
subsequently lost.
On 19/03/2016 8:27 AM, "Sean Parsons" notifications@github.com 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
#134 (comment)

@adelbertc
Copy link
Contributor

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..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants