-
Notifications
You must be signed in to change notification settings - Fork 76
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
4.7.0 to 4.7.1 causes InvalidValue error when importing code #666
Comments
Internal ticket number is FE-2847 |
Hi @carlpeaslee, thanks for bringing this up! I'll take a look and get back to you. |
@carlpeaslee Are you able to share code? |
@carlpeaslee this patch was motivated by the decision to validate that Client query arguments match existing typescript types. This just changes where the query gets denied, in the driver itself instead of as a 400 response from the database. In your case it's exposing the fact that your I'm not sure exactly how this would look without knowing more of your code and test framework. But it's a matter of waiting for one fauna query to finish before defining another query that depends on the first query's completion. For now we want to leave this change in as a patch, because it matches existing typescript definitions. Depending on your and other's feedback we would consider making this a major version change. |
thanks for the helpful responses. I will try to get back to you all soon with more code. |
I get the following error after upgrading from 4.7.0 to 4.7.1. The actual occurs when a test imports code that defines FQL that includes variables which reference other FQL. In this case
claim_links_id_unique.ref
=Index('claim_links_id_unique')
.This code runs fine (and actually isn't even executed by the test that fails since it's defining a UDF). My guess is that
faunadb
is doing some kind of pre-run static analysis and for whatever reason, because of the order of imports,claim_links_id_unique.ref
isundefined
at the timefaunadb
does its analysis?Also apologies, I didn't see a template for filing issues so let me know if there is additional information you'd like.
cc @thomas-franceschi
The text was updated successfully, but these errors were encountered: