vauxoo-dev / gist-vauxoo

All our gist files into organization
6 stars 7 forks source link

[ADD] assign_mx_uuid_to_moves.py: Add server action to assign l10n_mx_edi_cfdi_uuid T#70525 #133

Closed rolandojduartem closed 10 months ago

rolandojduartem commented 1 year ago

Moves comes with l10n_mx_edi_cfdi_uuid empty because there is no filestore in the migration process, so it is required to apply this code as a server action.

Co-authored-by: Luis Manuel luis.manuel@vauxoo.com

rolandojduartem commented 1 year ago

Hi, @luisg123v, please, could you review this PR? in the case of reynera, the search returned 40218 records and the whole server action took 1 minute and 16.26 seconds without any batch.

cc @desdelinux

luisg123v commented 1 year ago

I'd recommend a shorter name, e.g. assign_mx_uuid_to_moves

rolandojduartem commented 1 year ago

Hi, @luisg123v, please, could you review this PR?

cc @desdelinux @LuisMzz

rolandojduartem commented 12 months ago

Hi, @luisg123v, I applied your suggestions, please, could you review this PR

rolandojduartem commented 11 months ago

@luisg123v friendly reminder

rolandojduartem commented 11 months ago

@luisg123v ready!

rolandojduartem commented 11 months ago

@luisg123v friendly reminder

rolandojduartem commented 11 months ago

LGTM 👍

@rolandojduartem is this expected to be in draft status?

Yes, I just have to modify one line

rolandojduartem commented 10 months ago

Hi, @luisg123v, now this is ready.

rolandojduartem commented 10 months ago

@luisg123v friendly reminder