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

Entity Framework Dynamic LINQ support #79

Merged
merged 34 commits into from
Aug 24, 2024
Merged

Entity Framework Dynamic LINQ support #79

merged 34 commits into from
Aug 24, 2024

Conversation

ejsmith
Copy link
Contributor

@ejsmith ejsmith commented Mar 18, 2024

No description provided.

@CLAassistant
Copy link

CLAassistant commented Mar 18, 2024

CLA assistant check
All committers have signed the CLA.

@ejsmith ejsmith requested a review from niemyjski March 18, 2024 16:23
.gitignore Show resolved Hide resolved
Copy link
Member

@niemyjski niemyjski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, left some comments for improvement

ejsmith and others added 9 commits March 21, 2024 11:41
* test(sql): add tests for using navigations

* fix: correct tests to highlight navigation bug

* fix: add check if skip navigation is collection

* test: add checks to confirm skip navigations

* fix: respect ISkipNavigation#IsCollection
@ejsmith ejsmith merged commit 09a41f9 into master Aug 24, 2024
3 checks passed
@ejsmith ejsmith deleted the feature/sql branch August 24, 2024 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants