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
feat: Hydrate CustomError and Result. #344
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @yeldiRium 👋
I finally had the time to review your changes. Please find my notes inline 😊
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work, I like the idea. I have some questions regarding the details though, if you could please take a look @yeldiRium 🙂
5e3c9b6
to
e0d30e4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
# [8.2.0](8.1.0...8.2.0) (2022-02-22) ### Features * Hydrate CustomError and Result. ([#344](#344)) ([3f4343c](3f4343c))
RFC @dotKuro @goloroden @strangedev
As discussed with @strangedev via slack, I needed a way to hydrate Errors and Results that were JSON-serialized before. That means multiple things:
Result<undefined, ...>
is represented in JSON as an empty objectPlease take a look at the implementations and the documentation and reasoning for them and leave some feedback on wether you think this solution makes sense. Until I get some feedback I'll leave this as a draft.