-
Notifications
You must be signed in to change notification settings - Fork 742
Update query-over.md #9735
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
Update query-over.md #9735
Conversation
I tried to make it understandable but the terminology and the wording could be a bit off. |
One more thing. The only real issue here is that originally instead of But if you think the original wording is fine then you can also only replace |
Thanks - I'm having a look now, but trying to read the whole document so I have some context. |
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.
Thanks for the PR and the explanation of why it is needed.
I think I understand the issue and have made a couple of comments below. Let me know if you think they make sense.
content/en/docs/refguide/modeling/domain-model/associations/query-over.md
Outdated
Show resolved
Hide resolved
content/en/docs/refguide/modeling/domain-model/associations/query-over.md
Show resolved
Hide resolved
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.
Thanks for the PR.
Approving and merging.
No description provided.