-
Notifications
You must be signed in to change notification settings - Fork 106
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
Recomendation #49
Comments
So here goes my code to whoever wants it: On wordpress:
Then on Laravel:
Result:
Previous code:
Result:
From my result just deconstruct to get your "special field" data. Cheers, |
Looks a good solution, have you run into any limitations? |
None, I now do whatever I want with the information. Cheers, |
@drosendo can you make this a PR, please? |
@jgrossi Maybe I don't know what you mean with "PR" (Pull request?), but there is nothing here to be added to your code... except the WordPress part, but this is a case to case... |
@drosendo this is great! Great thinking, Thanks! This was exactly what I needed 👍 |
I think there is nothing much to add here... feel free to suggest better ways of getting the structured data. |
Update for WPML support: On Wordpress:
Then on laravel:
Any recommendation on this better? At least it's working... Cheers, |
anyone to create a PR with that solution, please? 🙏🎉 |
@drosendo @jgrossi - I was hitting a similar performance issue (and have raised a seperate PR discussion to try to resolve it in the core library). For now, I have taken a similar approach as @drosendo - this one deep dives, recursively. get_fields() seems to resolved nested content. For example - if you have a repeater of Posts, it won't get the meta on the Posts.
|
After trying to use this plugin I really had issues with perfomance, querying like 15000 times due to my ACF being built in a massive block with option (imagine a Visual Composer but built with ACF).
Due to this the result is very slow.
So my approach will be much cleaner, and want some opinion on it.
What i'm going to do is on wordpress add an action (https://codex.wordpress.org/Plugin_API/Action_Reference/save_post), that when the user hits save:
I think this will be much faster and easy, opinions?
Cheers,
David
The text was updated successfully, but these errors were encountered: