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

Fix marketplace test issue #353

Merged
merged 6 commits into from
Nov 4, 2024
Merged

Conversation

yuloma
Copy link
Contributor

@yuloma yuloma commented Oct 31, 2024

A partir de la versión 0.14.9 ha comenzado a fallar la subida de versiones en Magento, al parecer es posible que hayan cambiado algo en el entorno de pruebas de ellos.

El caso es que falla al generar fixtures.

Después de verlo en conjunto todo el equipo, vimos que agregando una columna con nombre skip_entity_id_binding y con llave foránea que hace referencia a catalog_product_entity.entity_id.

Además había que quitar la restricción que no permites adicionar duplicados a la tabla nuestra.

Haciendo las pruebas me percaté que entonces el collection que se usa para procesar el Update on Save, viene con elementos duplicados, por lo cual he introducido un cambio para filtrar esos duplicados a la hora de procesar, pero eliminar todos los elementos como siempre.

@yuloma yuloma self-assigned this Oct 31, 2024
@yuloma yuloma linked an issue Oct 31, 2024 that may be closed by this pull request
Copy link
Contributor

@davidmolinacano davidmolinacano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but increase the Magento version in the etc/module.xml and in composer.json 🏆 🎖️

Copy link
Contributor

@brunovesar brunovesar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job!
I see that the foreign key was necessary after all.

etc/db_schema.xml Outdated Show resolved Hide resolved
Cron/Processor.php Outdated Show resolved Hide resolved
Copy link
Member

@sofia-doofinder sofia-doofinder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Te lo dejo aprobado, sólo un comentario que creo sería mejor para la legibilidad

Cron/Processor.php Outdated Show resolved Hide resolved
Copy link
Contributor

@brunovesar brunovesar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job!

@yuloma yuloma merged commit 0aeaa7b into master Nov 4, 2024
1 check passed
@yuloma yuloma deleted the yuloma/fix-marketplace-test-issue branch November 4, 2024 09:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Magento] Corregir error al subir al markeplace
4 participants