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

Error when unknown currencies #213

Open
BenTen opened this issue May 17, 2024 · 4 comments
Open

Error when unknown currencies #213

BenTen opened this issue May 17, 2024 · 4 comments
Assignees

Comments

@BenTen
Copy link

BenTen commented May 17, 2024

Doofinder doesn't propose all currencies of the world.

Like #209, the module need to ignore unknown currencies.

After installed the module and logged to my Doofinder account, I received this message.

Screenshot 2024-05-17 at 14-12-29 Modules   Services

module 4.7.33
ps 1.7.5

@yuloma
Copy link
Contributor

yuloma commented Oct 22, 2024

Ahora mismo en todos los plugins se intenta crear por defecto los SE con todas las currencies de la tienda. Es desde Dooplugins que se controla las monedas aceptadas por Doofinder.

Para ignorar las monedas no soportadas y hacer la instalación con las otras se debe tocar directamente en Dooplugins y esto afectaría a todos los plugins.

@IgnacioPursalsZ tiramos esto adelante?

@IgnacioPursalsZ
Copy link

Lo que tendríamos que hacer es seguir mostrándoles el error, sin embargo, se deberían estar creando todos los SE/stores que sí sean compatibles.

De esta forma estaremos ayudando al equipo de soporte automatizando el proceso y que no tengan que invertir el tiempo en crear las que se podrían haber creado de forma automática.

Por otro lado los mensajes de error se tendrán que mejorar, ya que los actuales no son muy correctos, lo apuntamos como una tarea de producto.

  • Por el momento he podido ver:
    --> Prestashop:
    image (2) 12 39 47
    --> Shopify:
    323511736-da878e7d-9af3-4682-8685-8f2017546090

Me podría enseñar como se visualiza este mensaje de error en las demás por favor?

@sofia-doofinder
Copy link
Member

@IgnacioPursalsZ @yuloma estoy pensando que creo que esto no va a tener mucho sentido porque con el cambio del multi-price esto ya no va a pasar porque no utilizamos las monedas en el search engine, si no que se va a utilizar sólo a la hora de indexar.

@ogomezba me puedes confirmar si esto sería así? porque creo que el problema de la moneda se da sólo al configurarla en el search engine, pero para ponerla en la configuración de la tienda creo que no se mira que sea una moneda de las que nosotros consideramos como "válidas". Quizás a la hora de poner la moneda "por defecto" en el search engine podemos encontrar este problema, no?

Creo que esta tarea sería mejor hacerla una vez tengamos lo del multi-price implementado porque va a facilitar el trabajo creo.

@ogomezba
Copy link
Contributor

@sofia-doofinder Es correcto! Como comentas, con multi-currency el problema se reduciría a si la moneda default está aceptada o no porque esa es la que vamos a guardar en el SE. Y eso temporalmente, cuando esté más clara cómo va a quedar esta parte, quizá ni eso. Para lo que se mete en el mapeo de la Installation, realmente no se valida así que ahí no habría problema.

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

5 participants