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

Mai Post Grid -- testimonial categories not displaying/saving #93

Open
shay328 opened this issue May 15, 2020 · 7 comments
Open

Mai Post Grid -- testimonial categories not displaying/saving #93

shay328 opened this issue May 15, 2020 · 7 comments
Labels
bug Something isn't working

Comments

@shay328
Copy link

shay328 commented May 15, 2020

As of today, using the Mai Post Grid block, the testimonial post type stopped displaying/saving the taxonomy by term.

Image 2020-05-14 at 8 28 32 PM

https://dev.meaganwilliamson.com/contact/

@JiveDig
Copy link
Contributor

JiveDig commented May 15, 2020

Do you get an error or anything when you try to save?

So the taxonomy and terms are available to set but it won’t save and therefore won’t display?

@shay328
Copy link
Author

shay328 commented May 15, 2020

No error. it seems like i'm grabbing the terms, but the front end only ever displays all categories.

Side note, I think we need a byline parameter for testimonials. I don't know if I brought that up yet.

*edit: also are sliders available for the grid block?

@JiveDig
Copy link
Contributor

JiveDig commented May 15, 2020

This is def a bug. I can reproduce. Fixing now.

@JiveDig
Copy link
Contributor

JiveDig commented May 15, 2020

Argh front end fixed but still doesn't load correct taxos in backend.

@JiveDig JiveDig reopened this May 15, 2020
@JiveDig
Copy link
Contributor

JiveDig commented May 15, 2020

Update for myself...

mai_get_acf_request( 'post_type' ) is not returning the saved post type in mai_get_post_type_taxonomy_choices() function. WTH.

@JiveDig JiveDig added the bug Something isn't working label May 19, 2020
@JiveDig
Copy link
Contributor

JiveDig commented Dec 12, 2024

This came up again somehow, I think since 2.35.0.

@JiveDig JiveDig reopened this Dec 12, 2024
@JiveDig JiveDig reopened this Dec 12, 2024
@JiveDig
Copy link
Contributor

JiveDig commented Dec 12, 2024

Making a note that this now only seems to happen when the block is saved while in Edit mode. v2.35.0 limits most of our queries to only happen when doing ajax, but in Edit mode it's on page load.

The fix is to add acf/prepare_field filter for every field we're doing heavy queries, and only load the option that is the current $field['value']. This is a light query on page load, and uses the full query once the user clicks the field to initiate the ajax loading.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants