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

Save API result with 404 answer to yotpo_rich_snippets cache table. #210

Open
sergei-sss opened this issue Nov 12, 2019 · 1 comment
Open
Assignees

Comments

@sergei-sss
Copy link

Currently only data from answer with status code "200" can be saved in yotpo_rich_snippets table:

                if ($res["status"] != 200) {
                    //product not found or feature disabled.
                    return [];
                }

If you'll save the products without reviews (404 status) in yotpo_rich_snippets table the quantity of API queries can be reduced and using the rich snippets API can be faster.

@yotpaul
Copy link

yotpaul commented Nov 17, 2019

@sergei-sss Thanks for the report! Let me check about this and get back to you.

@yotpaul yotpaul self-assigned this Nov 17, 2019
@DanGlz DanGlz added the v3.1.0 label Feb 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants