-
-
Notifications
You must be signed in to change notification settings - Fork 541
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
Is it possible to sort by nested fields? #105
Comments
Seems like a bug. Do you wanna make a PR? :) |
Maybe, could you point me in the right direction?
…On Mon, Jun 17, 2019 at 3:29 PM Michael Yali ***@***.***> wrote:
Seems like a bug. Do you wanna make a PR? :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#105?email_source=notifications&email_token=AGEBMKJG6PNXPS45DGFZFBTP27QYHA5CNFSM4HYYG4T2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODX4GRLA#issuecomment-502818988>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AGEBMKLYHM6C32GVIHVM6UTP27QYHANCNFSM4HYYG4TQ>
.
--
--
Chris Marx
Senior Software Engineer
ZevRoss - Know Your Data
Data Science & Spatial Analysis
209 N. Aurora St, 2nd Floor
Ithaca, NY 14850
www.zevross.com
|
crud/packages/crud-typeorm/src/typeorm-crud.service.ts Lines 571 to 577 in f3d3c58
|
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
It seems like maybe this should be possible, if I make a crud request like:
It generates a query with this sort:
Which is the correct entity relationship, but not the correct way to generate the sql (it doesn't need that parent table reference to "Death")
The text was updated successfully, but these errors were encountered: