-
Notifications
You must be signed in to change notification settings - Fork 8
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
New secure endpoints to obtain products and custom items and dooplugins update on save #224
New secure endpoints to obtain products and custom items and dooplugins update on save #224
Conversation
…ers/refactor-cleanning-api-fields
…ers/refactor-cleanning-api-fields
- Refactor custom attributes - Token auth
doofinder-for-woocommerce/includes/api/endpoints/class-endpoint-product.php
Outdated
Show resolved
Hide resolved
doofinder-for-woocommerce/includes/api/endpoints/class-endpoint-product.php
Outdated
Show resolved
Hide resolved
doofinder-for-woocommerce/includes/api/endpoints/class-endpoint-product.php
Outdated
Show resolved
Hide resolved
doofinder-for-woocommerce/includes/api/endpoints/class-endpoint-product.php
Outdated
Show resolved
Hide resolved
doofinder-for-woocommerce/includes/api/endpoints/class-endpoint-product.php
Show resolved
Hide resolved
doofinder-for-woocommerce/includes/api/endpoints/class-endpoint-product.php
Show resolved
Hide resolved
doofinder-for-woocommerce/includes/api/endpoints/class-endpoint-product.php
Show resolved
Hide resolved
doofinder-for-woocommerce/includes/api/endpoints/class-endpoint-custom.php
Outdated
Show resolved
Hide resolved
doofinder-for-woocommerce/includes/api/endpoints/class-endpoint-product.php
Show resolved
Hide resolved
doofinder-for-woocommerce/includes/api/endpoints/class-endpoint-product.php
Outdated
Show resolved
Hide resolved
doofinder-for-woocommerce/includes/api/endpoints/class-endpoint-product.php
Outdated
Show resolved
Hide resolved
doofinder-for-woocommerce/includes/api/endpoints/class-endpoint-product.php
Outdated
Show resolved
Hide resolved
doofinder-for-woocommerce/includes/api/endpoints/class-endpoint-product.php
Show resolved
Hide resolved
…ers/refactor-cleanning-api-fields
- Get custom attributes with get_slugs
- Fix thumbnails autogenerate - New param at instalation plugin
$img_ori = wp_get_attachment_image_src($thumbnail_id, "full"); | ||
$img_original["w"] = $img_ori[1] ?? 0; | ||
$img_original["h"] = $img_ori[2] ?? 0; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
quizás sacaría esto a otra función igual que tienes la de get_thumbnail_size
que fuera get_image_size
para que al leer la función se entienda mejor lo que se está queriendo comparar... pero es sólo una sugerencia!
* @param string $thumbnail_id Thumb ID por check | ||
* @return string URL of Thumb or false | ||
*/ | ||
private function check_size_image_url($thumbnail_id){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
quizás en vez de check_size_image_url
podríai ser get_image_url_by_size
? se me hace raro que sea check pero después devuelva un string o un falso..
…ers/refactor-cleanning-api-fields
In this release we have the following changes: