Skip to content
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

Fix term arrays for ID & TAXONOMY_ID #23

Merged
merged 1 commit into from
Nov 8, 2022

Conversation

iamlili
Copy link
Contributor

@iamlili iamlili commented Nov 12, 2020

When querying by field ID or TAXONOMY_ID the results were incorrect because it was only using the last element in the original array

example:
The following will only show posts that are in Category id 30 (it doesn't show those in 10 or 20)

posts( where: { taxQuery: { relation: AND taxArray: [ { terms: [10, 20, 30], field: ID, taxonomy: CATEGORY, operator: IN } ] } } )

When querying by field ID or TAXONOMY_ID the results were incorrect because it was only using the last element in the original array
@dgaidula
Copy link

Any chance we can this merged and a new release pushed out? I just got bit by this on a project and found this bug, but when I went to submit a pull request for the fix, I saw that this had already been submitted and approved.

@eliawk
Copy link

eliawk commented Jul 4, 2022

@jasonbahl could you please merge this?

@jasonbahl jasonbahl merged commit a59e805 into wp-graphql:develop Nov 8, 2022
@guvkon
Copy link

guvkon commented Nov 8, 2022

🎉

@jasonbahl jasonbahl mentioned this pull request Nov 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants