Skip to content
This repository was archived by the owner on Aug 30, 2026. It is now read-only.

fix(java): fix error message returned for invalid snapshot listener inequality filter - #1093

Merged
milaGGL merged 14 commits into
mainfrom
mila/fix-error-message-for-invalid-inequality-filter
Jan 24, 2023
Merged

milaGGL merged 14 commits into
mainfrom
mila/fix-error-message-for-invalid-inequality-filter

Conversation

@milaGGL

@milaGGL milaGGL commented Nov 7, 2022 •

Copy link
Copy Markdown
Contributor

Fix #1085
While using multiple inequality filter, inequality on different properties are rejected as expected, with minor difference in error message.

In a normal query, the error message will display all the properties in the inequality filter.
While in a snapshot listener, the error message will only display the second property used the inequality filter.

@product-auto-label product-auto-label Bot added the api: firestore Issues related to the googleapis/java-firestore API. label Nov 7, 2022
@milaGGL milaGGL changed the title Update ITSystemTest.java Fix error message returned for invalid snapshot listener inequality filter Nov 7, 2022
@product-auto-label product-auto-label Bot added the size: m Pull request size is medium. label Jan 3, 2023
@milaGGL milaGGL changed the title Fix error message returned for invalid snapshot listener inequality filter Fix: Fix error message returned for invalid snapshot listener inequality filter Jan 17, 2023
@milaGGL milaGGL changed the title Fix: Fix error message returned for invalid snapshot listener inequality filter fix(java): fix error message returned for invalid snapshot listener inequality filter Jan 17, 2023
@milaGGL
milaGGL marked this pull request as ready for review January 18, 2023 18:43
@milaGGL
milaGGL requested a review from a team January 18, 2023 18:43
@dconeybe
dconeybe self-requested a review January 18, 2023 18:47
@gcf-owl-bot
gcf-owl-bot Bot requested a review from a team January 23, 2023 20:32
@milaGGL
milaGGL requested a review from dconeybe January 24, 2023 02:14
@milaGGL milaGGL self-assigned this Jan 24, 2023
@milaGGL
milaGGL merged commit bf7f4a3 into main Jan 24, 2023
@milaGGL
milaGGL deleted the mila/fix-error-message-for-invalid-inequality-filter branch January 24, 2023 17:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: firestore Issues related to the googleapis/java-firestore API. size: m Pull request size is medium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug in error message from snapshot listener

2 participants