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

If partition or row key expressions have complex lambda, property name should be null #100

Closed
kzu opened this issue May 17, 2022 · 0 comments · Fixed by #101
Closed

If partition or row key expressions have complex lambda, property name should be null #100

kzu opened this issue May 17, 2022 · 0 comments · Fixed by #101
Labels
bug Something isn't working

Comments

@kzu
Copy link
Member

kzu commented May 17, 2022

For example:

        var repo = TableRepository.Create<Show>(CloudStorageAccount.DevelopmentStorageAccount,
            "Shows",
            x => $"{x.FromDate.Year}",
            x => $"{x.FromDate:yyyy-MM-dd}-{x.Id}");

In this case, the partition key should not be assumed to be FromDate since we're actually using just the year and that would fail to deserialize the object (if it had a FromDate constructor/property of type DateOnly).

Back this issue
Back this issue

@kzu kzu added the bug Something isn't working label May 17, 2022
kzu added a commit that referenced this issue May 17, 2022
Otherwise, we can end up breaking up serialization.

Fixes #100
@kzu kzu closed this as completed in #101 May 17, 2022
kzu added a commit that referenced this issue May 17, 2022
Otherwise, we can end up breaking up serialization.

Fixes #100
@devlooped devlooped locked and limited conversation to collaborators Oct 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant