doofinder / doofinder-magento2

Open Software License 3.0
10 stars 8 forks source link

Try implementing interface #320

Closed sofia-doofinder closed 2 months ago

sofia-doofinder commented 2 months ago

El PR que nos propuso magento2 instanciando cada una de las instancias que necesitamos en vez de usar el constructor no ha funcionado porque no pasa la validación. image

Tras comentarlo en la issue que le abrimos a magento nos han propuesto que implementemos la interfaz en vez de extender la clase: https://github.com/magento/magento2/issues/38669

He estado haciendo pruebas y la indexación funciona, pero no se si esta forma le va a gustar a magento y no lo podemos saber hasta que subamos la versión al validador.