-
Notifications
You must be signed in to change notification settings - Fork 122
Ignore location rules when adding fields to the schema. Fixes #52. #82
base: master
Are you sure you want to change the base?
Conversation
@peterjanes I believe this will cause that filter to always return true, regardless of using within WPGraphQL or not. Can you describe what this solves so we can make sure we're not filtering too heavily? |
You're right, it does show all of the fields all the time. I'll figure out how to detect whether it's a query in GraphQL, or move it to a more appropriate place. (My original branch name was The goal is to show all ACF fields in the schema, even if they have a location rule that only displays them on an admin screen when a particular condition is met. For example, I have a number of field groups that are only displayed when a post's category is set to a specific value, and I want to be able to query them when I display that post. [edit: just noticed that "fixes #52" in the description doesn't link to #52, so I'll do that here.] |
Fixed allowed post type check
- Update version number
…matting selection
…ut-causes-issues #91 - Don't add flex fields to the Schema if there are no layouts configured
Implement graphql_field_name for fields
#93 - apply autop and nl2br on textarea fields depending on their formatting selection
added in ACF 5.8.6
Support fields for Google Maps field added in ACF 5.8.6
- Update Composer dependencies
Release/v0.3.1
+1 for this one, With
It's ok. But with
All my fields is never registered. We can't use the power of "set location" |
This hasn't obviously broken anything in my installation, but I haven't done exhaustive testing on it either.