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

[Est:2] [Prestashop] Cambiar el uso de la API 5 por la 6 #208

Closed
sofia-doofinder opened this issue Apr 26, 2024 · 2 comments · Fixed by #235
Closed

[Est:2] [Prestashop] Cambiar el uso de la API 5 por la 6 #208

sofia-doofinder opened this issue Apr 26, 2024 · 2 comments · Fixed by #235
Assignees
Labels
pluginRangers versioning Tech debt | Keeping infra, dependencies and the code updated

Comments

@sofia-doofinder
Copy link
Contributor

Parece que prestashop está usando la versión de la API de doofinder v5 en lugar de la v6.
https://github.com/doofinder/doofinder-prestashop/blob/master/lib/doofinder_api.php#L35

Esto parece que actualmente se utiliza para las landing pages, pero debería modificarse para utilizar la v6.

Es posible que se estuviera utilizando la v5 porque es la que se empezó a utilizar en el plugin y por algún motivo nunca se ha actualizado. Antes se utilizaba para el update on save y la búsqueda nativa, pero el update on save ahora se hace a través de dooplugins y la búsqueda nativa se quitó del plugin, por lo que sólo se debe de utilizar en la landing pero habría que revisarlo.

@sofia-doofinder sofia-doofinder added pluginRangers versioning Tech debt | Keeping infra, dependencies and the code updated labels Apr 26, 2024
@sofia-doofinder sofia-doofinder changed the title [Prestashop] Cambiar el uso de la API 5 por la 6 [Est:2] [Prestashop] Cambiar el uso de la API 5 por la 6 Jul 30, 2024
@eduardogomez97
Copy link
Member

He modificado a la v6 y he testeado que todo funcione como es debido. En principio no da problemas.

Image

Solo se utiliza además para hacer un test de que funciona la propia API. $this->testDoofinderApi();

@eduardogomez97 eduardogomez97 linked a pull request Aug 23, 2024 that will close this issue
@eduardogomez97
Copy link
Member

Ya está publicada la nueva versión y funcionando!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pluginRangers versioning Tech debt | Keeping infra, dependencies and the code updated
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants