Skip to content

[17.0][IMP] bc3_importer#43

Open
edescalona wants to merge 12 commits intoOCA:17.0from
BinhexTeam:17.0-imp-bc3_importer
Open

[17.0][IMP] bc3_importer#43
edescalona wants to merge 12 commits intoOCA:17.0from
BinhexTeam:17.0-imp-bc3_importer

Conversation

@edescalona
Copy link

@edescalona edescalona commented Aug 26, 2025

@BinhexTeam

This PR allows migrating the improvement made in [14.0] bc3_importer to [17.0] bc3_importer

This implementation is intended to print measurement lines, because order lines require more information.

The PRS were used as a base:

#40
#41

@edescalona edescalona marked this pull request as ready for review August 26, 2025 16:57
@edescalona
Copy link
Author

edescalona commented Sep 16, 2025

Hi @AlvaroRM11 @Christian-RB @szalatyzuzanna @mymage , if you could do a review of this improvement, thanks.

@christian-ramos-tecnativa

Hi @edescalona, maybe it’s better to add those improvements directly to PR #40?
Since this PR is marked as an IMP, merging it could also bring in the migration changes from #40 and cause conflicts down the line.

@jelenapoblet
Copy link

jelenapoblet commented Oct 29, 2025

What about if we focus on merging #40 and then push this imp ? @szalatyzuzanna coud you review the PR on the migration please ?

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.

8 participants