Skip to content

Conversation

@JiriHoffmann
Copy link
Contributor

@JiriHoffmann JiriHoffmann commented Nov 20, 2025

Description

This PR removes a query modifiers check that is no longer necessary. This check is preventing inequality filters from being combined with orderBy filters if they are applied to a different field.

Related issues

Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
    • Yes
  • My change supports the following platforms;
    • Android
    • iOS
    • Other (macOS, web)
  • My change includes tests;
    • e2e tests added or updated in packages/\*\*/e2e
    • jest tests added or updated in packages/\*\*/__tests__
  • I have updated TypeScript types that are affected by my change.
  • This is a breaking change;
    • Yes
    • No

Think react-native-firebase is great? Please consider supporting the project with any of the below:

🔥

@vercel
Copy link

vercel bot commented Nov 20, 2025

@JiriHoffmann is attempting to deploy a commit to the Invertase Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Collaborator

@mikehardy mikehardy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this! Really appreciated. The testing, specifically, is great to see

This limit did exist at one point but I verified that in the current set of orderBy limitations the only thing remaining is that it will filter out / only return docs where the orderBy key exists

https://firebase.google.com/docs/firestore/query-data/order-limit-data#limitations
https://firebase.google.com/docs/firestore/query-data/queries#query_limitations
https://firebase.google.com/docs/firestore/query-data/queries#orderby_and_existence

@mikehardy mikehardy added Workflow: Pending Merge Waiting on CI or similar and removed Workflow: Pending Merge Waiting on CI or similar labels Nov 21, 2025
@mikehardy mikehardy merged commit b388022 into invertase:main Nov 21, 2025
18 of 19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants