-
Notifications
You must be signed in to change notification settings - Fork 892
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
in
comparator: Bug Or Documentation Error?
#7029
Comments
I couldn't figure out how to label this issue, so I've labeled it for a human to triage. Hang tight. |
Hi @tylim88, thank you for reporting this issue. I reproduced the issue and have created an internal ticket to track it. I will keep you updated on the progress. Google internal users please see it here: b/269160257. |
thanks @milaGGL // =================// I would like to share my thoughts about this issue I don't think this is a bug, I think the current behavior is reasonable not only the current behavior is easier to reason with, it actually can do what the doc want it to do this is how the correct clause look: plus so I think they work fine and I like the way they work now, they are much easier to understand I WANT to believe this is simply documentation error that exist for long time (you guys never fix your documentation despite feedbacks) and they always work like this all these times anyway, you guys do your stuff, this is just my opinion |
I tested the same thing with old firebase sdk 9.6.0 and 9.0.0 (a year ago) The behavior are the same, it seem like the documentation is wrong all these times |
Hi @tylim88. There is an internal ticket sent to documentation team, and I will keep you posted on any further updates. Thank you again for looking into the documentations and raising this issue. |
according to https://firebase.google.com/docs/firestore/query-data/queries#array-contains-any
However, after some testing, this statement is not true, it returns empty result
After some experiments, I found out that below statement is also incorrect
it should be
array that contains EXACTLY west coast AND east coast
another thing is the comparator
not-in
also works with array type likein
in above example, you can do something likequery(citiesRef, where('regions', 'not-in', [['west_coast', 'east_coast']]))
but is not documented, I think you should add it, seriouslyif this is documentation error, don't ask me to submit feedback on the documentation site because you people never change anything and I have no record of what I have submitted. If I leave an issue instead, at least I can look back what is wrong
The text was updated successfully, but these errors were encountered: