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

ValueObject persistence and Optional<T> #49

Open
jezzsantos opened this issue Aug 15, 2024 · 0 comments
Open

ValueObject persistence and Optional<T> #49

jezzsantos opened this issue Aug 15, 2024 · 0 comments
Labels
design defect Something does not work the best way, or is missing something good first issue Good for newcomers

Comments

@jezzsantos
Copy link
Owner

jezzsantos commented Aug 15, 2024

We are using our implementation of Optional<T> loosely derived from dotNetNext. We did that becuase it was uncertain at the time, whether this would make it into C# at all, and we didn't want the extra dependencies on the distributed library, not use some of the library classes.

We have some places int eh code where we are mapping between T? and Optional<T> and its very clunky.
We need more implicit conversations to make it more fluid code, and more discoverable for developers.

Secondly, ValudaObjectBase<T> dehydration is not really Optional<T> aware and that needs to change too.

@jezzsantos jezzsantos added good first issue Good for newcomers design defect Something does not work the best way, or is missing something labels Aug 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design defect Something does not work the best way, or is missing something good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant