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
qe: fix prisma/14001 #3185
qe: fix prisma/14001 #3185
Conversation
81d4136
to
d4c474a
Compare
d4c474a
to
d138cc3
Compare
d138cc3
to
23b40ab
Compare
// Issue: https://github.com/prisma/prisma/issues/14001 | ||
// Here we can assume the field name is ASCII, because it is the _client_ field name, | ||
// not the mapped name. | ||
left[..1].make_ascii_lowercase(); |
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.
Sorry, I don't understand what left[..1]
is doing here?
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.
It just makes the first character of the left
String lowercase (it takes a mutable slice of just the first character, then makes the whole slice lowercase). I'll add a comment to make that less obscure.
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.
ok thanks.
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.
Could you ask someone from the typescript side to add a test for this in the client. Once that is done we can look to merge this.
Added the tests here prisma/prisma#15327 |
internal spec: https://www.notion.so/prismaio/MongoDB-aggregation-pipeline-variable-names-e6e5c660d39141c28d57040e36414b24
closes prisma/prisma#14001