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

[16.0][IMP] mrp_bom_report: Improve excess line error #505

Merged
merged 2 commits into from
Feb 6, 2025

Conversation

unaiberis
Copy link
Contributor

No description provided.

@unaiberis unaiberis force-pushed the 16.0-mrp_bom_report branch from e3735f3 to 335fa42 Compare February 5, 2025 09:42
@unaiberis
Copy link
Contributor Author

CLM11811

oihane
oihane previously requested changes Feb 5, 2025
Copy link
Contributor

@oihane oihane left a comment

Choose a reason for hiding this comment

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

Este módulo no depende del módulo que añade el layer y el designator. Por lo tanto no debería de imprimir esos campos.

@unaiberis unaiberis force-pushed the 16.0-mrp_bom_report branch 4 times, most recently from 8a4b7b8 to 6e75efb Compare February 5, 2025 10:16
@unaiberis
Copy link
Contributor Author

Se han cambiado los campos al report del módulo que los crea

@unaiberis unaiberis requested a review from oihane February 5, 2025 10:24
@unaiberis unaiberis force-pushed the 16.0-mrp_bom_report branch 3 times, most recently from 1a5dbbc to f0d89c4 Compare February 5, 2025 16:13
@oihane oihane force-pushed the 16.0-mrp_bom_report branch from f0d89c4 to 9750f79 Compare February 6, 2025 15:14
@oihane oihane merged commit 461a1bc into avanzosc:16.0 Feb 6, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants